Re: Small error in iio/iio.h

[Lars Engebretsen <lars@engebretsen.ch>]

Den mån 13 apr. 2020 kl 11:04 skrev Ardelean, Alexandru
<alexandru.Ardelean@analog.com>:
>
> On Mon, 2020-04-13 at 10:38 +0200, Lars Engebretsen wrote:
> > Hi all,
> >
> > I just came across a small thing: I think that there is an erroneous
> > semicolon at the end of line 603 in include/linux/iio/iio.h
>
> yep;
> good find :)
>
> >
> > https://urldefense.com/v3/__https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/include/linux/iio/iio.h?h=v5.7-rc1*n602__;Iw!!A3Ni8CS0y2Y!rXR19w3XDkuxpdAY4teDN4bSsfqvea7Sz5ZW26L2TCEeYhwf1mgTjt63tIp3Xld9oFWgcg$
> >
> > I'm completely new to kernel development, so I don't know the process for
> > submitting patches. Moreover, the macro seems to be gone in git://
> > git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git so maybe the issue
> > has been fixed already? In any case, I'd be happy to submit a patch if that
> > is the right thing to do. But then I'd need a bit of help to get started.
> >
>
> You're probably looking at the wrong branch.
> It's a good idea to look at the ones that have been updated more recently.
> It's still there.
>
> Check 'togreg' or 'testing'.
>
> I suggest editing those.
> Look for patterns in the commit title & descriptions, don't forget to add your
> Signed-off-by tag with  'git commit -s'
> If you want, an idea for a commit title would be:
> 'iio: core: remove extra semi-colon from devm_iio_device_register() macro'
>
> Add a comment describing the change.
>
> Generate the patch with:
> git format-patch HEAD~1..
>
> then run ./scripts/get_maintainers.pl <patch-file>
>
> The output suggests which lists to target.
> What I do, is put lists with --to=<list>, maintainers [other people] with --
> cc=<maintainer> --cc=<driver-author> --cc=<other-potentially-interested-people>
>
> You first need to configure git-email to send emails; there should be some docs
> on the web about that.
>
> Lastly:
> git send-email --to=linux-iio@vger.kernel.org --to=linux-kernel@vger.kernel.org
> --cc=jic@kernel.org <patch-file>
>
>
> That's roughly it.
>
> We should see your patch on the list.
> https://lore.kernel.org/linux-iio/
> &
> https://patchwork.kernel.org/project/linux-iio/list/

Thanks!

I've sent the e-mail now. Got one address wrong the first time, so
please disregard the first e-mail. :-)

    /Lars