Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1387404

Re: [PATCH 10/18] ASoC: sti: Update example to include assigned-clocks and mclk-fs

Show all headers | View raw

Hi Mark,

On Thu, 21 Apr 2016, Mark Brown wrote:

> On Thu, Apr 21, 2016 at 12:04:27PM +0100, Peter Griffin wrote:
> 
> >  		clocks = <&clk_s_d0_flexgen CLK_PCM_2>;
> > +		assigned-clocks = <&clk_s_d0_flexgen CLK_PCM_2>;
> > +		assigned-clock-parents = <&clk_s_d0_quadfs 2>;
> > +		assigned-clock-rates = <50000000>;
> 
> This may be true for the particular system you're looking at but isn't
> really relevant to the device.

Ok I will drop this from v4 then.
> 
> > @@ -133,6 +139,7 @@ Example of audio card declaration:
> >  			/* DAC */
> >  			format = "i2s";
> >  			dai-tdm-slot-width = <32>;
> > +			mclk-fs = <256>;
> >  			cpu {
> >  				sound-dai = <&sti_uni_player2>;
> >  			};
> 
> This one is more relevant though I'm still a bit concerned that there's
> an expectation that the examples can just be pasted in...

I agree with your point in a more general case (e.g. seperate
audio codec chip connected via SPI), but the reality of the situation for
STi arch is that this device is found in STiH407 family silicon, of which
the two supported upstream devices (STiH407/410) are almost identical SoC's.

Given the recent announcement of "no new SoC's" for STB market, there is unlikely
to be support upstream for new variants and as there is no community board
available new board variants upstream are also unlikely (so far only
ST reference boards are supported upstreamed).

So although I would only consider copying an example as the starting
point for getting something working (and indeed it was the case here). On STi
arch there *should* be a very high probability that copying the exanple 
is all that will be required.

Peter.

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCH 10/18] ASoC: sti: Update example to include assigned-clocks and mclk-fs Peter Griffin <peter.griffin@linaro.org> - 2016-04-21 13:10 +0200
  Re: [PATCH 10/18] ASoC: sti: Update example to include  assigned-clocks and mclk-fs Mark Brown <broonie@kernel.org> - 2016-04-21 18:00 +0200
    Re: [PATCH 10/18] ASoC: sti: Update example to include  assigned-clocks and mclk-fs Peter Griffin <peter.griffin@linaro.org> - 2016-04-26 14:00 +0200
      Re: [PATCH 10/18] ASoC: sti: Update example to include  assigned-clocks and mclk-fs Mark Brown <broonie@kernel.org> - 2016-04-26 16:30 +0200
        Re: [PATCH 10/18] ASoC: sti: Update example to include  assigned-clocks and mclk-fs Peter Griffin <peter.griffin@linaro.org> - 2016-04-26 17:00 +0200
          Re: [PATCH 10/18] ASoC: sti: Update example to include  assigned-clocks and mclk-fs Mark Brown <broonie@kernel.org> - 2016-04-26 17:10 +0200
            Re: [PATCH 10/18] ASoC: sti: Update example to include  assigned-clocks and mclk-fs Peter Griffin <peter.griffin@linaro.org> - 2016-04-26 18:20 +0200
              Re: [PATCH 10/18] ASoC: sti: Update example to include  assigned-clocks and mclk-fs Mark Brown <broonie@kernel.org> - 2016-04-26 18:50 +0200
                Re: [PATCH 10/18] ASoC: sti: Update example to include  assigned-clocks and mclk-fs Peter Griffin <peter.griffin@linaro.org> - 2016-04-26 19:50 +0200

csiph-web