From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kevin Hilman Subject: Re: [PATCH 2/3] AM35xx: Introduce am35xx.h file Date: Thu, 07 Jan 2010 11:14:16 -0800 Message-ID: <878wc920dz.fsf@deeprootsystems.com> References: <1262613116-20185-3-git-send-email-hvaibhav@ti.com> <873a2iaj42.fsf@deeprootsystems.com> <19F8576C6E063C45BE387C64729E7394044A397EDB@dbde02.ent.ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Received: from mail-gx0-f211.google.com ([209.85.217.211]:42756 "EHLO mail-gx0-f211.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752542Ab0AGTOU (ORCPT ); Thu, 7 Jan 2010 14:14:20 -0500 Received: by gxk3 with SMTP id 3so7891526gxk.1 for ; Thu, 07 Jan 2010 11:14:19 -0800 (PST) In-Reply-To: <19F8576C6E063C45BE387C64729E7394044A397EDB@dbde02.ent.ti.com> (Vaibhav Hiremath's message of "Thu\, 7 Jan 2010 23\:08\:54 +0530") Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: "Hiremath, Vaibhav" Cc: "linux-omap@vger.kernel.org" , "tony@atomide.com" , "Govindarajan, Sriramakrishnan" "Hiremath, Vaibhav" writes: >> -----Original Message----- >> From: Kevin Hilman [mailto:khilman@deeprootsystems.com] >> Sent: Thursday, January 07, 2010 5:20 AM >> To: Hiremath, Vaibhav >> Cc: linux-omap@vger.kernel.org; tony@atomide.com; Govindarajan, >> Sriramakrishnan >> Subject: Re: [PATCH 2/3] AM35xx: Introduce am35xx.h file >> >> hvaibhav@ti.com writes: >> >> > From: Vaibhav Hiremath >> > >> > Add base address definations for new AM35xx IPSS modules, like >> > VPFE, USBOTG, CPGMAC. >> > >> > Signed-off-by: Vaibhav Hiremath >> > Signed-off-by: Sriramakrishnan >> > --- >> > arch/arm/plat-omap/include/plat/am35xx.h | 25 >> +++++++++++++++++++++++++ >> > arch/arm/plat-omap/include/plat/hardware.h | 1 + >> > 2 files changed, 26 insertions(+), 0 deletions(-) >> > create mode 100644 arch/arm/plat-omap/include/plat/am35xx.h >> > >> > diff --git a/arch/arm/plat-omap/include/plat/am35xx.h >> b/arch/arm/plat-omap/include/plat/am35xx.h >> > new file mode 100644 >> > index 0000000..d41bd61 >> > --- /dev/null >> > +++ b/arch/arm/plat-omap/include/plat/am35xx.h >> > @@ -0,0 +1,25 @@ >> > +/*: >> > + * Address mappings and base address for AM35XX specific >> interconnects >> > + * and peripherals. >> > + * >> > + * Copyright (C) 2009 Texas Instruments >> > + * >> > + * Author: Sriramakrishnan >> > + * Vaibhav Hiremath >> > + * >> > + * This program is free software; you can redistribute it and/or >> modify >> > + * it under the terms of the GNU General Public License version 2 >> as >> > + * published by the Free Software Foundation. >> > + */ >> > +#ifndef __ASM_ARCH_AM35XX_H >> > +#define __ASM_ARCH_AM35XX_H >> > + >> > +/* >> > + * Base addresses >> > + */ >> > +#define AM35XX_IPSS_EMAC_BASE 0x5C000000 >> > +#define AM35XX_IPSS_USBOTGSS_BASE 0x5C040000 >> > +#define AM35XX_IPSS_HECC_BASE 0x5C050000 >> > +#define AM35XX_IPSS_VPFE_BASE 0x5C060000 >> >> I think this secont needs ome more comments. In the 34xx TRM memory >> map, this region is shown as IVA2, so folks familiar with 34xx might >> be confused. >> > [Hiremath, Vaibhav] I will update the comment with sufficient info. > >> That being said, is there any reason to have these in a global >> header >> file? Nobody should use these except SoC/board init code, so these >> should probably stay in arch/arm/mach-omap2 in AM35x specific code. >> > [Hiremath, Vaibhav] Agreed, but there are some occurrences where these macros may required to use in plat-omap/ directory. Just for an example - > > Macro OMAP443X_SCM_BASE, defined in > arch/arm/plat-omap/include/plat/omap44xx.h file is getting used in > arch/arm/plat-omap/common.c file. OK, but SCM_BASE has nothing to do with the base addresses you're adding in this patch. My point is that I don't see a need for the values in this patch to be defined in a global header. Kevin