From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH 12/13] OMAP: hwmod data: add class for DSP hwmods Date: Thu, 24 Jun 2010 14:26:43 -0700 Message-ID: <87y6e415rg.fsf@deeprootsystems.com> References: <1277336563-24988-1-git-send-email-khilman@deeprootsystems.com> <1277336563-24988-13-git-send-email-khilman@deeprootsystems.com> <87fx0c6udw.fsf@deeprootsystems.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-pw0-f46.google.com ([209.85.160.46]:57486 "EHLO mail-pw0-f46.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754221Ab0FXV0q (ORCPT ); Thu, 24 Jun 2010 17:26:46 -0400 Received: by pwj8 with SMTP id 8so2041977pwj.19 for ; Thu, 24 Jun 2010 14:26:45 -0700 (PDT) In-Reply-To: <87fx0c6udw.fsf@deeprootsystems.com> (Kevin Hilman's message of "Thu, 24 Jun 2010 13:35:55 -0700") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: Paul Walmsley Cc: linux-omap@vger.kernel.org Kevin Hilman writes: > Paul Walmsley 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/ Kevin