linux-kernel - Re: [PATCH 1/3] remoteproc: fix checkpatch errors in remoteproc code

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Sun, 30 Jun 2013 11:29:53 +0300
From:	Ohad Ben-Cohen <ohad@...ery.com>
To:	Suman Anna <s-anna@...com>
Cc:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 1/3] remoteproc: fix checkpatch errors in remoteproc code

Hi Suman,

On Sat, Jun 1, 2013 at 12:16 AM, Suman Anna <s-anna@...com> wrote:
> This patch fixes all the existing checkpatch errors and warnings
> in the remoteproc source files.
>
> Signed-off-by: Suman Anna <s-anna@...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
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/