[alsa-devel] [PATCH 3/6] ALSA: hda: intel-nhlt: remove useless OR operation

Takashi Iwai tiwai at suse.de
Fri Jul 19 20:30:02 CEST 2019


On Fri, 19 Jul 2019 20:20:58 +0200,
Pierre-Louis Bossart wrote:
> 
> Thanks for the quick review Takashi, much appreciated.
> 
> On 7/19/19 1:09 PM, Takashi Iwai wrote:
> > On Fri, 19 Jul 2019 19:06:07 +0200,
> > Pierre-Louis Bossart wrote:
> >>
> >> Each assignment is final so there's no point in doing an OR.
> >>
> >> Signed-off-by: Pierre-Louis Bossart <pierre-louis.bossart at linux.intel.com>
> >
> > Please fold into the patch 2.  There is no reason to split.
> 
> Sure. I just wanted to keep this separate since patch2 is mostly about
> moving code. No problem to squash it.

FWIW, if it's really a code movement, it'd make sense to split, yes.
But in this case, the patch 2 simply puts a new code.  The actual code
"move" happens in the patch 5.


thanks,

Takashi


More information about the Alsa-devel mailing list