[PATCH 0/2] MIPS: Enable BCM63xx partition parser in defconfigs

[PATCH 0/2] MIPS: Enable BCM63xx partition parser in defconfigs

[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

[PATCH 1/2] MIPS: BCM63xx: Enable partition parser in defconfig

[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

[PATCH 2/2] MIPS: BMIPS: Enable partition parser in defconfig

[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

Re: [PATCH 1/2] MIPS: BCM63xx: Enable partition parser in defconfig

[Brian Norris]

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>

Re: [PATCH 2/2] MIPS: BMIPS: Enable partition parser in defconfig

[Brian Norris]

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
> 

Re: [PATCH 2/2] MIPS: BMIPS: Enable partition parser in defconfig

[Florian Fainelli]

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

Re: [PATCH 2/2] MIPS: BMIPS: Enable partition parser in defconfig

[Brian Norris]

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