Re: [PATCH 1/1] MFD: Add U300 AB3100 core support v2

[Trilok Soni <soni.trilok-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>]

Hi Linus,

On Tue, May 19, 2009 at 9:53 PM, Linus Walleij
<linus.ml.walleij-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> Hm I shouldn't been so trigger-happy as to send v3 out so soon... :-/
>
> Thanks for you quick review Mike!
>
> 2009/5/19 Mike Rapoport <mike-UTxiZqZC01RS1MOuV/RT9w@public.gmane.org>:
>
>>> +     /* Set a pointer back to the container in device data */
>>> +     for (i = 0; i < ARRAY_SIZE(ab3100_platform_devs); i++)
>>> +             platform_set_drvdata(ab3100_platform_devs[i], ab3100_local);
>>> +
>>> +     /* Register the platform devices */
>>> +     platform_add_devices(ab3100_platform_devs,
>>> +                          ARRAY_SIZE(ab3100_platform_devs));
>>
>> If you register sub-devices this way, they won't appear as ab3100 children.
>
> I'll set the parent in the first loop then (will test that this works) because I
> really like that device table. (Similar to how the board setup does things.)
>

I don't know but why people are not encouraging to use mfd_xxx apis
here? Lot's of drivers are available under drivers/mfd, but only
couple of them are using real mfd_xxx apis? Is there anything missing
in those apis?

-- 
---Trilok Soni
http://triloksoni.wordpress.com
http://www.linkedin.com/in/triloksoni