From: Nishanth Menon <nm@ti.com>
To: "Premi, Sanjeev" <premi@ti.com>
Cc: "Nayak, Rajendra" <rnayak@ti.com>,
"linux-omap@vger.kernel.org" <linux-omap@vger.kernel.org>
Subject: Re: hwmod and insertable modules
Date: Fri, 22 Oct 2010 05:26:14 -0500 [thread overview]
Message-ID: <4CC166C6.7060403@ti.com> (raw)
In-Reply-To: <B85A65D85D7EB246BE421B3FB0FBB593023458C2BE@dbde02.ent.ti.com>
Premi, Sanjeev had written, on 10/22/2010 05:01 AM, the following:
>>>>> During module_init I used omap_device_build() to create the
>>>> omap_device.
>>>>> But when trying to implement the module_exit, I couldn't find the
>>>>> corresponding 'destructor'.
>>>> omap_device_build essentially does a platform_device
>> register today
>>>> and hence its not something to be done from a insmod'able 'driver'.
>>> [sp] How does this work for - say dspbridge - where the DSP
>> as device
>>> may not be 'registered' until it is really expected to be used?
>>>
>> arch/arm/mach-omap2/dspbridge.c? ;)
>
> Which repo? On the dspbridge branch on linux-omap, there is not such
:) it is not there as dspbridge is in staging at the moment - I believe
there is some sort of rule of not depending on staging drivers or
something like that.. but the point I was attempting to make is backing
Rajendra's statement -> split your driver into silicon specific data and
silicon independent driver -> the silicon dependent data (like hwmod)
can be collected by a file located in arch/arm/mach-omap2 -> pass the
data as platform_data(with stuff like baseaddress etc..) to the driver
and things can happily function.. Would suggest posting out a RFC patch
if you'd like better ideas from the community I guess.
--
Regards,
Nishanth Menon
next prev parent reply other threads:[~2010-10-22 10:26 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-10-22 9:13 hwmod and insertable modules Premi, Sanjeev
2010-10-22 9:21 ` Premi, Sanjeev
2010-10-22 9:37 ` Nayak, Rajendra
2010-10-22 9:41 ` Premi, Sanjeev
2010-10-22 9:43 ` Nishanth Menon
2010-10-22 10:01 ` Premi, Sanjeev
2010-10-22 10:26 ` Nishanth Menon [this message]
2010-10-22 12:17 ` Premi, Sanjeev
2010-10-22 16:59 ` Kevin Hilman
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4CC166C6.7060403@ti.com \
--to=nm@ti.com \
--cc=linux-omap@vger.kernel.org \
--cc=premi@ti.com \
--cc=rnayak@ti.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.