From: Roland Stigge <stigge@antcom.de>
To: Brian Norris <computersforpeace@gmail.com>
Cc: Marek Vasut <marex@denx.de>, Huang Shijie <b32955@freescale.com>,
linux-mtd@lists.infradead.org,
Linux Kernel <linux-kernel@vger.kernel.org>,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2 05/12] ARM: lpc32xx: add MTD_SPI_NOR (new dependency for M25P80)
Date: Thu, 01 May 2014 19:21:06 +0200 [thread overview]
Message-ID: <53628282.10201@antcom.de> (raw)
In-Reply-To: <1398925607-7482-6-git-send-email-computersforpeace@gmail.com>
On 01/05/14 08:26, Brian Norris wrote:
> This defconfig contains the CONFIG_M25P80 symbol, which is now
> dependent on the MTD_SPI_NOR symbol. Add CONFIG_MTD_SPI_NOR to satisfy
> the new dependency.
Can't the CONFIG_M25P80 declare its dependency on CONFIG_MTD_SPI_NOR
itself, or put CONFIG_M25P80 etc. into a submenu under CONFIG_MTD_SPI_NOR?
>
> At the same time, drop the now-nonexistent CONFIG_MTD_CHAR symbol.
>
> Signed-off-by: Brian Norris <computersforpeace@gmail.com>
> Cc: Roland Stigge <stigge@antcom.de>
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-kernel@vger.kernel.org
> ---
> arch/arm/configs/lpc32xx_defconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/configs/lpc32xx_defconfig b/arch/arm/configs/lpc32xx_defconfig
> index 398a367ffce8..2de54c35fb47 100644
> --- a/arch/arm/configs/lpc32xx_defconfig
> +++ b/arch/arm/configs/lpc32xx_defconfig
> @@ -53,12 +53,12 @@ CONFIG_DEVTMPFS_MOUNT=y
> # CONFIG_FW_LOADER is not set
> CONFIG_MTD=y
> CONFIG_MTD_CMDLINE_PARTS=y
> -CONFIG_MTD_CHAR=y
> CONFIG_MTD_BLOCK=y
> CONFIG_MTD_M25P80=y
> CONFIG_MTD_NAND=y
> CONFIG_MTD_NAND_SLC_LPC32XX=y
> CONFIG_MTD_NAND_MLC_LPC32XX=y
> +CONFIG_MTD_SPI_NOR=y
> CONFIG_BLK_DEV_LOOP=y
> CONFIG_BLK_DEV_CRYPTOLOOP=y
> CONFIG_BLK_DEV_RAM=y
>
WARNING: multiple messages have this Message-ID (diff)
From: stigge@antcom.de (Roland Stigge)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v2 05/12] ARM: lpc32xx: add MTD_SPI_NOR (new dependency for M25P80)
Date: Thu, 01 May 2014 19:21:06 +0200 [thread overview]
Message-ID: <53628282.10201@antcom.de> (raw)
In-Reply-To: <1398925607-7482-6-git-send-email-computersforpeace@gmail.com>
On 01/05/14 08:26, Brian Norris wrote:
> This defconfig contains the CONFIG_M25P80 symbol, which is now
> dependent on the MTD_SPI_NOR symbol. Add CONFIG_MTD_SPI_NOR to satisfy
> the new dependency.
Can't the CONFIG_M25P80 declare its dependency on CONFIG_MTD_SPI_NOR
itself, or put CONFIG_M25P80 etc. into a submenu under CONFIG_MTD_SPI_NOR?
>
> At the same time, drop the now-nonexistent CONFIG_MTD_CHAR symbol.
>
> Signed-off-by: Brian Norris <computersforpeace@gmail.com>
> Cc: Roland Stigge <stigge@antcom.de>
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: linux-kernel at vger.kernel.org
> ---
> arch/arm/configs/lpc32xx_defconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/configs/lpc32xx_defconfig b/arch/arm/configs/lpc32xx_defconfig
> index 398a367ffce8..2de54c35fb47 100644
> --- a/arch/arm/configs/lpc32xx_defconfig
> +++ b/arch/arm/configs/lpc32xx_defconfig
> @@ -53,12 +53,12 @@ CONFIG_DEVTMPFS_MOUNT=y
> # CONFIG_FW_LOADER is not set
> CONFIG_MTD=y
> CONFIG_MTD_CMDLINE_PARTS=y
> -CONFIG_MTD_CHAR=y
> CONFIG_MTD_BLOCK=y
> CONFIG_MTD_M25P80=y
> CONFIG_MTD_NAND=y
> CONFIG_MTD_NAND_SLC_LPC32XX=y
> CONFIG_MTD_NAND_MLC_LPC32XX=y
> +CONFIG_MTD_SPI_NOR=y
> CONFIG_BLK_DEV_LOOP=y
> CONFIG_BLK_DEV_CRYPTOLOOP=y
> CONFIG_BLK_DEV_RAM=y
>
WARNING: multiple messages have this Message-ID (diff)
From: Roland Stigge <stigge@antcom.de>
To: Brian Norris <computersforpeace@gmail.com>
Cc: Linux Kernel <linux-kernel@vger.kernel.org>,
Huang Shijie <b32955@freescale.com>, Marek Vasut <marex@denx.de>,
linux-mtd@lists.infradead.org,
linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2 05/12] ARM: lpc32xx: add MTD_SPI_NOR (new dependency for M25P80)
Date: Thu, 01 May 2014 19:21:06 +0200 [thread overview]
Message-ID: <53628282.10201@antcom.de> (raw)
In-Reply-To: <1398925607-7482-6-git-send-email-computersforpeace@gmail.com>
On 01/05/14 08:26, Brian Norris wrote:
> This defconfig contains the CONFIG_M25P80 symbol, which is now
> dependent on the MTD_SPI_NOR symbol. Add CONFIG_MTD_SPI_NOR to satisfy
> the new dependency.
Can't the CONFIG_M25P80 declare its dependency on CONFIG_MTD_SPI_NOR
itself, or put CONFIG_M25P80 etc. into a submenu under CONFIG_MTD_SPI_NOR?
>
> At the same time, drop the now-nonexistent CONFIG_MTD_CHAR symbol.
>
> Signed-off-by: Brian Norris <computersforpeace@gmail.com>
> Cc: Roland Stigge <stigge@antcom.de>
> Cc: linux-arm-kernel@lists.infradead.org
> Cc: linux-kernel@vger.kernel.org
> ---
> arch/arm/configs/lpc32xx_defconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/arch/arm/configs/lpc32xx_defconfig b/arch/arm/configs/lpc32xx_defconfig
> index 398a367ffce8..2de54c35fb47 100644
> --- a/arch/arm/configs/lpc32xx_defconfig
> +++ b/arch/arm/configs/lpc32xx_defconfig
> @@ -53,12 +53,12 @@ CONFIG_DEVTMPFS_MOUNT=y
> # CONFIG_FW_LOADER is not set
> CONFIG_MTD=y
> CONFIG_MTD_CMDLINE_PARTS=y
> -CONFIG_MTD_CHAR=y
> CONFIG_MTD_BLOCK=y
> CONFIG_MTD_M25P80=y
> CONFIG_MTD_NAND=y
> CONFIG_MTD_NAND_SLC_LPC32XX=y
> CONFIG_MTD_NAND_MLC_LPC32XX=y
> +CONFIG_MTD_SPI_NOR=y
> CONFIG_BLK_DEV_LOOP=y
> CONFIG_BLK_DEV_CRYPTOLOOP=y
> CONFIG_BLK_DEV_RAM=y
>
next prev parent reply other threads:[~2014-05-01 17:21 UTC|newest]
Thread overview: 86+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-05-01 6:26 [PATCH v2 00/12] defconfigs: add MTD_SPI_NOR (new subsystem dependency for M25P80) Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` [PATCH v2 01/12] ARM: imx/mxs defconfigs: add MTD_SPI_NOR (new " Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:43 ` Paul Bolle
2014-05-01 6:43 ` Paul Bolle
2014-05-01 6:43 ` Paul Bolle
2014-05-06 13:55 ` Shawn Guo
2014-05-06 13:55 ` Shawn Guo
2014-05-06 13:55 ` Shawn Guo
2014-05-01 6:26 ` [PATCH v2 02/12] ARM: " Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 8:54 ` Simon Horman
2014-05-01 8:54 ` Simon Horman
2014-05-01 8:54 ` Simon Horman
2014-05-01 8:54 ` Simon Horman
2014-05-06 18:12 ` Brian Norris
2014-05-06 18:12 ` Brian Norris
2014-05-06 18:12 ` Brian Norris
2014-05-06 18:12 ` Brian Norris
2014-05-11 0:43 ` Simon Horman
2014-05-11 0:43 ` Simon Horman
2014-05-11 0:43 ` Simon Horman
2014-05-11 0:43 ` Simon Horman
2014-05-01 6:26 ` [PATCH v2 03/12] ARM: keystone: " Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 14:16 ` Santosh Shilimkar
2014-05-01 14:16 ` Santosh Shilimkar
2014-05-01 14:16 ` Santosh Shilimkar
2014-05-01 6:26 ` [PATCH v2 04/12] ARM: tegra: " Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 17:26 ` Stephen Warren
2014-05-01 17:26 ` Stephen Warren
2014-05-01 17:26 ` Stephen Warren
2014-05-01 6:26 ` [PATCH v2 05/12] ARM: lpc32xx: " Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 17:21 ` Roland Stigge [this message]
2014-05-01 17:21 ` Roland Stigge
2014-05-01 17:21 ` Roland Stigge
2014-05-06 17:15 ` Brian Norris
2014-05-06 17:15 ` Brian Norris
2014-05-06 17:15 ` Brian Norris
2014-05-01 6:26 ` [PATCH v2 06/12] ARM: at91: " Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-09-02 8:36 ` Nicolas Ferre
2014-09-02 8:36 ` Nicolas Ferre
2014-09-02 8:36 ` Nicolas Ferre
2014-05-01 6:26 ` [PATCH v2 07/12] ARM: shmobile: " Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` [PATCH v2 08/12] ARM: marvell: " Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 17:53 ` Ezequiel Garcia
2014-05-01 17:53 ` Ezequiel Garcia
2014-05-01 17:53 ` Ezequiel Garcia
2014-05-05 1:03 ` Jason Cooper
2014-05-05 1:03 ` Jason Cooper
2014-05-05 1:03 ` Jason Cooper
2014-05-01 6:26 ` [PATCH v2 09/12] blackfin: defconfigs: " Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-06-12 9:00 ` Steven Miao
2014-06-12 9:00 ` Steven Miao
2014-05-01 6:26 ` [PATCH v2 10/12] mips: " Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` [PATCH v2 11/12] powerpc: " Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` [PATCH v2 12/12] sh: defconfig: " Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-05-01 6:26 ` Brian Norris
2014-08-07 21:36 ` [PATCH v2 00/12] defconfigs: add MTD_SPI_NOR (new subsystem " Geert Uytterhoeven
2014-08-07 21:36 ` Geert Uytterhoeven
2014-08-07 21:36 ` Geert Uytterhoeven
2014-08-07 21:36 ` Geert Uytterhoeven
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=53628282.10201@antcom.de \
--to=stigge@antcom.de \
--cc=b32955@freescale.com \
--cc=computersforpeace@gmail.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mtd@lists.infradead.org \
--cc=marex@denx.de \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.