2018-08-14 15:54:55

by Marcel Ziswiler

[permalink] [raw]
Subject: [RESEND PATCH 1/6] ARM: config: multi_v5: enable dynamic debug

From: Marcel Ziswiler <[email protected]>

Dynamic debug is really helpful.

Signed-off-by: Marcel Ziswiler <[email protected]>

---

arch/arm/configs/multi_v5_defconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/multi_v5_defconfig b/arch/arm/configs/multi_v5_defconfig
index 318b76fa26d1..b302ac68d587 100644
--- a/arch/arm/configs/multi_v5_defconfig
+++ b/arch/arm/configs/multi_v5_defconfig
@@ -275,6 +275,7 @@ CONFIG_NLS_CODEPAGE_850=y
CONFIG_NLS_ISO8859_1=y
CONFIG_NLS_ISO8859_2=y
CONFIG_NLS_UTF8=y
+CONFIG_DYNAMIC_DEBUG=y
CONFIG_DEBUG_INFO=y
CONFIG_DEBUG_FS=y
CONFIG_MAGIC_SYSRQ=y
--
2.14.4



2018-08-14 14:12:03

by Marcel Ziswiler

[permalink] [raw]
Subject: [RESEND PATCH 2/6] ARM: multi_v7_defconfig: update order

From: Marcel Ziswiler <[email protected]>

The order of some kernel configurations changed.

Signed-off-by: Marcel Ziswiler <[email protected]>

---

arch/arm/configs/multi_v7_defconfig | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index fc33444e94f0..1299c0cf2f3a 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -48,8 +48,8 @@ CONFIG_SOC_IMX6Q=y
CONFIG_SOC_IMX6SL=y
CONFIG_SOC_IMX6SX=y
CONFIG_SOC_IMX6UL=y
-CONFIG_SOC_IMX7D=y
CONFIG_SOC_LS1021A=y
+CONFIG_SOC_IMX7D=y
CONFIG_SOC_VF610=y
CONFIG_ARCH_KEYSTONE=y
CONFIG_ARCH_MEDIATEK=y
@@ -495,8 +495,6 @@ CONFIG_MFD_AC100=y
CONFIG_MFD_AXP20X_I2C=y
CONFIG_MFD_AXP20X_RSB=y
CONFIG_MFD_CROS_EC=m
-CONFIG_CROS_EC_I2C=m
-CONFIG_CROS_EC_SPI=m
CONFIG_MFD_DA9063=m
CONFIG_MFD_MAX14577=y
CONFIG_MFD_MAX77686=y
@@ -866,6 +864,8 @@ CONFIG_SERIO_NVEC_PS2=y
CONFIG_NVEC_POWER=y
CONFIG_NVEC_PAZ00=y
CONFIG_STAGING_BOARD=y
+CONFIG_CROS_EC_I2C=m
+CONFIG_CROS_EC_SPI=m
CONFIG_COMMON_CLK_MAX77686=y
CONFIG_COMMON_CLK_RK808=m
CONFIG_COMMON_CLK_S2MPS11=m
--
2.14.4


2018-08-14 14:12:17

by Marcel Ziswiler

[permalink] [raw]
Subject: [RESEND PATCH 3/6] ARM: multi_v7_defconfig: enable dynamic debug

From: Marcel Ziswiler <[email protected]>

Dynamic debug is really helpful.

Signed-off-by: Marcel Ziswiler <[email protected]>

---

arch/arm/configs/multi_v7_defconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index 1299c0cf2f3a..65646b39430b 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -976,6 +976,7 @@ CONFIG_NLS_CODEPAGE_437=y
CONFIG_NLS_ISO8859_1=y
CONFIG_NLS_UTF8=y
CONFIG_PRINTK_TIME=y
+CONFIG_DYNAMIC_DEBUG=y
CONFIG_MAGIC_SYSRQ=y
CONFIG_CRYPTO_USER=m
CONFIG_CRYPTO_USER_API_HASH=m
--
2.14.4


2018-08-14 14:12:18

by Marcel Ziswiler

[permalink] [raw]
Subject: [RESEND PATCH 4/6] ARM: defconfig: tegra: enable dynamic debug

From: Marcel Ziswiler <[email protected]>

Dynamic debug is really helpful.

Signed-off-by: Marcel Ziswiler <[email protected]>

---

arch/arm/configs/tegra_defconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/tegra_defconfig b/arch/arm/configs/tegra_defconfig
index c7b99ebf5fcf..24e297659b55 100644
--- a/arch/arm/configs/tegra_defconfig
+++ b/arch/arm/configs/tegra_defconfig
@@ -289,6 +289,7 @@ CONFIG_ROOT_NFS=y
CONFIG_NLS_CODEPAGE_437=y
CONFIG_NLS_ISO8859_1=y
CONFIG_PRINTK_TIME=y
+CONFIG_DYNAMIC_DEBUG=y
CONFIG_DEBUG_INFO=y
CONFIG_MAGIC_SYSRQ=y
CONFIG_DEBUG_SLAB=y
--
2.14.4


2018-08-14 14:12:21

by Marcel Ziswiler

[permalink] [raw]
Subject: [RESEND PATCH 6/6] ARM: defconfig: tegra: enable tegra raw nand driver

From: Marcel Ziswiler <[email protected]>

Now with everything being merged actually enable it. Also enable
MTD_OF_PARTS, MTD_BLOCK, MTD_NAND, MTD_UBI and UBIFS without which
the former would not make much sense and is also in-line with
multi_v7_defconfig.

Signed-off-by: Marcel Ziswiler <[email protected]>

---

arch/arm/configs/tegra_defconfig | 6 ++++++
1 file changed, 6 insertions(+)

diff --git a/arch/arm/configs/tegra_defconfig b/arch/arm/configs/tegra_defconfig
index 24e297659b55..b248e231900f 100644
--- a/arch/arm/configs/tegra_defconfig
+++ b/arch/arm/configs/tegra_defconfig
@@ -79,8 +79,13 @@ CONFIG_DMA_CMA=y
CONFIG_CMA_SIZE_MBYTES=64
CONFIG_TEGRA_GMI=y
CONFIG_MTD=y
+CONFIG_MTD_OF_PARTS=y
+CONFIG_MTD_BLOCK=y
CONFIG_MTD_M25P80=y
+CONFIG_MTD_NAND=y
+CONFIG_MTD_NAND_TEGRA=y
CONFIG_MTD_SPI_NOR=y
+CONFIG_MTD_UBI=y
CONFIG_BLK_DEV_LOOP=y
CONFIG_AD525X_DPOT=y
CONFIG_AD525X_DPOT_I2C=y
@@ -280,6 +285,7 @@ CONFIG_EXT3_FS_SECURITY=y
CONFIG_VFAT_FS=y
CONFIG_TMPFS=y
CONFIG_TMPFS_POSIX_ACL=y
+CONFIG_UBIFS_FS=y
CONFIG_SQUASHFS=y
CONFIG_SQUASHFS_LZO=y
CONFIG_SQUASHFS_XZ=y
--
2.14.4


2018-08-14 14:13:21

by Marcel Ziswiler

[permalink] [raw]
Subject: [RESEND PATCH 5/6] ARM: multi_v7_defconfig: enable tegra raw nand driver

From: Marcel Ziswiler <[email protected]>

Now with everything being merged actually enable it.

Signed-off-by: Marcel Ziswiler <[email protected]>

---

arch/arm/configs/multi_v7_defconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index 65646b39430b..5420b48ded2a 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -190,6 +190,7 @@ CONFIG_MTD_NAND_GPMI_NAND=y
CONFIG_MTD_NAND_BRCMNAND=y
CONFIG_MTD_NAND_VF610_NFC=y
CONFIG_MTD_NAND_DAVINCI=y
+CONFIG_MTD_NAND_TEGRA=y
CONFIG_MTD_SPI_NOR=y
CONFIG_SPI_FSL_QUADSPI=m
CONFIG_MTD_UBI=y
--
2.14.4


2018-08-14 14:18:48

by Joe Perches

[permalink] [raw]
Subject: Re: [RESEND PATCH 1/6] ARM: config: multi_v5: enable dynamic debug

On Tue, 2018-08-14 at 16:06 +0200, Marcel Ziswiler wrote:
> From: Marcel Ziswiler <[email protected]>
>
> Dynamic debug is really helpful.

It's useful, but perhaps not appropriate.

Most arm configs are for small resource constrained systems.
dynamic_debug adds a lot of additional text and a fair amount of data.


> Signed-off-by: Marcel Ziswiler <[email protected]>
>
> ---
>
> arch/arm/configs/multi_v5_defconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm/configs/multi_v5_defconfig b/arch/arm/configs/multi_v5_defconfig
> index 318b76fa26d1..b302ac68d587 100644
> --- a/arch/arm/configs/multi_v5_defconfig
> +++ b/arch/arm/configs/multi_v5_defconfig
> @@ -275,6 +275,7 @@ CONFIG_NLS_CODEPAGE_850=y
> CONFIG_NLS_ISO8859_1=y
> CONFIG_NLS_ISO8859_2=y
> CONFIG_NLS_UTF8=y
> +CONFIG_DYNAMIC_DEBUG=y
> CONFIG_DEBUG_INFO=y
> CONFIG_DEBUG_FS=y
> CONFIG_MAGIC_SYSRQ=y

2018-08-16 04:25:25

by Florian Fainelli

[permalink] [raw]
Subject: Re: [RESEND PATCH 1/6] ARM: config: multi_v5: enable dynamic debug



On 08/14/2018 07:16 AM, Joe Perches wrote:
> On Tue, 2018-08-14 at 16:06 +0200, Marcel Ziswiler wrote:
>> From: Marcel Ziswiler <[email protected]>
>>
>> Dynamic debug is really helpful.
>
> It's useful, but perhaps not appropriate.
>
> Most arm configs are for small resource constrained systems.

Yes, but multi_v*_defconfig, by definition are not, so this really seems
acceptable to me here.

> dynamic_debug adds a lot of additional text and a fair amount of data.
>
>
>> Signed-off-by: Marcel Ziswiler <[email protected]>
>>
>> ---
>>
>> arch/arm/configs/multi_v5_defconfig | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/arch/arm/configs/multi_v5_defconfig b/arch/arm/configs/multi_v5_defconfig
>> index 318b76fa26d1..b302ac68d587 100644
>> --- a/arch/arm/configs/multi_v5_defconfig
>> +++ b/arch/arm/configs/multi_v5_defconfig
>> @@ -275,6 +275,7 @@ CONFIG_NLS_CODEPAGE_850=y
>> CONFIG_NLS_ISO8859_1=y
>> CONFIG_NLS_ISO8859_2=y
>> CONFIG_NLS_UTF8=y
>> +CONFIG_DYNAMIC_DEBUG=y
>> CONFIG_DEBUG_INFO=y
>> CONFIG_DEBUG_FS=y
>> CONFIG_MAGIC_SYSRQ=y
>
> _______________________________________________
> linux-arm-kernel mailing list
> [email protected]
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
>

--
Florian

2018-09-21 17:01:00

by Marcel Ziswiler

[permalink] [raw]
Subject: Re: [RESEND PATCH 1/6] ARM: config: multi_v5: enable dynamic debug

On Wed, 2018-08-15 at 18:29 -0700, Florian Fainelli wrote:
>
> On 08/14/2018 07:16 AM, Joe Perches wrote:
> > On Tue, 2018-08-14 at 16:06 +0200, Marcel Ziswiler wrote:
> > > From: Marcel Ziswiler <[email protected]>
> > >
> > > Dynamic debug is really helpful.
> >
> > It's useful, but perhaps not appropriate.
> >
> > Most arm configs are for small resource constrained systems.
>
> Yes, but multi_v*_defconfig, by definition are not, so this really
> seems
> acceptable to me here.

What is the conclusion now? Is anybody gona pull this patchset in? Or
are any changes needed?

> > dynamic_debug adds a lot of additional text and a fair amount of
> > data.
> >
> >
> > > Signed-off-by: Marcel Ziswiler <[email protected]>
> > >
> > > ---
> > >
> > > arch/arm/configs/multi_v5_defconfig | 1 +
> > > 1 file changed, 1 insertion(+)
> > >
> > > diff --git a/arch/arm/configs/multi_v5_defconfig
> > > b/arch/arm/configs/multi_v5_defconfig
> > > index 318b76fa26d1..b302ac68d587 100644
> > > --- a/arch/arm/configs/multi_v5_defconfig
> > > +++ b/arch/arm/configs/multi_v5_defconfig
> > > @@ -275,6 +275,7 @@ CONFIG_NLS_CODEPAGE_850=y
> > > CONFIG_NLS_ISO8859_1=y
> > > CONFIG_NLS_ISO8859_2=y
> > > CONFIG_NLS_UTF8=y
> > > +CONFIG_DYNAMIC_DEBUG=y
> > > CONFIG_DEBUG_INFO=y
> > > CONFIG_DEBUG_FS=y
> > > CONFIG_MAGIC_SYSRQ=y
> >
> > _______________________________________________
> > linux-arm-kernel mailing list
> > [email protected]
> > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel