[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <555406DA.8040709@rock-chips.com>
Date: Thu, 14 May 2015 10:22:18 +0800
From: zhengxing <zhengxing@...k-chips.com>
To: Dylan Reid <dgreid@...omium.org>
CC: Mark Brown <broonie@...nel.org>, Xing Zheng <acgzxing@...il.com>,
Douglas Anderson <dianders@...omium.org>,
Heiko Stübner <heiko@...ech.de>,
Sonny Rao <sonnyrao@...omium.org>,
linux-rockchip@...ts.infradead.org,
Liam Girdwood <lgirdwood@...il.com>,
Jaroslav Kysela <perex@...ex.cz>, Takashi Iwai <tiwai@...e.de>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"alsa-devel@...a-project.org" <alsa-devel@...a-project.org>,
"linux-arm-kernel@...ts.infradead.org"
<linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH 1/4] ASoC: rockchip: add rockchip machine driver
On 2015年05月14日 07:11, Dylan Reid wrote:
> On Wed, May 13, 2015 at 10:21 AM, Dylan Reid<dgreid@...omium.org> wrote:
>> On Wed, May 13, 2015 at 9:42 AM, Mark Brown<broonie@...nel.org> wrote:
>>> On Wed, May 13, 2015 at 09:23:01PM +0800, zhengxing wrote:
>>>> On 2015年05月13日 03:22, Mark Brown wrote:
>>>>> Is it not possible to extend simple card to handle your use cases?
>>>>> Given the very generic naming and the fact that things like jack
>>>>> detection and so on should depend on the CODEC and board usually rather
>>>>> than on the SoC it doesn't sound like this is Rockchip specific.
>>>>> This also looks like you're reimplementing some device model enumeration
>>>>> stuff which probably shouldn't be happening but let's understand the
>>>>> problem you're trying to solve here before going too far into the code.
>>>> Because we are trying to bring rt5650 in the project, so we intend to
>>>> describe supported codecs with DTS via only a rockchip machine driver file,
>>>> others remain pre-implement(like max98090 / rt5645 that vendor machine
>>>> driver).
>>> I don't undertand what you're saying here, sorry - why is this not just
>>> a case of writing multiple machine drivers?
>> I don't understand this either. I'd think the best solution is
>> simple-card, configured through DTS for each device.
Hi Mark, Dylan,
I tried to use simple-card and had some trouble that jack detection. So
my idea is simply:
/-- rockchip_max98090
(to configure max98090, and no need to via DTS)
rockchip_machine_driver |-- rockchip_rt5645 (to configure
rt5650/rt5645, and no need to via DTS)
(supported codecs via DTS) \-- rockchip_(maybe other codecs)
It is right that it is base on original rockchip_max98090, but the old
rockchip_max98090 used DTS.
I tried to explain my ideas. Your kind suggestions will be highly
appreciated.
Thanks.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists