From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gupta, Ajay Kumar Date: Wed, 7 Jul 2010 19:09:54 +0530 Subject: [U-Boot] [PATCH 1/4] AM35x: Adding SCM general register definitions In-Reply-To: <0554BEF07D437848AF01B9C9B5F0BC5D9F0B6589@dlee01.ent.ti.com> References: <1278503860-12052-1-git-send-email-ajay.gupta@ti.com> <1278503860-12052-2-git-send-email-ajay.gupta@ti.com> <0554BEF07D437848AF01B9C9B5F0BC5D9F0B6559@dlee01.ent.ti.com> <19F8576C6E063C45BE387C64729E7394044EB959E4@dbde02.ent.ti.com> <0554BEF07D437848AF01B9C9B5F0BC5D9F0B6589@dlee01.ent.ti.com> Message-ID: <19F8576C6E063C45BE387C64729E7394044EB959F1@dbde02.ent.ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de Hi > > > > Adding general register structure of system control module (SCM) > > > > of AM35x. This would be required to access devconf2 and ip_sw_reset > > > > register in musb module. > > > > > > > > Signed-off-by: Ajay Kumar Gupta > > > > --- > > > > arch/arm/include/asm/arch-omap3/cpu.h | 20 ++++++++++++++++++++ > > > > 1 files changed, 20 insertions(+), 0 deletions(-) > > > > > > > > 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. Are you concerned of same name file instead? If so then we can have Different names for these files. Thanks, Ajay > > > Thanks, > Sandeep