From: Kevin Hilman <khilman@deeprootsystems.com>
To: Paul Walmsley <paul@pwsan.com>
Cc: linux-omap@vger.kernel.org
Subject: Re: [PATCH 12/13] OMAP: hwmod data: add class for DSP hwmods
Date: Thu, 24 Jun 2010 14:34:41 -0700 [thread overview]
Message-ID: <87mxukyv0u.fsf@deeprootsystems.com> (raw)
In-Reply-To: <87y6e415rg.fsf@deeprootsystems.com> (Kevin Hilman's message of "Thu, 24 Jun 2010 14:26:43 -0700")
Kevin Hilman <khilman@deeprootsystems.com> writes:
> Kevin Hilman <khilman@deeprootsystems.com> writes:
>
>> Paul Walmsley <paul@pwsan.com> writes:
>>
>>> Hi Kevin,
>>>
>>> On Wed, 23 Jun 2010, Kevin Hilman wrote:
>>>
>>>> Add a new hwmod class for DSP devices. To be used when hwmods
>>>> are created for DSP on OMAP3 (a.k.a. IVA2.)
>>>
>>> I guess this patch is not needed any more? But if it is, it might be best
>>> to name the class "iva2" or something like that, since the IVA2 subsystem
>>> contains more than just the DSP; it also contains the dedicated hardware
>>> video accelerator blocks, which are separate from the DSP.
>>
>> Right, I'll be dropping this patch too as I'm just switching to use
>> omap_hwmod_lookup() instead of the class iterators.
>
> Turns out I need a class even when doing the omap_hwmod_lookup()
> as omap_hwmod_register() fails if there is no class or class name.
>
> I'll keep this pach but s/dsp/iva2/
>
On second thought, I'll use "iva" to match how the naming was done for
OMAP4.
Kevin
next prev parent reply other threads:[~2010-06-24 21:34 UTC|newest]
Thread overview: 50+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-23 23:42 [PATCH 00/13] OMAP: CM, hwmod, omap_device fixes and updates Kevin Hilman
2010-06-23 23:42 ` [PATCH 01/13] OMAP24xx: CM: fix mask used for checking IDLEST status Kevin Hilman
2010-06-23 23:42 ` [PATCH 02/13] OMAP: hwmod: allow idle after HWMOD_INIT_NO_IDLE Kevin Hilman
2010-06-24 5:02 ` Paul Walmsley
2010-06-24 17:48 ` Kevin Hilman
2010-06-29 21:47 ` Kevin Hilman
2010-06-23 23:42 ` [PATCH 03/13] OMAP2/3: hwmod: L3 and L4 CORE/PER/WKUP hwmods don't have IDLEST Kevin Hilman
2010-06-23 23:42 ` [PATCH 04/13] OMAP: hwmod: Fix the missing braces Kevin Hilman
2010-06-23 23:46 ` Gadiyar, Anand
2010-06-24 0:19 ` Paul Walmsley
2010-06-23 23:42 ` [PATCH 05/13] OMAP2&3: hwmod: Remove _hwmod prefix in name string Kevin Hilman
2010-06-24 5:04 ` Paul Walmsley
2010-06-24 11:53 ` Cousson, Benoit
2010-07-02 11:47 ` Paul Walmsley
2010-06-23 23:42 ` [PATCH 06/13] OMAP: hwmod: add non-locking versions of enable and idle functions Kevin Hilman
2010-06-24 5:08 ` Paul Walmsley
2010-06-24 12:59 ` Basak, Partha
2010-06-24 17:55 ` Kevin Hilman
2010-06-24 17:55 ` Kevin Hilman
2010-06-23 23:42 ` [PATCH 07/13] OMAP: hwmod: don't auto-disable hwmod when !CONFIG_PM_RUNTIME Kevin Hilman
2010-07-02 11:38 ` Paul Walmsley
2010-06-23 23:42 ` [PATCH 08/13] OMAP4: hwmod: Enable omap_device build for OMAP4 Kevin Hilman
2010-06-24 5:11 ` Paul Walmsley
2010-06-24 14:23 ` Nayak, Rajendra
2010-06-24 17:52 ` Kevin Hilman
2010-06-24 19:29 ` Nayak, Rajendra
2010-06-24 21:47 ` Kevin Hilman
2010-07-02 11:46 ` Paul Walmsley
2010-06-23 23:42 ` [PATCH 09/13] OMAP: omap_device: ensure hwmod tracks attached omap_device pointer Kevin Hilman
2010-07-02 11:40 ` Paul Walmsley
2010-06-23 23:42 ` [PATCH 10/13] OMAP: omap_device: add flag to disable automatic bus-level suspend/resume Kevin Hilman
2010-06-24 3:40 ` Paul Walmsley
2010-06-24 17:39 ` Kevin Hilman
2010-06-24 18:06 ` Paul Walmsley
2010-06-24 18:28 ` Kevin Hilman
2010-06-23 23:42 ` [PATCH 11/13] OMAP: create omap_devices for MPU, DSP, L3 Kevin Hilman
2010-06-24 6:26 ` Paul Walmsley
2010-06-24 6:55 ` Paul Walmsley
2010-06-24 17:59 ` Kevin Hilman
2010-06-24 6:58 ` Shilimkar, Santosh
2010-06-24 7:19 ` Paul Walmsley
2010-06-24 7:27 ` Shilimkar, Santosh
2010-06-23 23:42 ` [PATCH 12/13] OMAP: hwmod data: add class for DSP hwmods Kevin Hilman
2010-06-24 18:44 ` Paul Walmsley
2010-06-24 20:35 ` Kevin Hilman
2010-06-24 21:26 ` Kevin Hilman
2010-06-24 21:34 ` Kevin Hilman [this message]
2010-06-23 23:42 ` [PATCH 13/13] OMAP3: hwmod data: add data for OMAP3 IVA2 Kevin Hilman
2010-06-24 20:43 ` Kevin Hilman
2010-06-24 21:36 ` Cousson, Benoit
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=87mxukyv0u.fsf@deeprootsystems.com \
--to=khilman@deeprootsystems.com \
--cc=linux-omap@vger.kernel.org \
--cc=paul@pwsan.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).