[PATCH v2 00/17] omap: mailbox: reorganize init
Felipe Contreras
felipe.contreras at gmail.com
Tue May 18 12:57:55 EDT 2010
On Tue, May 18, 2010 at 4:31 PM, Hiroshi DOYU <Hiroshi.DOYU at nokia.com> wrote:
> From: ext Felipe Contreras <felipe.contreras at gmail.com>
>> I'm not familiar with this kind of module loading, but certainly not
>> all systems have udev.
>>
>> I realized the problem because I have a bare-bones system in my
>> beagleboard where I had to manually load mailbox_mach.
>
> With udev or something equivalent, it should work fine.
But still, you are relying on udev. I don't think we should, and I
don't think there's any need.
>>> The platform device omap2-mailbox accomodatess logical mboxes.
>>
>> Wouldn't it be better the way my patches handle them?
>> * devices.ko: doesn't care about mbox
>> * mach_omap1/mailbox.ko: registers "omap-mailbox" platform device
>> with a list of logical mboxes for OMAP1
>> * mach_omap2/mailbox.ko: registers "omap-mailbox" platform device
>> with a list of logical mboxes for OMAP2
>> * plat-omap/mailbox.ko: creates platform driver based on logical list
>> of mboxes from platform device and handles the list internally
>>
>> The end result would be the exactly same, except that there's no hard
>> dependency between the mach, and plat code.
>
> With the attachemnt, which is just 17 lines of code, you will get the
> structure of logical mbox devices, which are located under the single
> platform device, "omap2-mailbox". This is more _descriptive_ since
> it displays the connection of a platform device and logical devices,
> IOW, where these logical ones come. I don't see any point of not using
> this struture.
That's neat. Am I understanding correctly that that's not an argument
against the split of mailbox_mach and plat mailbox?
>>> Would it be possible to rebase them on the top of my coming patch to
>>> get back logical device entry?
>>
>> Where is that?
>
> Attached. Would it be possible to rebase the cleanups on this one?
I think it would be easier to apply the patch on top of my patch
series... specially since it's so small. Would that be acceptable?
--
Felipe Contreras
More information about the linux-arm-kernel
mailing list