* [PATCH 1/3] remoteproc: fix checkpatch errors in remoteproc code
@ 2013-05-31 21:16 Suman Anna
2013-06-30 8:29 ` Ohad Ben-Cohen
0 siblings, 1 reply; 3+ messages in thread
From: Suman Anna @ 2013-05-31 21:16 UTC (permalink / raw)
To: Ohad Ben-Cohen; +Cc: linux-kernel, Suman Anna
This patch fixes all the existing checkpatch errors and warnings
in the remoteproc source files.
Signed-off-by: Suman Anna <s-anna@ti.com>
---
drivers/remoteproc/remoteproc_core.c | 6 ++++--
drivers/remoteproc/remoteproc_debugfs.c | 3 +--
drivers/remoteproc/remoteproc_internal.h | 4 ++--
3 files changed, 7 insertions(+), 6 deletions(-)
diff --git a/drivers/remoteproc/remoteproc_core.c b/drivers/remoteproc/remoteproc_core.c
index 022dc63..bc078c0 100644
--- a/drivers/remoteproc/remoteproc_core.c
+++ b/drivers/remoteproc/remoteproc_core.c
@@ -764,7 +764,8 @@ static void rproc_resource_cleanup(struct rproc *rproc)
/* clean up carveout allocations */
list_for_each_entry_safe(entry, tmp, &rproc->carveouts, node) {
- dma_free_coherent(dev->parent, entry->len, entry->va, entry->dma);
+ dma_free_coherent(dev->parent, entry->len, entry->va,
+ entry->dma);
list_del(&entry->node);
kfree(entry);
}
@@ -922,7 +923,8 @@ static void rproc_fw_config_virtio(const struct firmware *fw, void *context)
/* count the number of notify-ids */
rproc->max_notifyid = -1;
- ret = rproc_handle_resources(rproc, tablesz, rproc_count_vrings_handler);
+ ret = rproc_handle_resources(rproc, tablesz,
+ rproc_count_vrings_handler);
if (ret)
goto out;
diff --git a/drivers/remoteproc/remoteproc_debugfs.c b/drivers/remoteproc/remoteproc_debugfs.c
index 157a573..9d30809 100644
--- a/drivers/remoteproc/remoteproc_debugfs.c
+++ b/drivers/remoteproc/remoteproc_debugfs.c
@@ -248,6 +248,5 @@ void __init rproc_init_debugfs(void)
void __exit rproc_exit_debugfs(void)
{
- if (rproc_dbg)
- debugfs_remove(rproc_dbg);
+ debugfs_remove(rproc_dbg);
}
diff --git a/drivers/remoteproc/remoteproc_internal.h b/drivers/remoteproc/remoteproc_internal.h
index 157e762..2f879b0 100644
--- a/drivers/remoteproc/remoteproc_internal.h
+++ b/drivers/remoteproc/remoteproc_internal.h
@@ -107,12 +107,12 @@ struct resource_table *rproc_find_rsc_table(struct rproc *rproc,
static inline
struct resource_table *rproc_find_loaded_rsc_table(struct rproc *rproc,
- const struct firmware *fw)
+ const struct firmware *fw)
{
if (rproc->fw_ops->find_loaded_rsc_table)
return rproc->fw_ops->find_loaded_rsc_table(rproc, fw);
- return NULL;
+ return NULL;
}
extern const struct rproc_fw_ops rproc_elf_fw_ops;
--
1.8.2
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/3] remoteproc: fix checkpatch errors in remoteproc code
2013-05-31 21:16 [PATCH 1/3] remoteproc: fix checkpatch errors in remoteproc code Suman Anna
@ 2013-06-30 8:29 ` Ohad Ben-Cohen
2013-07-01 15:47 ` Suman Anna
0 siblings, 1 reply; 3+ messages in thread
From: Ohad Ben-Cohen @ 2013-06-30 8:29 UTC (permalink / raw)
To: Suman Anna; +Cc: linux-kernel
Hi Suman,
On Sat, Jun 1, 2013 at 12:16 AM, Suman Anna <s-anna@ti.com> wrote:
> This patch fixes all the existing checkpatch errors and warnings
> in the remoteproc source files.
>
> Signed-off-by: Suman Anna <s-anna@ti.com>
I'm applying the last three fixes, but ignoring the first two (having
lines that slightly exceed 80 chars isn't too bad and makes the code a
bit more readable).
Thanks,
Ohad.
> ---
> drivers/remoteproc/remoteproc_core.c | 6 ++++--
> drivers/remoteproc/remoteproc_debugfs.c | 3 +--
> drivers/remoteproc/remoteproc_internal.h | 4 ++--
> 3 files changed, 7 insertions(+), 6 deletions(-)
>
> diff --git a/drivers/remoteproc/remoteproc_core.c b/drivers/remoteproc/remoteproc_core.c
> index 022dc63..bc078c0 100644
> --- a/drivers/remoteproc/remoteproc_core.c
> +++ b/drivers/remoteproc/remoteproc_core.c
> @@ -764,7 +764,8 @@ static void rproc_resource_cleanup(struct rproc *rproc)
>
> /* clean up carveout allocations */
> list_for_each_entry_safe(entry, tmp, &rproc->carveouts, node) {
> - dma_free_coherent(dev->parent, entry->len, entry->va, entry->dma);
> + dma_free_coherent(dev->parent, entry->len, entry->va,
> + entry->dma);
> list_del(&entry->node);
> kfree(entry);
> }
> @@ -922,7 +923,8 @@ static void rproc_fw_config_virtio(const struct firmware *fw, void *context)
>
> /* count the number of notify-ids */
> rproc->max_notifyid = -1;
> - ret = rproc_handle_resources(rproc, tablesz, rproc_count_vrings_handler);
> + ret = rproc_handle_resources(rproc, tablesz,
> + rproc_count_vrings_handler);
> if (ret)
> goto out;
>
> diff --git a/drivers/remoteproc/remoteproc_debugfs.c b/drivers/remoteproc/remoteproc_debugfs.c
> index 157a573..9d30809 100644
> --- a/drivers/remoteproc/remoteproc_debugfs.c
> +++ b/drivers/remoteproc/remoteproc_debugfs.c
> @@ -248,6 +248,5 @@ void __init rproc_init_debugfs(void)
>
> void __exit rproc_exit_debugfs(void)
> {
> - if (rproc_dbg)
> - debugfs_remove(rproc_dbg);
> + debugfs_remove(rproc_dbg);
> }
> diff --git a/drivers/remoteproc/remoteproc_internal.h b/drivers/remoteproc/remoteproc_internal.h
> index 157e762..2f879b0 100644
> --- a/drivers/remoteproc/remoteproc_internal.h
> +++ b/drivers/remoteproc/remoteproc_internal.h
> @@ -107,12 +107,12 @@ struct resource_table *rproc_find_rsc_table(struct rproc *rproc,
>
> static inline
> struct resource_table *rproc_find_loaded_rsc_table(struct rproc *rproc,
> - const struct firmware *fw)
> + const struct firmware *fw)
> {
> if (rproc->fw_ops->find_loaded_rsc_table)
> return rproc->fw_ops->find_loaded_rsc_table(rproc, fw);
>
> - return NULL;
> + return NULL;
> }
>
> extern const struct rproc_fw_ops rproc_elf_fw_ops;
> --
> 1.8.2
>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH 1/3] remoteproc: fix checkpatch errors in remoteproc code
2013-06-30 8:29 ` Ohad Ben-Cohen
@ 2013-07-01 15:47 ` Suman Anna
0 siblings, 0 replies; 3+ messages in thread
From: Suman Anna @ 2013-07-01 15:47 UTC (permalink / raw)
To: Ohad Ben-Cohen; +Cc: linux-kernel
On 06/30/2013 03:29 AM, Ohad Ben-Cohen wrote:
> Hi Suman,
>
> On Sat, Jun 1, 2013 at 12:16 AM, Suman Anna <s-anna@ti.com> wrote:
>> This patch fixes all the existing checkpatch errors and warnings
>> in the remoteproc source files.
>>
>> Signed-off-by: Suman Anna <s-anna@ti.com>
>
> I'm applying the last three fixes, but ignoring the first two (having
> lines that slightly exceed 80 chars isn't too bad and makes the code a
> bit more readable).
Yeah, no issues. Thanks.
regards
Suman
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2013-07-01 15:53 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-05-31 21:16 [PATCH 1/3] remoteproc: fix checkpatch errors in remoteproc code Suman Anna
2013-06-30 8:29 ` Ohad Ben-Cohen
2013-07-01 15:47 ` Suman Anna
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.