Re: [alsa-devel] [PATCH v4 22/22] soundwire: add missing newlines in dynamic debug logs

From: Pierre-Louis Bossart
Date: Thu May 02 2019 - 10:56:07 EST

On 5/2/19 1:32 AM, Greg KH wrote:
On Thu, May 02, 2019 at 11:07:46AM +0530, Vinod Koul wrote:
On 01-05-19, 10:57, Pierre-Louis Bossart wrote:
For some reason the newlines are not used everywhere. Fix as needed.

Reported-by: Joe Perches <joe@xxxxxxxxxxx>
Reviewed-by: Takashi Iwai <tiwai@xxxxxxx>
Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart@xxxxxxxxxxxxxxx>
drivers/soundwire/bus.c | 74 +++++++++----------
drivers/soundwire/cadence_master.c | 12 ++--
drivers/soundwire/intel.c | 12 ++--
drivers/soundwire/stream.c | 110 ++++++++++++++---------------

Sorry this needs to be split up. I think bus.c and stream.c should go
in patch with cadence_master.c and intel.c in different ones

Again, _way_ too picky. You can't look a gift horse _too_ closely in
the mouth...

greg k-h

Vinod, your distinction between subsystem and driver is quite arbitrary and borderline unreasonable. I would counter that the Intel parts have actually nothing to do in this drivers/soundwire directory and should be moved to the SOF tree if you push your own logic one step. There already known variations on capabilities and number of links which would be better handled in sound/soc/sof/intel, just like SoundWire slaves are expected to be in sound/soc/codecs.

Besides this fixes *newlines*...

I will send an update but I am not happy about the directions here.