Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1366283
| From | PC Liao <pc.liao@mediatek.com> |
|---|---|
| Newsgroups | linux.kernel |
| Subject | Re: [PATCH] ASoC: mediatek: Add second I2S on mt8173-rt5650 machine driver |
| Date | 2016-03-29 15:30 +0200 |
| Message-ID | <ri1Sy-6PD-1@gated-at.bofh.it> (permalink) |
| References | <rgqV3-48a-1@gated-at.bofh.it> <rhWzw-2QS-15@gated-at.bofh.it> |
| Organization | linux.* mail to news gateway |
Hi Mark,
On Tue, 2016-03-29 at 03:11 +0800, Mark Brown wrote:
> On Fri, Mar 25, 2016 at 11:42:05AM +0800, PC Liao wrote:
>
> > +Optional properties:
> > +- mediatek,rt5650_i2s: I2S mode of rt5650
> > + 0: using I2S1 on rt5650 for record
> > + 1: using I2S2 on rt5650 for record
> > +
>
> I would expect the machine driver to have a reference to the DAI that is
> in use rather than a property like this. This is the way pretty much
> all DT based machine drivers work...
Thanks for comment.
I think I need to comment more clearly.
This setting is different connection from MT8173 to rt5650 on hardware.
- mediatek,rt5650_i2s: I2S mode of rt5650
0: Default setting. Playback and record path use same set of I2S.
Playback/Record path using same I2S clock connect from MT8173 I2S1 to
rt5650 I2S1.
1: Playback and record path use different set of I2S. Playback path
connects from MT8173 I2S1 to rt5650 I2S1 and record path connects from
MT8173 I2S2 to rt5650 I2S2.
Because we use codec ASRC function and need to initialize on different
setting about this parts:
+ switch (mt8173_rt5650_priv.rt5650_i2s) {
+ case MT8173_RT5650_I2S1:
+ rt5645_sel_asrc_clk_src(codec,
+ RT5645_DA_STEREO_FILTER |
+ RT5645_AD_STEREO_FILTER,
+ RT5645_CLK_SEL_I2S1_ASRC);
+ break;
+ case MT8173_RT5650_I2S2:
+ rt5645_sel_asrc_clk_src(codec,
+ RT5645_DA_STEREO_FILTER,
+ RT5645_CLK_SEL_I2S1_ASRC);
+ rt5645_sel_asrc_clk_src(codec,
+ RT5645_AD_STEREO_FILTER,
+ RT5645_CLK_SEL_I2S2_ASRC);
+ break;
+ default:
+ break;
+ }
, we use device tree to determine which connection we choose.
Does this explain can be accepted?
Thanks!
Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread
[PATCH] ASoC: mediatek: Add second I2S on mt8173-rt5650 machine driver PC Liao <pc.liao@mediatek.com> - 2016-03-25 04:50 +0100
Re: [PATCH] ASoC: mediatek: Add second I2S on mt8173-rt5650 machine driver Mark Brown <broonie@kernel.org> - 2016-03-29 09:50 +0200
Re: [PATCH] ASoC: mediatek: Add second I2S on mt8173-rt5650 machine driver PC Liao <pc.liao@mediatek.com> - 2016-03-29 15:30 +0200
Re: [PATCH] ASoC: mediatek: Add second I2S on mt8173-rt5650 machine driver Mark Brown <broonie@kernel.org> - 2016-03-29 18:10 +0200
Re: [PATCH] ASoC: mediatek: Add second I2S on mt8173-rt5650 machine driver PC Liao <pc.liao@mediatek.com> - 2016-04-01 05:00 +0200
Re: [PATCH] ASoC: mediatek: Add second I2S on mt8173-rt5650 machine driver Mark Brown <broonie@kernel.org> - 2016-04-02 19:10 +0200
csiph-web