Skip to content

Commit

Permalink
---
Browse files Browse the repository at this point in the history
yaml
---
r: 318903
b: refs/heads/master
c: 160e7c8
h: refs/heads/master
i:
  318901: 2ed53d3
  318899: 825d9bd
  318895: c1a0fbb
v: v3
  • Loading branch information
Ohad Ben-Cohen committed Jul 5, 2012
1 parent c184c24 commit 59de528
Show file tree
Hide file tree
Showing 5 changed files with 33 additions and 33 deletions.
2 changes: 1 addition & 1 deletion [refs]
Original file line number Diff line number Diff line change
@@ -1,2 +1,2 @@
---
refs/heads/master: 40e575b1d0b34b38519d361c10bdf8e0c688957b
refs/heads/master: 160e7c840fe85836040c43e0058d5afced470c85
18 changes: 9 additions & 9 deletions trunk/Documentation/remoteproc.txt
Original file line number Diff line number Diff line change
Expand Up @@ -90,21 +90,21 @@ int dummy_rproc_example(struct rproc *my_rproc)
This function should be used by rproc implementations during
initialization of the remote processor.
After creating an rproc handle using this function, and when ready,
implementations should then call rproc_register() to complete
implementations should then call rproc_add() to complete
the registration of the remote processor.
On success, the new rproc is returned, and on failure, NULL.

Note: _never_ directly deallocate @rproc, even if it was not registered
yet. Instead, when you need to unroll rproc_alloc(), use rproc_free().
yet. Instead, when you need to unroll rproc_alloc(), use rproc_put().

void rproc_free(struct rproc *rproc)
void rproc_put(struct rproc *rproc)
- Free an rproc handle that was allocated by rproc_alloc.
This function essentially unrolls rproc_alloc(), by decrementing the
rproc's refcount. It doesn't directly free rproc; that would happen
only if there are no other references to rproc and its refcount now
dropped to zero.

int rproc_register(struct rproc *rproc)
int rproc_add(struct rproc *rproc)
- Register @rproc with the remoteproc framework, after it has been
allocated with rproc_alloc().
This is called by the platform-specific rproc implementation, whenever
Expand All @@ -117,15 +117,15 @@ int dummy_rproc_example(struct rproc *my_rproc)
of registering this remote processor, additional virtio drivers might get
probed.

int rproc_unregister(struct rproc *rproc)
- Unroll rproc_register().
int rproc_del(struct rproc *rproc)
- Unroll rproc_add().
This function should be called when the platform specific rproc
implementation decides to remove the rproc device. it should
_only_ be called if a previous invocation of rproc_register()
_only_ be called if a previous invocation of rproc_add()
has completed successfully.

After rproc_unregister() returns, @rproc is still valid, and its
last refcount should be decremented by calling rproc_free().
After rproc_del() returns, @rproc is still valid, and its
last refcount should be decremented by calling rproc_put().

Returns 0 on success and -EINVAL if @rproc isn't valid.

Expand Down
8 changes: 4 additions & 4 deletions trunk/drivers/remoteproc/omap_remoteproc.c
Original file line number Diff line number Diff line change
Expand Up @@ -199,23 +199,23 @@ static int __devinit omap_rproc_probe(struct platform_device *pdev)

platform_set_drvdata(pdev, rproc);

ret = rproc_register(rproc);
ret = rproc_add(rproc);
if (ret)
goto free_rproc;

return 0;

free_rproc:
rproc_free(rproc);
rproc_put(rproc);
return ret;
}

static int __devexit omap_rproc_remove(struct platform_device *pdev)
{
struct rproc *rproc = platform_get_drvdata(pdev);

rproc_unregister(rproc);
rproc_free(rproc);
rproc_del(rproc);
rproc_put(rproc);

return 0;
}
Expand Down
32 changes: 16 additions & 16 deletions trunk/drivers/remoteproc/remoteproc_core.c
Original file line number Diff line number Diff line change
Expand Up @@ -1101,7 +1101,7 @@ static void rproc_fw_config_virtio(const struct firmware *fw, void *context)

out:
release_firmware(fw);
/* allow rproc_unregister() contexts, if any, to proceed */
/* allow rproc_del() contexts, if any, to proceed */
complete_all(&rproc->firmware_loading_complete);
}

Expand Down Expand Up @@ -1238,7 +1238,7 @@ void rproc_shutdown(struct rproc *rproc)
EXPORT_SYMBOL(rproc_shutdown);

/**
* rproc_register() - register a remote processor
* rproc_add() - register a remote processor
* @rproc: the remote processor handle to register
*
* Registers @rproc with the remoteproc framework, after it has been
Expand All @@ -1257,7 +1257,7 @@ EXPORT_SYMBOL(rproc_shutdown);
* of registering this remote processor, additional virtio drivers might be
* probed.
*/
int rproc_register(struct rproc *rproc)
int rproc_add(struct rproc *rproc)
{
struct device *dev = &rproc->dev;
int ret = 0;
Expand All @@ -1274,7 +1274,7 @@ int rproc_register(struct rproc *rproc)
/* create debugfs entries */
rproc_create_debug_dir(rproc);

/* rproc_unregister() calls must wait until async loader completes */
/* rproc_del() calls must wait until async loader completes */
init_completion(&rproc->firmware_loading_complete);

/*
Expand All @@ -1295,7 +1295,7 @@ int rproc_register(struct rproc *rproc)

return ret;
}
EXPORT_SYMBOL(rproc_register);
EXPORT_SYMBOL(rproc_add);

/**
* rproc_type_release() - release a remote processor instance
Expand Down Expand Up @@ -1343,13 +1343,13 @@ static struct device_type rproc_type = {
* of the remote processor.
*
* After creating an rproc handle using this function, and when ready,
* implementations should then call rproc_register() to complete
* implementations should then call rproc_add() to complete
* the registration of the remote processor.
*
* On success the new rproc is returned, and on failure, NULL.
*
* Note: _never_ directly deallocate @rproc, even if it was not registered
* yet. Instead, when you need to unroll rproc_alloc(), use rproc_free().
* yet. Instead, when you need to unroll rproc_alloc(), use rproc_put().
*/
struct rproc *rproc_alloc(struct device *dev, const char *name,
const struct rproc_ops *ops,
Expand Down Expand Up @@ -1403,36 +1403,36 @@ struct rproc *rproc_alloc(struct device *dev, const char *name,
EXPORT_SYMBOL(rproc_alloc);

/**
* rproc_free() - unroll rproc_alloc()
* rproc_put() - unroll rproc_alloc()
* @rproc: the remote processor handle
*
* This function decrements the rproc dev refcount.
*
* If no one holds any reference to rproc anymore, then its refcount would
* now drop to zero, and it would be freed.
*/
void rproc_free(struct rproc *rproc)
void rproc_put(struct rproc *rproc)
{
put_device(&rproc->dev);
}
EXPORT_SYMBOL(rproc_free);
EXPORT_SYMBOL(rproc_put);

/**
* rproc_unregister() - unregister a remote processor
* rproc_del() - unregister a remote processor
* @rproc: rproc handle to unregister
*
* This function should be called when the platform specific rproc
* implementation decides to remove the rproc device. it should
* _only_ be called if a previous invocation of rproc_register()
* _only_ be called if a previous invocation of rproc_add()
* has completed successfully.
*
* After rproc_unregister() returns, @rproc isn't freed yet, because
* After rproc_del() returns, @rproc isn't freed yet, because
* of the outstanding reference created by rproc_alloc. To decrement that
* one last refcount, one still needs to call rproc_free().
* one last refcount, one still needs to call rproc_put().
*
* Returns 0 on success and -EINVAL if @rproc isn't valid.
*/
int rproc_unregister(struct rproc *rproc)
int rproc_del(struct rproc *rproc)
{
struct rproc_vdev *rvdev, *tmp;

Expand All @@ -1450,7 +1450,7 @@ int rproc_unregister(struct rproc *rproc)

return 0;
}
EXPORT_SYMBOL(rproc_unregister);
EXPORT_SYMBOL(rproc_del);

static int __init remoteproc_init(void)
{
Expand Down
6 changes: 3 additions & 3 deletions trunk/include/linux/remoteproc.h
Original file line number Diff line number Diff line change
Expand Up @@ -452,9 +452,9 @@ struct rproc_vdev {
struct rproc *rproc_alloc(struct device *dev, const char *name,
const struct rproc_ops *ops,
const char *firmware, int len);
void rproc_free(struct rproc *rproc);
int rproc_register(struct rproc *rproc);
int rproc_unregister(struct rproc *rproc);
void rproc_put(struct rproc *rproc);
int rproc_add(struct rproc *rproc);
int rproc_del(struct rproc *rproc);

int rproc_boot(struct rproc *rproc);
void rproc_shutdown(struct rproc *rproc);
Expand Down

0 comments on commit 59de528

Please sign in to comment.