Messages in this thread | | | Date | Mon, 10 Oct 2011 14:11:28 -0600 (MDT) | From | Paul Walmsley <> | Subject | Re: [PATCHv9 03/18] TEMP: OMAP3xxx: hwmod data: add PRM hwmod |
| |
On Mon, 10 Oct 2011, Cousson, Benoit wrote:
> Hi Paul, > > On 10/10/2011 9:24 PM, Paul Walmsley wrote: > > Hi Tero > > > > On Fri, 23 Sep 2011, Tero Kristo wrote: > > > > > This patch is temporary until Paul can provide a final version. > > > > > > Signed-off-by: Tero Kristo<t-kristo@ti.com> > > > > Here's an updated version of this one. The one change is that the hwmod's > > name is now "prm3xxx" to reflect that the register layout of this IP block > > is quite different from its OMAP2 predecessors and OMAP4 successors. > > This should avoid some of the special-purpose driver probing code. > > This is not really aligned with the naming convention we've been using so far. > In both cases, the hwmod should just be named "prm". If a version information > is needed, then it should be added in the revision class field. > > It will make the device creation even simpler and not dependent of the SoC > version. > I did not check the whole series, but I'm not sure we need a special treatment > in the case of the prm.
OK, sounds like this needs more discussion, so will drop this patch from the pull request.
- Paul
| |