Re: [PATCH] mfd: Support SiRF audio modules

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

 




2014-03-11 0:48 GMT+08:00 Mark Brown <broonie@xxxxxxxxxx>:
> On Mon, Mar 10, 2014 at 04:31:58PM +0000, Lee Jones wrote:
>
>> > > I'm not sure there's any need for this. Why don't you just parse the
>> > > child nodes? Are you even sure you need an MFD at all? It appears
>> > > you're just using an MFD to share a regmap. Seems like over-kill to me.
>
>> > In my case, the prima2 and atlas6 codec have small different. But
>> > they use the same register address space.
>> > The sound codec driver need know what codec device is applied.
>
>> I understand it was you who made the original request for the regmap
>> to be shared between devices. Was it an MFD that you had in mind?
>
> I was expecting an audio driver that registered several interfaces with
> the ASoC core rather than a MFD.  It's possible there's some good reason
> for doing things this way, I've not looked at the new patches (I only
> seem to have one for the MFD itself...).

I think the MFD driver is better than the ASoC core. Because the mfd_add_devices
can register all child device into system and the parent driver data
can be got the regmap
to all child drivers.

Hi Mark Brown:
I will send new patches of sound, if the MFD driver is applied.

Thanks.
-- 
------------------------------
Rongjun Ying
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux