From: b-cousson@ti•com (Cousson, Benoit)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH v2] OMAP2PLUS: DSS: Ensure DSS works correctly if display is enabled in bootloader
Date: Mon, 3 Oct 2011 11:46:16 +0200 [thread overview]
Message-ID: <4E898468.2060508@ti.com> (raw)
In-Reply-To: <1317630070.2026.27.camel@deskari>
Hi Tomi,
On 10/3/2011 10:21 AM, Valkeinen, Tomi wrote:
> Hi Paul,
>
> On Sun, 2011-10-02 at 22:45 -0600, Paul Walmsley wrote:
[...]
>> +struct omap_dss_dispc_dev_attr {
>> + u8 manager_count;
>> + bool has_framedonetv_irq;
>> +};
>> +
>> +#endif
>> diff --git a/arch/arm/mach-omap2/omap_hwmod_2420_data.c b/arch/arm/mach-omap2/omap_hwmod_2420_data.c
>> index 09d9395..8e32cb3 100644
>> --- a/arch/arm/mach-omap2/omap_hwmod_2420_data.c
>> +++ b/arch/arm/mach-omap2/omap_hwmod_2420_data.c
>> @@ -945,6 +945,7 @@ static struct omap_hwmod omap2420_dss_dispc_hwmod = {
>> .slaves_cnt = ARRAY_SIZE(omap2420_dss_dispc_slaves),
>> .omap_chip = OMAP_CHIP_INIT(CHIP_IS_OMAP2420),
>> .flags = HWMOD_NO_IDLEST,
>> + .dev_attr =&omap2_3_dss_dispc_dev_attr
>> };
>
> I didn't know you can add arbitrary data like that to hwmods. What kind
> of data is it meant for? Can the data be used by the driver, or is it
> meant just for arch stuff?
It was added in order to add HW related information for an IP.
So most of the time, this is use for IP version, since this information
is not necessarily accessible from the IP itself. Some time it can be
the number of entries in the mailbox IP that will change depending of
the version too.
> I'm wondering this as we have a complex mechanism in the dss driver to
> find out about the differences of DSS hardware
> (drivers/video/omap2/dss/dss_features.[ch]). The dss_features system is
> currently part of the driver, but should be moved under arch/arm/*omap*
> at some point, and this hwmod dev_attr sounds like it could possibly be
> a right place to handle these.
Please note that I made that kind of comment to Archit when he started
submitted this dss_feature series. That feature management mechanism
could have been useful for any other IPs / driver at that time.
> I looked at how the dev_attrs are used, and all of them seemed to be
> very small, a few fields at max. The DSS features set is, on the other
> hand, quite big amount of data, and meant for the driver.
That's why, most of the time, only the version is in the dev_attr, and
the various information that will depend of that version are stored in
the driver.
But at that time, device tree was not there...
Now, the whole dev_attr stuff will be replaced because device tree is
able to provide the driver any kind of custom information that can be
retrieved directly from the driver without having to use a pdata in
between. So I'm not sure it worth spending too much time on that feature
stuff.
As an example here is the ongoing GPIO DT migration:
http://www.mail-archive.com/linux-omap at vger.kernel.org/msg56505.html
3.2 will have the basic DT support using hwmod as a backend, but the
idea is that for 3.3, we start removing some information from hwmod to
rely on device tree only.
Regards,
Benoit
next prev parent reply other threads:[~2011-10-03 9:46 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1315811306-11001-1-git-send-email-archit@ti.com>
2011-10-03 4:45 ` [PATCH v2] OMAP2PLUS: DSS: Ensure DSS works correctly if display is enabled in bootloader Paul Walmsley
2011-10-03 5:22 ` Archit Taneja
2011-10-03 19:30 ` Paul Walmsley
2011-10-03 8:21 ` Tomi Valkeinen
2011-10-03 9:46 ` Cousson, Benoit [this message]
2011-10-03 17:34 ` Paul Walmsley
2011-10-04 18:27 ` Grant Likely
2011-10-03 17:21 ` Paul Walmsley
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=4E898468.2060508@ti.com \
--to=b-cousson@ti$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox