mbox series

[v3,0/9] Regenerate arm64 defconfig for current kernel

Message ID 49a5d199-4eeb-a7c2-b539-13356162547f@free.fr (mailing list archive)
Headers show
Series Regenerate arm64 defconfig for current kernel | expand

Message

Marc Gonzalez Nov. 9, 2018, 12:54 a.m. UTC
Hello ARM maintainers,

v3: Keep CRYPTO_DEV_FSL_DPAA2_CAAM in patch 7/9

v2: Improve commit message for a few patches

The set of Kconfig options slowly changes with every kernel version.
This patch series regenerates the arm64 defconfig for v4.20
No functional change intended, except adding PINCTRL_MTK_MOORE to
keep PINCTRL_MT7622, and adding FSL_MC_BUS and FSL_MC_DPIO to
keep CRYPTO_DEV_FSL_DPAA2_CAAM.

Marc Gonzalez (9):
  arm64: defconfig: Regenerate for v4.20
  arm64: defconfig: Drop ARM_BIG_LITTLE_CPUFREQ
  arm64: defconfig: Drop NF_CONNTRACK_IPV{4,6}
  arm64: defconfig: Drop USB_ULPI_BUS and USB_CHIPIDEA_ULPI
  arm64: defconfig: Replace PINCTRL_MT7622 with PINCTRL_MTK_MOORE
  arm64: defconfig: Drop redundant SND_SOC_ROCKCHIP_I2S
  arm64: defconfig: Enable FSL_MC_BUS and FSL_MC_DPIO
  arm64: defconfig: Change SCSI_UFS_HISI=m
  arm64: defconfig: Drop redundant sound codec options

 arch/arm64/configs/defconfig | 108 ++++++++++++++++++++-----------------------
 1 file changed, 49 insertions(+), 59 deletions(-)

Comments

Marc Gonzalez Dec. 5, 2018, 3:33 p.m. UTC | #1
On 09/11/2018 01:54, Marc Gonzalez wrote:

> Hello ARM maintainers,
> 
> v3: Keep CRYPTO_DEV_FSL_DPAA2_CAAM in patch 7/9
> 
> v2: Improve commit message for a few patches
> 
> The set of Kconfig options slowly changes with every kernel version.
> This patch series regenerates the arm64 defconfig for v4.20
> No functional change intended, except adding PINCTRL_MTK_MOORE to
> keep PINCTRL_MT7622, and adding FSL_MC_BUS and FSL_MC_DPIO to
> keep CRYPTO_DEV_FSL_DPAA2_CAAM.
> 
> Marc Gonzalez (9):
>   arm64: defconfig: Regenerate for v4.20
>   arm64: defconfig: Drop ARM_BIG_LITTLE_CPUFREQ
>   arm64: defconfig: Drop NF_CONNTRACK_IPV{4,6}
>   arm64: defconfig: Drop USB_ULPI_BUS and USB_CHIPIDEA_ULPI
>   arm64: defconfig: Replace PINCTRL_MT7622 with PINCTRL_MTK_MOORE
>   arm64: defconfig: Drop redundant SND_SOC_ROCKCHIP_I2S
>   arm64: defconfig: Enable FSL_MC_BUS and FSL_MC_DPIO
>   arm64: defconfig: Change SCSI_UFS_HISI=m
>   arm64: defconfig: Drop redundant sound codec options
> 
>  arch/arm64/configs/defconfig | 108 ++++++++++++++++++++-----------------------
>  1 file changed, 49 insertions(+), 59 deletions(-)

Arnd, Olof,

Would you take this series for 4.21-rc1?

Regards.
Olof Johansson Dec. 11, 2018, 3:41 p.m. UTC | #2
On Wed, Dec 05, 2018 at 04:33:37PM +0100, Marc Gonzalez wrote:
> On 09/11/2018 01:54, Marc Gonzalez wrote:
> 
> > Hello ARM maintainers,
> > 
> > v3: Keep CRYPTO_DEV_FSL_DPAA2_CAAM in patch 7/9
> > 
> > v2: Improve commit message for a few patches
> > 
> > The set of Kconfig options slowly changes with every kernel version.
> > This patch series regenerates the arm64 defconfig for v4.20
> > No functional change intended, except adding PINCTRL_MTK_MOORE to
> > keep PINCTRL_MT7622, and adding FSL_MC_BUS and FSL_MC_DPIO to
> > keep CRYPTO_DEV_FSL_DPAA2_CAAM.
> > 
> > Marc Gonzalez (9):
> >   arm64: defconfig: Regenerate for v4.20
> >   arm64: defconfig: Drop ARM_BIG_LITTLE_CPUFREQ
> >   arm64: defconfig: Drop NF_CONNTRACK_IPV{4,6}
> >   arm64: defconfig: Drop USB_ULPI_BUS and USB_CHIPIDEA_ULPI
> >   arm64: defconfig: Replace PINCTRL_MT7622 with PINCTRL_MTK_MOORE
> >   arm64: defconfig: Drop redundant SND_SOC_ROCKCHIP_I2S
> >   arm64: defconfig: Enable FSL_MC_BUS and FSL_MC_DPIO
> >   arm64: defconfig: Change SCSI_UFS_HISI=m
> >   arm64: defconfig: Drop redundant sound codec options
> > 
> >  arch/arm64/configs/defconfig | 108 ++++++++++++++++++++-----------------------
> >  1 file changed, 49 insertions(+), 59 deletions(-)
> 
> Arnd, Olof,
> 
> Would you take this series for 4.21-rc1?

Hi,

Yeah, I'm happy to look at applying it but will do it towards the end of the
staging of new contents so we don't get too much conflicts with pull requests
that come in.


-Olof
Olof Johansson Dec. 13, 2018, 6:51 a.m. UTC | #3
On Tue, Dec 11, 2018 at 07:41:59AM -0800, Olof Johansson wrote:
> On Wed, Dec 05, 2018 at 04:33:37PM +0100, Marc Gonzalez wrote:
> > On 09/11/2018 01:54, Marc Gonzalez wrote:
> > 
> > > Hello ARM maintainers,
> > > 
> > > v3: Keep CRYPTO_DEV_FSL_DPAA2_CAAM in patch 7/9
> > > 
> > > v2: Improve commit message for a few patches
> > > 
> > > The set of Kconfig options slowly changes with every kernel version.
> > > This patch series regenerates the arm64 defconfig for v4.20
> > > No functional change intended, except adding PINCTRL_MTK_MOORE to
> > > keep PINCTRL_MT7622, and adding FSL_MC_BUS and FSL_MC_DPIO to
> > > keep CRYPTO_DEV_FSL_DPAA2_CAAM.
> > > 
> > > Marc Gonzalez (9):
> > >   arm64: defconfig: Regenerate for v4.20
> > >   arm64: defconfig: Drop ARM_BIG_LITTLE_CPUFREQ
> > >   arm64: defconfig: Drop NF_CONNTRACK_IPV{4,6}
> > >   arm64: defconfig: Drop USB_ULPI_BUS and USB_CHIPIDEA_ULPI
> > >   arm64: defconfig: Replace PINCTRL_MT7622 with PINCTRL_MTK_MOORE
> > >   arm64: defconfig: Drop redundant SND_SOC_ROCKCHIP_I2S
> > >   arm64: defconfig: Enable FSL_MC_BUS and FSL_MC_DPIO
> > >   arm64: defconfig: Change SCSI_UFS_HISI=m
> > >   arm64: defconfig: Drop redundant sound codec options
> > > 
> > >  arch/arm64/configs/defconfig | 108 ++++++++++++++++++++-----------------------
> > >  1 file changed, 49 insertions(+), 59 deletions(-)
> > 
> > Arnd, Olof,
> > 
> > Would you take this series for 4.21-rc1?
> 
> Hi,
> 
> Yeah, I'm happy to look at applying it but will do it towards the end of the
> staging of new contents so we don't get too much conflicts with pull requests
> that come in.

I've applied the patches from the series that were still needed. When
re-running savedefconfig as part of patch 1, several of them were no longer
applicable.


-Olof
Marc Gonzalez Dec. 14, 2018, 3:44 p.m. UTC | #4
On 13/12/2018 07:51, Olof Johansson wrote:

> I've applied the patches from the series that were still needed. When
> re-running savedefconfig as part of patch 1, several of them were no longer
> applicable.

FWIW, Arnd had suggested that the first patch should only deal with the
trivial case (moving options around), and subsequent patches should deal
with removed options, and corner cases.

IIUC, you folded "trivial + removed" into a single patch. I suppose it
does make the commit log less bloated ;-)

Thanks for merging the series.

Regards.
Arnd Bergmann Dec. 18, 2018, 3:27 p.m. UTC | #5
On Fri, Dec 14, 2018 at 4:44 PM Marc Gonzalez <marc.w.gonzalez@free.fr> wrote:
>
> On 13/12/2018 07:51, Olof Johansson wrote:
>
> > I've applied the patches from the series that were still needed. When
> > re-running savedefconfig as part of patch 1, several of them were no longer
> > applicable.
>
> FWIW, Arnd had suggested that the first patch should only deal with the
> trivial case (moving options around), and subsequent patches should deal
> with removed options, and corner cases.
>
> IIUC, you folded "trivial + removed" into a single patch. I suppose it
> does make the commit log less bloated ;-)
>
> Thanks for merging the series.

I suppose I should have asked for the patch to reorder the series to be
the /last/ one in the series instead, to keep the useful changelog texts.

     Arnd
Olof Johansson Dec. 18, 2018, 6:31 p.m. UTC | #6
On Tue, Dec 18, 2018 at 7:28 AM Arnd Bergmann <arnd@arndb.de> wrote:
>
> On Fri, Dec 14, 2018 at 4:44 PM Marc Gonzalez <marc.w.gonzalez@free.fr> wrote:
> >
> > On 13/12/2018 07:51, Olof Johansson wrote:
> >
> > > I've applied the patches from the series that were still needed. When
> > > re-running savedefconfig as part of patch 1, several of them were no longer
> > > applicable.
> >
> > FWIW, Arnd had suggested that the first patch should only deal with the
> > trivial case (moving options around), and subsequent patches should deal
> > with removed options, and corner cases.
> >
> > IIUC, you folded "trivial + removed" into a single patch. I suppose it
> > does make the commit log less bloated ;-)
> >
> > Thanks for merging the series.
>
> I suppose I should have asked for the patch to reorder the series to be
> the /last/ one in the series instead, to keep the useful changelog texts.

In my case I was a bit confused about what the series did when it was
first posted; I thought it actually enabled and disabled new
functionality, not that it "just" reflected changes that happened
implicit due to Kconfig changes. I.e. the patches that "fell out" of
the series were not actually making a difference to the generated
config, since they were setting what was now the default (or what
dependencies were no longer fulfilled).

So, I think there was a bit of miscommunication all around. Either way
-- thanks Marc for doing this work and following up on it, and
apologies for the confusion.


-Olof