Re: [PATCH] staging: greybus: Fix warning to limit chars per line

From: Viresh Kumar
Date: Fri Apr 06 2018 - 06:25:24 EST


On 06-04-18, 15:47, Gaurav Dhingra wrote:
> I sent in an updated patchset. Though I forgot to add
> viresh.kumar@xxxxxxxxxx to "To" in mail.

That's fine, but you still haven't sent it to all the relevant people.
You should have used the get_maintainer script (present in kernel
source) for that.

$ scripts/get_maintainer.pl drivers/staging/greybus/audio_codec.h

Vaibhav Agarwal <vaibhav.sr@xxxxxxxxx> (maintainer:GREYBUS AUDIO PROTOCOLS DRIVERS)
Mark Greer <mgreer@xxxxxxxxxxxxxxx> (maintainer:GREYBUS AUDIO PROTOCOLS DRIVERS)
Johan Hovold <johan@xxxxxxxxxx> (maintainer:GREYBUS SUBSYSTEM)
Alex Elder <elder@xxxxxxxxxx> (maintainer:GREYBUS SUBSYSTEM)
Greg Kroah-Hartman <gregkh@xxxxxxxxxxxxxxxxxxx> (maintainer:GREYBUS SUBSYSTEM)
greybus-dev@xxxxxxxxxxxxxxxx (moderated list:GREYBUS SUBSYSTEM)
devel@xxxxxxxxxxxxxxxxxxxx (open list:STAGING SUBSYSTEM)
linux-kernel@xxxxxxxxxxxxxxx (open list)

> I tried to follow instructions
> described on https://kernelnewbies.org/FirstKernelPatch for updating my
> patch. Do you think I followed the instructions correctly?

Mostly yes, you did it fine.

> I was thinking
> may be I need to update the already sent patch by adding *new commit* to my
> already existing commit on that git branch, but instead I tried to do what I
> understood from the website I mentioned above.

Sorry, I find it difficult to understand what you wrote ;)

--
viresh