Re: [PATCH] staging: iio: ad5933: add binding doc for ad5933

From: Joe Perches
Date: Sun Dec 02 2018 - 18:22:22 EST


On Mon, 2018-12-03 at 00:20 +0530, Himanshu Jha wrote:
> On Sun, Dec 02, 2018 at 02:57:12PM -0200, Marcelo Schmitt wrote:
> > Add a devicetree documentation for the ad5933 and ad5934 impedance
> > converter, network analyzer.
> >
> > Co-Developed-by: Gabriel Capella <gabriel@xxxxxxxxxxx>
>
> checkpatch spits out:
>
> WARNING: Non-standard signature: Co-Developed-by:
>
> Co-developed-by Vs Co-Developed-by ?
>
> Documentation/process/5.Posting.rst: - Co-developed-by: states that the patch was also created by another developer
> Documentation/process/submitting-patches.rst:12) When to use Acked-by:, Cc:, and Co-Developed-by:
>
> Confusing! Don't know which one is correct.

I think neither one.

What's the real purpose or value of it?
There isn't one as far as I can tell.

Just use Signed-off-by:

Or maybe add multiple "Authored-by:" if
anyone is all that concerned about authorship
crediting...


>
> > Signed-off-by: Marcelo Schmitt <marcelo.schmitt1@xxxxxxxxx>
> > Signed-off-by: Gabriel Capella <gabriel@xxxxxxxxxxx>
> > ---
>
> Use `./scripts/get_maintainer.pl <your_patch>` to list the DT
> maintainers and the relevant mailing list.
>
>