> -----Original Message----- > From: Mark Brown <broonie@xxxxxxxxxx> > Sent: Friday, 7 August, 2020 10:31 PM > To: Sit, Michael Wei Hong <michael.wei.hong.sit@xxxxxxxxx> > Cc: alsa-devel@xxxxxxxxxxxxxxxx; tiwai@xxxxxxxx; pierre- > louis.bossart@xxxxxxxxxxxxxxx; Rojewski, Cezary > <cezary.rojewski@xxxxxxxxx>; Shevchenko, Andriy > <andriy.shevchenko@xxxxxxxxx>; liam.r.girdwood@xxxxxxxxxxxxxxx; > Sia, Jee Heng <jee.heng.sia@xxxxxxxxx> > Subject: Re: [PATCH 4/4] dt-bindings: sound: intel,keembay-i2s: Add > channel-max property > > On Wed, Aug 05, 2020 at 06:21:14AM +0000, Sit, Michael Wei Hong > wrote: > > > > Are you *sure* that's the only difference, or is that just the only > > > difference you know about right now? > > > Yes, I am fairy sure that is the only difference according to the > design, as per the platform use case. > > It would still be safer to have a separate compatible, it wouldn't be > the first time that additional changes that the hardware people had > failed to communicate were discovered. > I will create a separate compatible string in the next revision of the patch. Since part of the patch series has already been merged, do I send in the new patch as a new review or on top of these patch series? > Please fix your mail client to word wrap within paragraphs at > something substantially less than 80 columns. Doing this makes > your messages much easier to read and reply to.