Re: [PATCH][ASoC]Add ability to remove rate constraints from generic ASoC AC'97 CODEC driver

[Mark Brown <broonie@kernel.org>]

[-- Attachment #1: Type: text/plain, Size: 1700 bytes --]

On Sun, Apr 05, 2015 at 12:16:40AM +0200, Maciej S. Szmigiero wrote:
> W dniu 30.03.2015 06:19, Mark Brown pisze:
> > On Sun, Mar 29, 2015 at 06:00:33PM +0200, Maciej S. Szmigiero wrote:

> > Alternatively if you're just trying to open up the constraints why not
> > just open them up by default and make sure that the AC'97 generic code
> > is constraining things appropriately if it's not doing so already?

> By 'open them up by default' do you mean removing them altogether 
> from ASOC AC'97 generic CODEC to let the AC'97 bus driver constrain
> rates on its own or do you mean retaining ability to constraint in
> ASOC AC'97 driver, just have it switched off by default?

I mean make the default setting be the maximum possible set of rates and
then allow the CODEC code to constrain based on what it finds on the
link.

Please also note that ASoC is spelt ASoC.

> (that is, what the current patch did, just change the default setting).

No, it added a device tree property for controlling things.

> If the rate constraints are removed (or disabled) from the ASOC AC'97
> generic CODEC driver then it can be used as-is in the (at least) fsl_ssi
> driver by registering AC'97 platform device there using index taken from
> DT property (cell-index) already defined as required in
> Documentation/devicetree/bindings/sound/fsl,ssi.txt.

> Now the question is what is more binding: the documentation which
> defines this property as required or DT files which don't define it at all?

The driver really ought to continue to support existing DTs, it can
complain about them and assume the default value if the property really
is required for some reason though.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]