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