From: Mauro Carvalho Chehab <firstname.lastname@example.org> To: Hans Verkuil <email@example.com> Cc: firstname.lastname@example.org, email@example.com, firstname.lastname@example.org, email@example.com, firstname.lastname@example.org, email@example.com, Hans Verkuil <firstname.lastname@example.org> Subject: Re: [PATCHv16 10/13] cec: adv7842: add cec support Date: Fri, 17 Jun 2016 07:12:12 -0300 [thread overview] Message-ID: <email@example.com> (raw) In-Reply-To: <5763AF87.firstname.lastname@example.org> Em Fri, 17 Jun 2016 10:06:31 +0200 Hans Verkuil <email@example.com> escreveu: > On 06/16/2016 11:22 PM, Mauro Carvalho Chehab wrote: > > Em Fri, 29 Apr 2016 15:52:25 +0200 > > Hans Verkuil <firstname.lastname@example.org> escreveu: > > > >> From: Hans Verkuil <email@example.com> > >> > >> Add CEC support to the adv7842 driver. > >> > >> Signed-off-by: Hans Verkuil <firstname.lastname@example.org> > > > > Won't review patches 10-13, as the same reviews I made for patch 9 > > very likely applies. > > > > As this series is causing non-staging drivers to be dependent of a > > staging driver, I'll wait for the next version that should be > > solving this issue. > > > > For the new 9-13 patches, please be sure that checkpatch will be > > happy. For the staging stuff, the checkpatch issues can be solved > > later, as I'll re-check against checkpatch when it moves from staging > > to mainstream. > > I have to make changes anyway so I'll make a new pull request later > today fixing all the comments and replacing unsigned with unsigned int > (which is a majority of all the checkpatch warnings). Ok. > Did I mention yet how much I hate this new checkpatch warning? In almost all > cases I agree with the checkpatch rules, but this one is just stupid IMHO. This is the commit that added such rule: commit a1ce18e4f941d2039aa3bdeee17db968919eac2f Author: Joe Perches <email@example.com> Date: Tue Mar 15 14:58:03 2016 -0700 checkpatch: warn on bare unsigned or signed declarations without int Kernel style prefers "unsigned int <foo>" over "unsigned <foo>" and "signed int <foo>" over "signed <foo>". Emit a warning for these simple signed/unsigned <foo> declarations. Fix it too if desired. Signed-off-by: Joe Perches <firstname.lastname@example.org> Acked-by: David S. Miller <email@example.com> Signed-off-by: Andrew Morton <firstname.lastname@example.org> Signed-off-by: Linus Torvalds <email@example.com> So, the people to blame are mentioned on it. I was actually expecting to see any rationale for that decision, but the log is useless on that sense. Maybe there are some discussions at LKML explaining that. At least on the patch thread, no mention why it was done: https://www.spinics.net/lists/kernel/msg2205100.html That's said, it sounds that the checkpatch autofix rule should do the changes for you, according with the comments. There is a patch to sparc that does this: git ls-files arch/sparc | \ xargs ./scripts/checkpatch.pl -f --fix-inplace --types=unspecified_int I guess I'll just run that on our subsystem, and we're done with that, removing the risc of having to merge hundreds of stupid checkpatch fixup stuff for the existing code, and distracting ourselves from patches that really matters. > > Oh well, I'll make the change. Perhaps it will grow on me over time. > > Regards, > > Hans Thanks, Mauro
next prev parent reply other threads:[~2016-06-17 10:12 UTC|newest] Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top 2016-04-29 13:52 [PATCHv16 00/13] HDMI CEC framework Hans Verkuil 2016-04-29 13:52 ` [PATCHv16 01/13] input.h: add BUS_CEC type Hans Verkuil 2016-04-29 13:52 ` [PATCHv16 02/13] HID: add HDMI CEC specific keycodes Hans Verkuil 2016-04-29 13:52 ` [PATCHv16 03/13] rc: Add HDMI CEC protocol handling Hans Verkuil 2016-04-29 13:52 ` [PATCHv16 04/13] cec: add HDMI CEC framework Hans Verkuil 2016-06-16 16:00 ` Mauro Carvalho Chehab 2016-06-25 12:27 ` Hans Verkuil 2016-04-29 13:52 ` [PATCHv16 05/13] cec/TODO: add TODO file so we know why this is still in staging Hans Verkuil 2016-04-29 13:52 ` Hans Verkuil 2016-06-16 19:53 ` Mauro Carvalho Chehab 2016-04-29 13:52 ` [PATCHv16 06/13] cec: add compat32 ioctl support Hans Verkuil 2016-04-29 13:52 ` [PATCHv16 07/13] cec.txt: add CEC framework documentation Hans Verkuil 2016-04-29 13:52 ` Hans Verkuil 2016-06-16 20:12 ` Mauro Carvalho Chehab 2016-06-17 7:22 ` Hans Verkuil 2016-06-17 8:55 ` Mauro Carvalho Chehab 2016-04-29 13:52 ` [PATCHv16 08/13] DocBook/media: add CEC documentation Hans Verkuil 2016-06-16 21:09 ` Mauro Carvalho Chehab 2016-06-17 7:58 ` Hans Verkuil 2016-06-17 9:50 ` Mauro Carvalho Chehab 2016-06-17 11:09 ` Hans Verkuil 2016-06-17 11:37 ` Mauro Carvalho Chehab 2016-06-18 16:25 ` Dmitry Torokhov 2016-04-29 13:52 ` [PATCHv16 09/13] cec: adv7604: add cec support Hans Verkuil 2016-06-16 21:17 ` Mauro Carvalho Chehab 2016-06-17 8:03 ` Hans Verkuil 2016-06-17 9:53 ` Mauro Carvalho Chehab 2016-04-29 13:52 ` [PATCHv16 10/13] cec: adv7842: " Hans Verkuil 2016-06-16 21:22 ` Mauro Carvalho Chehab 2016-06-17 8:06 ` Hans Verkuil 2016-06-17 8:06 ` Hans Verkuil 2016-06-17 10:12 ` Mauro Carvalho Chehab [this message] 2016-04-29 13:52 ` [PATCHv16 11/13] cec: adv7511: " Hans Verkuil 2016-04-29 13:52 ` [PATCHv16 12/13] cec: s5p-cec: Add s5p-cec driver Hans Verkuil 2016-04-29 13:52 ` [PATCHv16 13/13] vivid: add CEC emulation Hans Verkuil
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --firstname.lastname@example.org \ --email@example.com \ --subject='Re: [PATCHv16 10/13] cec: adv7842: add cec support' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
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.