> > > > >
> > > > > diff --git a/arch/arm/include/asm/arch-omap3/cpu.h
> > > > > b/arch/arm/include/asm/arch-omap3/cpu.h
> > > > > index c072c27..773ea0c 100644
> > > > > --- a/arch/arm/include/asm/arch-omap3/cpu.h
> > > > > +++ b/arch/arm/include/asm/arch-omap3/cpu.h
> > > > > @@ -483,4 +483,24 @@ struct pm {
> > > > >  #define I2C_BASE2            (OMAP34XX_CORE_L4_IO_BASE + 0x72000)
> > > > >  #define I2C_BASE3            (OMAP34XX_CORE_L4_IO_BASE + 0x60000)
> > > > >
> > > > > +/* AM35x specific */
> > > > > +#ifndef __KERNEL_STRICT_NAMES
> > > > > +#ifndef __ASSEMBLY__
> > > > > +#define AM35X_SCM_GEN_BASE   0x48002270
> > > > > +/* General register mappings of system control module */
> > > > > +struct am35x_scm_general {
> > > > > +     u32 res1[0xC4];         /* 0x000 - 0x30C */
> > > > > +     u32 devconf2;           /* 0x310 */
> > > > > +     u32 devconf3;           /* 0x314 */
> > > > > +     u32 res2[0x2];          /* 0x318 - 0x31C */
> > > > > +     u32 cba_priority;       /* 0x320 */
> > > > > +     u32 lvl_intr_clr;       /* 0x324 */
> > > > > +     u32 ip_sw_reset;        /* 0x328 */
> > > > > +     u32 ipss_clk_ctrl;      /* 0x32C */
> > > > > +};
> > > >
> > > > Shouldn't this structure be in a am35 specific header?
> > >
> > > AM35x is based on OMAP3x and moreover there is no AM35x specific
> > understood
> > > File so I added them here. Do you want me to add a header for
> > > AM35x as arch/arm/include/asm/arch-omap3/am35x.h ?
> > >
> > > Thanks,
> > > Ajay
> > > > > --
> >
> > What about
> > drivers/usb/musb/am35x.h
> > which you just created in your patch set
> 
> am35x_scm_general is a system register sets and can be used by non usb
> modules as well so I think drivers/usb/musb/am35x.h would not be a right
> place.


Agree.
Then in that case it is better in my opinion to have  
arch/arm/include/asm/arch-omap3/am35x.h as you mention above rather than 
clutter cpu.h

> Are you concerned of same name file instead? If so then we can have
> Different names for these files.
> 


I will also urge you to wait just a little because I have just sent Wolfgang a 
pull request with lots of changes OMAP3 and OMAP4. I would urge you to wait 
till he accepts the pull request.

Or you can make patches against u-boot-ti


_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to