On Wed, Dec 15, 2010 at 7:53 PM, Nori, Sekhar <[email protected]> wrote:

> On Wed, Dec 15, 2010 at 18:31:43, Sergei Shtylyov wrote:
> > Hello.
> >
> > On 03-12-2010 18:11, Subhasish Ghosh wrote:
> >
> > > The patch adds support for emulated UART controllers
> > > on the programmable realtime unit (PRU) available on OMAPL138.
> > > This defines the system resource requirements such as pin mux,
> > > clock, iomem, interrupt etc and registers the platform device
> > > as per the Linux driver model.
> >
> > > Signed-off-by: Subhasish Ghosh<[email protected]>
> > [...]
> >
> > > diff --git a/arch/arm/mach-davinci/da850.c
> b/arch/arm/mach-davinci/da850.c
> > > index 63916b9..85508c2 100644
> > > --- a/arch/arm/mach-davinci/da850.c
> > > +++ b/arch/arm/mach-davinci/da850.c
> > > @@ -238,6 +238,12 @@ static struct clk tptc2_clk = {
> > >     .flags          = ALWAYS_ENABLED,
> > >   };
> > >
> > > +static struct clk pru_clk = {
> > > +   .name   = "pruss",
> > > +   .parent = &pll0_sysclk2,
> > > +   .lpsc   = DA8XX_LPSC0_DMAX,
> > > +};
> > > +
> >
> >     Shouldn't it be called dmax_clk. And please align the intializer like
> it's
> > done for other clocks.
>
> This module is known as PRU (Programmable Realtime Unit) in TI
> documentation
> So, the LPSC define is better corrected.
>

[SG] -- Will rename it to  DA8XX_LPSC0_PRU

>
> Thanks,
> Sekhar
>
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to