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

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




> -----Original Message-----
> From: Mark Brown <broonie@xxxxxxxxxx>
> Sent: Monday, 3 August, 2020 6:49 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 Mon, Aug 03, 2020 at 01:59:10AM +0000, Sit, Michael Wei Hong
> wrote:
> > > On Thu, Jul 30, 2020 at 01:53:19PM +0800, Michael Sit Wei Hong
> wrote:
> 
> > > > Add a property that configures the interface to the desired max
> > > > number of capture channels. The platform may have multiple
> > > > interfaces with different number of capture channels.
> 
> > > Why?  None of the other platforms which support many
> channels need
> > > this defining and the constraint code and/or machine driver
> would
> > > normally be where this would be handled.
> 
> > The platform uses the audio-graph-card to create the dai-links,
> and doesn't use a specific machine driver code.
> 
> The audio-graph-card is very flexible and if it doesn't support
> something which it is useful to configure per platform then that's
> the place to add the extension, not some DAI specific driver.
> 
> > The platform also has 2 different interfaces which have different
> supported max-channels.
> > Using this value in the device-tree to determine the maximum
> supported channel of the interface.
> 
> These should have different compatible strings, there are likely
> further differences between them (even if they are not currently
> documented).
> 
The 2 different I2S ports are from the same SoC which supports different
number of channels, do we need different compatible strings for this?
Considering the only difference is the maximum supported channels is 8 and 2?

> 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.




[Index of Archives]     [ALSA User]     [Linux Audio Users]     [Pulse Audio]     [Kernel Archive]     [Asterisk PBX]     [Photo Sharing]     [Linux Sound]     [Video 4 Linux]     [Gimp]     [Yosemite News]

  Powered by Linux