lkml.org 
[lkml]   [2019]   [Apr]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [alsa-devel] [PATCH v3 2/5] soundwire: fix style issues
On Mon, Apr 15, 2019 at 08:09:59AM -0500, Pierre-Louis Bossart wrote:
>
> >>
> >> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
> >> ---
> >> drivers/soundwire/Kconfig | 2 +-
> >> drivers/soundwire/bus.c | 87 ++++++++--------
> >> drivers/soundwire/bus.h | 16 +--
> >> drivers/soundwire/bus_type.c | 4 +-
> >> drivers/soundwire/cadence_master.c | 87 ++++++++--------
> >> drivers/soundwire/cadence_master.h | 22 ++--
> >> drivers/soundwire/intel.c | 87 ++++++++--------
> >> drivers/soundwire/intel.h | 4 +-
> >> drivers/soundwire/intel_init.c | 12 +--
> >> drivers/soundwire/mipi_disco.c | 116 +++++++++++----------
> >> drivers/soundwire/slave.c | 10 +-
> >> drivers/soundwire/stream.c | 161 +++++++++++++++--------------
> >
> > I would prefer this to be a patch per module. It doesnt help to have a
> > single patch for all the files!
> >
> > It would be great to have cleanup done per logical group, for example
> > typos in a patch, aligns in another etc...
>
> You've got to be kidding. I've never seen people ask for this sort of
> detail.
>
> >
> >> 12 files changed, 313 insertions(+), 295 deletions(-)
> >>
> >> diff --git a/drivers/soundwire/Kconfig b/drivers/soundwire/Kconfig
> >> index 19c8efb9a5ee..84876a74874f 100644
> >> --- a/drivers/soundwire/Kconfig
> >> +++ b/drivers/soundwire/Kconfig
> >> @@ -4,7 +4,7 @@
> >>
> >> menuconfig SOUNDWIRE
> >> bool "SoundWire support"
> >> - ---help---
> >> + help
> >
> > Not sure if this is a style issue, kernel seems to have 2990 instances
> > of this!
>
> this is reported by checkpatch.pl --strict.

Please don't run checkpatch on code that's already in the kernel, and
especially not with the --strict (a.k.a. --subjective) option enabled.

Don't try to fix what isn't broken.

Thanks,
Johan

\
 
 \ /
  Last update: 2019-04-17 11:34    [W:0.162 / U:0.256 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site