[PATCH 4/4] dt-bindings: sound: intel,keembay-i2s: Add channel-max property

Sit, Michael Wei Hong michael.wei.hong.sit at intel.com
Mon Aug 10 11:47:15 CEST 2020



> -----Original Message-----
> From: Mark Brown <broonie at kernel.org>
> Sent: Friday, 7 August, 2020 10:31 PM
> To: Sit, Michael Wei Hong <michael.wei.hong.sit at intel.com>
> Cc: alsa-devel at alsa-project.org; tiwai at suse.com; pierre-
> louis.bossart at linux.intel.com; Rojewski, Cezary
> <cezary.rojewski at intel.com>; Shevchenko, Andriy
> <andriy.shevchenko at intel.com>; liam.r.girdwood at linux.intel.com;
> Sia, Jee Heng <jee.heng.sia at intel.com>
> 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.


More information about the Alsa-devel mailing list