* [PATCH 0/2] MIPS: Enable BCM63xx partition parser in defconfigs @ 2016-01-27 20:13 Florian Fainelli 2016-01-27 20:13 ` [PATCH 1/2] MIPS: BCM63xx: Enable partition parser in defconfig Florian Fainelli 2016-01-27 20:13 ` [PATCH 2/2] MIPS: BMIPS: " Florian Fainelli 0 siblings, 2 replies; 7+ messages in thread From: Florian Fainelli @ 2016-01-27 20:13 UTC (permalink / raw) To: linux-mips Cc: ralf, computersforpeace, cernekee, jogo, simon, Florian Fainelli This patch series enables the BCM63xx partition parser where it is relevant: BCM63xx SoCs DSL defconfig, and BMIPS big-endian defconfig (for DSL and CM SoCS). Having this will improve the build coverage for that driver moving forward, and is also needed for booting from MTD devices. Florian Fainelli (2): MIPS: BCM63xx: Enable partition parser in defconfig MIPS: BMIPS: Enable partition parser in defconfig arch/mips/configs/bcm63xx_defconfig | 1 + arch/mips/configs/bmips_be_defconfig | 1 + 2 files changed, 2 insertions(+) -- 2.1.0 ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 1/2] MIPS: BCM63xx: Enable partition parser in defconfig 2016-01-27 20:13 [PATCH 0/2] MIPS: Enable BCM63xx partition parser in defconfigs Florian Fainelli @ 2016-01-27 20:13 ` Florian Fainelli 2016-01-27 20:21 ` Brian Norris 2016-01-27 20:13 ` [PATCH 2/2] MIPS: BMIPS: " Florian Fainelli 1 sibling, 1 reply; 7+ messages in thread From: Florian Fainelli @ 2016-01-27 20:13 UTC (permalink / raw) To: linux-mips Cc: ralf, computersforpeace, cernekee, jogo, simon, Florian Fainelli Enable CONFIG_MTD_BCM63XX_PARTS in arch/mips/configs/bcm63xx_defconfig since this is a necessary option to parse the built-in flash partition table on BCM63xx SoCs. Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> --- arch/mips/configs/bcm63xx_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/mips/configs/bcm63xx_defconfig b/arch/mips/configs/bcm63xx_defconfig index 3fec26410f34..5599a9f1e3c6 100644 --- a/arch/mips/configs/bcm63xx_defconfig +++ b/arch/mips/configs/bcm63xx_defconfig @@ -44,6 +44,7 @@ CONFIG_UEVENT_HELPER_PATH="/sbin/hotplug" # CONFIG_STANDALONE is not set # CONFIG_PREVENT_FIRMWARE_BUILD is not set CONFIG_MTD=y +CONFIG_MTD_BCM63XX_PARTS=y CONFIG_MTD_CFI=y CONFIG_MTD_CFI_INTELEXT=y CONFIG_MTD_CFI_AMDSTD=y -- 2.1.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] MIPS: BCM63xx: Enable partition parser in defconfig 2016-01-27 20:13 ` [PATCH 1/2] MIPS: BCM63xx: Enable partition parser in defconfig Florian Fainelli @ 2016-01-27 20:21 ` Brian Norris 0 siblings, 0 replies; 7+ messages in thread From: Brian Norris @ 2016-01-27 20:21 UTC (permalink / raw) To: Florian Fainelli; +Cc: linux-mips, ralf, cernekee, jogo, simon On Wed, Jan 27, 2016 at 12:13:15PM -0800, Florian Fainelli wrote: > Enable CONFIG_MTD_BCM63XX_PARTS in arch/mips/configs/bcm63xx_defconfig > since this is a necessary option to parse the built-in flash partition > table on BCM63xx SoCs. > > Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> Acked-by: Brian Norris <computersforpeace@gmail.com> ^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCH 2/2] MIPS: BMIPS: Enable partition parser in defconfig 2016-01-27 20:13 [PATCH 0/2] MIPS: Enable BCM63xx partition parser in defconfigs Florian Fainelli 2016-01-27 20:13 ` [PATCH 1/2] MIPS: BCM63xx: Enable partition parser in defconfig Florian Fainelli @ 2016-01-27 20:13 ` Florian Fainelli 2016-01-27 20:23 ` Brian Norris 1 sibling, 1 reply; 7+ messages in thread From: Florian Fainelli @ 2016-01-27 20:13 UTC (permalink / raw) To: linux-mips Cc: ralf, computersforpeace, cernekee, jogo, simon, Florian Fainelli Enable CONFIG_MTD_BCM63XX_PARTS in arch/mips/configs/bmips_be_defconfig since this is a necessary option to parse the built-in flash partition table on BMIPS big-endian SoCs (Cable Modem and DSL). Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> --- arch/mips/configs/bmips_be_defconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/arch/mips/configs/bmips_be_defconfig b/arch/mips/configs/bmips_be_defconfig index 24dcb90b0f64..acf7785c4cdb 100644 --- a/arch/mips/configs/bmips_be_defconfig +++ b/arch/mips/configs/bmips_be_defconfig @@ -36,6 +36,7 @@ CONFIG_DEVTMPFS_MOUNT=y CONFIG_PRINTK_TIME=y CONFIG_BRCMSTB_GISB_ARB=y CONFIG_MTD=y +CONFIG_MTD_BCM63XX_PARTS=y CONFIG_MTD_CFI=y CONFIG_MTD_CFI_INTELEXT=y CONFIG_MTD_CFI_AMDSTD=y -- 2.1.0 ^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] MIPS: BMIPS: Enable partition parser in defconfig 2016-01-27 20:13 ` [PATCH 2/2] MIPS: BMIPS: " Florian Fainelli @ 2016-01-27 20:23 ` Brian Norris 2016-01-27 20:29 ` Florian Fainelli 0 siblings, 1 reply; 7+ messages in thread From: Brian Norris @ 2016-01-27 20:23 UTC (permalink / raw) To: Florian Fainelli; +Cc: linux-mips, ralf, cernekee, jogo, simon On Wed, Jan 27, 2016 at 12:13:16PM -0800, Florian Fainelli wrote: > Enable CONFIG_MTD_BCM63XX_PARTS in arch/mips/configs/bmips_be_defconfig > since this is a necessary option to parse the built-in flash partition > table on BMIPS big-endian SoCs (Cable Modem and DSL). > > Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> > --- > arch/mips/configs/bmips_be_defconfig | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/mips/configs/bmips_be_defconfig b/arch/mips/configs/bmips_be_defconfig > index 24dcb90b0f64..acf7785c4cdb 100644 > --- a/arch/mips/configs/bmips_be_defconfig > +++ b/arch/mips/configs/bmips_be_defconfig > @@ -36,6 +36,7 @@ CONFIG_DEVTMPFS_MOUNT=y > CONFIG_PRINTK_TIME=y > CONFIG_BRCMSTB_GISB_ARB=y > CONFIG_MTD=y > +CONFIG_MTD_BCM63XX_PARTS=y ^^ This doens't help, AFAICT, because this config doesn't have CONFIG_BCM63XX yet, which CONFIG_MTD_BCM63XX_PARTS depends on. Or, is that part of what this series will help with: removing the dependency on CONFIG_BCM63XX? http://lists.infradead.org/pipermail/linux-mtd/2015-December/064380.html Brian > CONFIG_MTD_CFI=y > CONFIG_MTD_CFI_INTELEXT=y > CONFIG_MTD_CFI_AMDSTD=y > -- > 2.1.0 > ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] MIPS: BMIPS: Enable partition parser in defconfig 2016-01-27 20:23 ` Brian Norris @ 2016-01-27 20:29 ` Florian Fainelli 2016-01-27 22:21 ` Brian Norris 0 siblings, 1 reply; 7+ messages in thread From: Florian Fainelli @ 2016-01-27 20:29 UTC (permalink / raw) To: Brian Norris; +Cc: linux-mips, ralf, cernekee, jogo, simon On 27/01/16 12:23, Brian Norris wrote: > On Wed, Jan 27, 2016 at 12:13:16PM -0800, Florian Fainelli wrote: >> Enable CONFIG_MTD_BCM63XX_PARTS in arch/mips/configs/bmips_be_defconfig >> since this is a necessary option to parse the built-in flash partition >> table on BMIPS big-endian SoCs (Cable Modem and DSL). >> >> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> >> --- >> arch/mips/configs/bmips_be_defconfig | 1 + >> 1 file changed, 1 insertion(+) >> >> diff --git a/arch/mips/configs/bmips_be_defconfig b/arch/mips/configs/bmips_be_defconfig >> index 24dcb90b0f64..acf7785c4cdb 100644 >> --- a/arch/mips/configs/bmips_be_defconfig >> +++ b/arch/mips/configs/bmips_be_defconfig >> @@ -36,6 +36,7 @@ CONFIG_DEVTMPFS_MOUNT=y >> CONFIG_PRINTK_TIME=y >> CONFIG_BRCMSTB_GISB_ARB=y >> CONFIG_MTD=y >> +CONFIG_MTD_BCM63XX_PARTS=y > > ^^ This doens't help, AFAICT, because this config doesn't have > CONFIG_BCM63XX yet, which CONFIG_MTD_BCM63XX_PARTS depends on. > > Or, is that part of what this series will help with: removing the > dependency on CONFIG_BCM63XX? Yes, I was assuming this would be relative to the patch series below so this driver gets usable on BMIPS as well. I leave to you and Ralf to resolve how you want to pick up patches in this series and resolve the merge. > > http://lists.infradead.org/pipermail/linux-mtd/2015-December/064380.html > > Brian > >> CONFIG_MTD_CFI=y >> CONFIG_MTD_CFI_INTELEXT=y >> CONFIG_MTD_CFI_AMDSTD=y >> -- >> 2.1.0 >> -- Florian ^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] MIPS: BMIPS: Enable partition parser in defconfig 2016-01-27 20:29 ` Florian Fainelli @ 2016-01-27 22:21 ` Brian Norris 0 siblings, 0 replies; 7+ messages in thread From: Brian Norris @ 2016-01-27 22:21 UTC (permalink / raw) To: Florian Fainelli; +Cc: linux-mips, ralf, cernekee, jogo, simon On Wed, Jan 27, 2016 at 12:29:37PM -0800, Florian Fainelli wrote: > On 27/01/16 12:23, Brian Norris wrote: > > On Wed, Jan 27, 2016 at 12:13:16PM -0800, Florian Fainelli wrote: > >> Enable CONFIG_MTD_BCM63XX_PARTS in arch/mips/configs/bmips_be_defconfig > >> since this is a necessary option to parse the built-in flash partition > >> table on BMIPS big-endian SoCs (Cable Modem and DSL). > >> > >> Signed-off-by: Florian Fainelli <f.fainelli@gmail.com> > >> --- > >> --- a/arch/mips/configs/bmips_be_defconfig > >> +++ b/arch/mips/configs/bmips_be_defconfig > >> @@ -36,6 +36,7 @@ CONFIG_DEVTMPFS_MOUNT=y > >> CONFIG_PRINTK_TIME=y > >> CONFIG_BRCMSTB_GISB_ARB=y > >> CONFIG_MTD=y > >> +CONFIG_MTD_BCM63XX_PARTS=y > > > > ^^ This doens't help, AFAICT, because this config doesn't have > > CONFIG_BCM63XX yet, which CONFIG_MTD_BCM63XX_PARTS depends on. > > > > Or, is that part of what this series will help with: removing the > > dependency on CONFIG_BCM63XX? > > Yes, I was assuming this would be relative to the patch series below so > this driver gets usable on BMIPS as well. OK, that's fine. This patch can actually be taken independently, as this entry will just get dropped, until the MTD patch gets merged. In that case: Acked-by: Brian Norris <computersforpeace@gmail.com> This will make sure we get immediate defconfig build coverage as soon as the CONFIG_BCM63XX dependency is relaxed in drivers/mtd/. > I leave to you and Ralf to resolve how you want to pick up patches in > this series and resolve the merge. I think Ralf is now taking care of un-breaking the build, and I'll review Simon's MTD patches soon, I expect. > > > > http://lists.infradead.org/pipermail/linux-mtd/2015-December/064380.html Thanks, Brian ^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-01-27 22:21 UTC | newest] Thread overview: 7+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2016-01-27 20:13 [PATCH 0/2] MIPS: Enable BCM63xx partition parser in defconfigs Florian Fainelli 2016-01-27 20:13 ` [PATCH 1/2] MIPS: BCM63xx: Enable partition parser in defconfig Florian Fainelli 2016-01-27 20:21 ` Brian Norris 2016-01-27 20:13 ` [PATCH 2/2] MIPS: BMIPS: " Florian Fainelli 2016-01-27 20:23 ` Brian Norris 2016-01-27 20:29 ` Florian Fainelli 2016-01-27 22:21 ` Brian Norris
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.