mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Belisko Marek <marek.belisko@gmail.com>
To: "Premi, Sanjeev" <premi@ti.com>
Cc: "barebox@lists.infradead.org" <barebox@lists.infradead.org>
Subject: Re: [PATCH 1/2] omap: Remove OMAP_GPMC prompt and usage.
Date: Wed, 1 Feb 2012 11:16:19 +0100	[thread overview]
Message-ID: <CAAfyv35gncmFgiHkqmS6u2so9TcDYM6yodiCw6YAdOARMPwMFw@mail.gmail.com> (raw)
In-Reply-To: <E28AAFD00EFAA646AE3DF9B89CD24A89318237E1@DBDE01.ent.ti.com>

On Wed, Feb 1, 2012 at 10:32 AM, Premi, Sanjeev <premi@ti.com> wrote:
>> -----Original Message-----
>> From: barebox-bounces@lists.infradead.org
>> [mailto:barebox-bounces@lists.infradead.org] On Behalf Of
>> Marek Belisko
>> Sent: Wednesday, February 01, 2012 2:21 AM
>> To: barebox@lists.infradead.org
>> Cc: Marek Belisko
>> Subject: [PATCH 1/2] omap: Remove OMAP_GPMC prompt and usage.
>>
>> Build for omap platform by default.
>>
>> Signed-off-by: Marek Belisko <marek.belisko@open-nandra.com>
>> ---
>>  arch/arm/boards/beagle/board.c          |    4 ++--
>>  arch/arm/boards/omap343xdsp/board.c     |    2 --
>>  arch/arm/boards/omap3evm/board.c        |    3 +--
>>  arch/arm/boards/phycard-a-l1/pca-a-l1.c |    3 ---
>>  arch/arm/configs/panda_xload_defconfig  |    1 -
>>  arch/arm/configs/phycard_a_l1_defconfig |    1 -
>>  arch/arm/mach-omap/Kconfig              |   10 ----------
>>  arch/arm/mach-omap/Makefile             |    3 +--
>>  drivers/mtd/nand/Kconfig                |    1 -
>>  9 files changed, 4 insertions(+), 24 deletions(-)
>>
>> diff --git a/arch/arm/boards/beagle/board.c
>> b/arch/arm/boards/beagle/board.c
>> index faeaf8e..caeb578 100644
>> --- a/arch/arm/boards/beagle/board.c
>> +++ b/arch/arm/boards/beagle/board.c
>> @@ -302,10 +302,10 @@ static int beagle_devices_init(void)
>>               add_usb_ehci_device(-1, OMAP_EHCI_BASE,
>>                                   OMAP_EHCI_BASE + 0x10, &ehci_pdata);
>>  #endif /* CONFIG_USB_EHCI_OMAP */
>> -#ifdef CONFIG_OMAP_GPMC
>> +
>>       /* WP is made high and WAIT1 active Low */
>>       gpmc_generic_init(0x10);
>> -#endif
>> +
>>       gpmc_generic_nand_devices_init(0, 16,
>> OMAP_ECC_HAMMING_CODE_HW_ROMCODE);
>>
>>       add_generic_device("omap-hsmmc", -1, NULL,
>> OMAP_MMC1_BASE, SZ_4K,
>> diff --git a/arch/arm/boards/omap343xdsp/board.c
>> b/arch/arm/boards/omap343xdsp/board.c
>> index 4ad6f13..4cf6f31 100644
>> --- a/arch/arm/boards/omap343xdsp/board.c
>> +++ b/arch/arm/boards/omap343xdsp/board.c
>> @@ -641,10 +641,8 @@ mem_initcall(sdp3430_mem_init);
>>
>>  static int sdp3430_devices_init(void)
>>  {
>> -#ifdef CONFIG_OMAP_GPMC
>>       /* WP is made high and WAIT1 active Low */
>>       gpmc_generic_init(0x10);
>> -#endif
>>
>>       return 0;
>>  }
>> diff --git a/arch/arm/boards/omap3evm/board.c
>> b/arch/arm/boards/omap3evm/board.c
>> index fdd5227..ffcb6a6 100644
>> --- a/arch/arm/boards/omap3evm/board.c
>> +++ b/arch/arm/boards/omap3evm/board.c
>> @@ -253,12 +253,11 @@ mem_initcall(omap3evm_mem_init);
>>
>>  static int omap3evm_init_devices(void)
>>  {
>> -#ifdef CONFIG_OMAP_GPMC
>>       /*
>>        * WP is made high and WAIT1 active Low
>>        */
>>       gpmc_generic_init(0x10);
>> -#endif
>> +
>>  #ifdef CONFIG_MCI_OMAP_HSMMC
>>       add_generic_device("omap-hsmmc", -1, NULL,
>> OMAP_MMC1_BASE, SZ_4K,
>>                               IORESOURCE_MEM, NULL);
>> diff --git a/arch/arm/boards/phycard-a-l1/pca-a-l1.c
>> b/arch/arm/boards/phycard-a-l1/pca-a-l1.c
>> index f7d3185..7a1196e 100644
>> --- a/arch/arm/boards/phycard-a-l1/pca-a-l1.c
>> +++ b/arch/arm/boards/phycard-a-l1/pca-a-l1.c
>> @@ -288,13 +288,10 @@ static void pcaal1_setup_net_chip(void)
>>
>>  static int pcaal1_mem_init(void)
>>  {
>> -
>> -#ifdef CONFIG_OMAP_GPMC
>>       /*
>>        * WP is made high and WAIT1 active Low
>>        */
>>       gpmc_generic_init(0x10);
>> -#endif
>>
>>       arm_add_mem_device("ram0", OMAP_SDRC_CS0,
>> get_sdr_cs_size(SDRC_CS0_OSET));
>>
>> diff --git a/arch/arm/configs/panda_xload_defconfig
>> b/arch/arm/configs/panda_xload_defconfig
>> index f8f0ae9..bdfbeb9 100644
>> --- a/arch/arm/configs/panda_xload_defconfig
>> +++ b/arch/arm/configs/panda_xload_defconfig
>> @@ -1,6 +1,5 @@
>>  CONFIG_ARCH_OMAP=y
>>  CONFIG_ARCH_OMAP4=y
>> -# CONFIG_OMAP_GPMC is not set
>>  CONFIG_OMAP_BUILD_IFT=y
>>  CONFIG_AEABI=y
>>  # CONFIG_CMD_ARM_CPUINFO is not set
>> diff --git a/arch/arm/configs/phycard_a_l1_defconfig
>> b/arch/arm/configs/phycard_a_l1_defconfig
>> index 95dffbb..9ccf43d 100644
>> --- a/arch/arm/configs/phycard_a_l1_defconfig
>> +++ b/arch/arm/configs/phycard_a_l1_defconfig
>> @@ -11,7 +11,6 @@ CONFIG_OMAP_CLOCK_ALL=y
>>  CONFIG_OMAP_CLOCK_SOURCE_S32K=y
>>  CONFIG_OMAP3_CLOCK_CONFIG=y
>>  CONFIG_OMAP3_COPY_CLOCK_SRAM=n
>> -CONFIG_OMAP_GPMC=y
>>  CONFIG_MACH_PCAAL1=y
>>  CONFIG_HAS_OMAP_NAND=y
>>  CONFIG_AEABI=y
>> diff --git a/arch/arm/mach-omap/Kconfig b/arch/arm/mach-omap/Kconfig
>> index 50f53dd..11b9661 100644
>> --- a/arch/arm/mach-omap/Kconfig
>> +++ b/arch/arm/mach-omap/Kconfig
>> @@ -77,16 +77,6 @@ config OMAP3_CLOCK_CONFIG
>>       help
>>         Say Y here if you like to have OMAP3 Clock configuration done.
>>
>> -config OMAP_GPMC
>> -     prompt "Support for GPMC configuration"
>> -     bool
>> -     depends on  (ARCH_OMAP2 || ARCH_OMAP3 || ARCH_OMAP4)
>> -     default y
>> -     help
>> -       Enable this if you use Texas Instrument's General
>> purpose Memory
>> -       Controller(GPMC). GPMC allows you to configure
>> devices such as NOR,
>> -       NAND, OneNAND etc.
>> -
>>  config OMAP_BUILD_IFT
>>       prompt "build ift binary"
>>       bool
>> diff --git a/arch/arm/mach-omap/Makefile b/arch/arm/mach-omap/Makefile
>> index 87078ae..26d3f7d 100644
>> --- a/arch/arm/mach-omap/Makefile
>> +++ b/arch/arm/mach-omap/Makefile
>> @@ -24,7 +24,6 @@ obj-$(CONFIG_OMAP_CLOCK_SOURCE_S32K) +=
>> s32k_clksource.o
>>  obj-$(CONFIG_ARCH_OMAP3) += omap3_core.o omap3_generic.o auxcr.o
>>  obj-$(CONFIG_ARCH_OMAP4) += omap4_generic.o omap4_clock.o
>>  obj-$(CONFIG_OMAP3_CLOCK_CONFIG) += omap3_clock.o
>> -obj-$(CONFIG_OMAP_GPMC) += gpmc.o devices-gpmc-nand.o
>>  obj-$(CONFIG_SHELL_NONE) += xload.o
>>  obj-$(CONFIG_I2C_TWL6030) += omap4_twl6030_mmc.o
>> -obj-y += gpio.o
>> +obj-y += gpio.o gpmc.o devices-gpmc-nand.o
>> diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig
>> index 926a64b..d846ce0 100644
>> --- a/drivers/mtd/nand/Kconfig
>> +++ b/drivers/mtd/nand/Kconfig
>> @@ -55,7 +55,6 @@ config NAND_IMX
>>
>>  config NAND_OMAP_GPMC
>>       tristate "NAND Flash Support for GPMC based OMAP platforms"
>> -     depends on OMAP_GPMC
>>       help
>>        Support for NAND flash using GPMC. GPMC is a common memory
>>        interface found on Texas Instrument's OMAP platforms
>> --
>
> The config option helps in reducing the extra code being built into
> the "xload" configurations - where the size matters.
>
> Instead of making this change, I suggest you add this in the defconfig
> for pandaboard:
> CONFIG_OMAP_GPMC=y
>
> It is much smaller change.
GPMC is common for omap platform so I don't see a benefit to add user
possibility to disable it and
then make his board un-bootable. Solution was proposed by Sascha :
http://www.spinics.net/lists/u-boot-v2/msg05897.html

>
> ~sanjeev
>
>> 1.7.5.4
>>
>>
>> _______________________________________________
>> barebox mailing list
>> barebox@lists.infradead.org
>> http://lists.infradead.org/mailman/listinfo/barebox
>>

regards,

marek

-- 
as simple and primitive as possible
-------------------------------------------------
Marek Belisko - OPEN-NANDRA
Freelance Developer

Ruska Nova Ves 219 | Presov, 08005 Slovak Republic
Tel: +421 915 052 184
skype: marekwhite
twitter: #opennandra
web: http://open-nandra.com

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox

  reply	other threads:[~2012-02-01 10:16 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-31 20:51 Marek Belisko
2012-01-31 20:51 ` [PATCH 2/2] pandaboard: Add gpmc_generic_init to board init Marek Belisko
2012-03-02  8:04   ` Jan Weitzel
2012-03-03 12:11     ` Belisko Marek
2012-02-01  9:32 ` [PATCH 1/2] omap: Remove OMAP_GPMC prompt and usage Premi, Sanjeev
2012-02-01 10:16   ` Belisko Marek [this message]
2012-02-01 11:05     ` Premi, Sanjeev
2012-02-01 11:09       ` Belisko Marek
2012-02-01 13:52         ` Premi, Sanjeev
2012-02-01 13:51       ` Sascha Hauer
2012-02-01 14:10         ` Premi, Sanjeev
2012-02-07 20:22           ` Belisko Marek

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=CAAfyv35gncmFgiHkqmS6u2so9TcDYM6yodiCw6YAdOARMPwMFw@mail.gmail.com \
    --to=marek.belisko@gmail.com \
    --cc=barebox@lists.infradead.org \
    --cc=premi@ti.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox