mbox series

[GIT,PULL,1/2] Renesas ARM defconfig updates for v5.9

Message ID 20200703120642.5128-2-geert+renesas@glider.be (mailing list archive)
State Mainlined
Commit b104533aaffc0efb9ea16c45d6020654841ce4a2
Headers show
Series Renesas ARM SoC updates for v5.9 | expand

Pull-request

git://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-devel.git tags/renesas-arm-defconfig-for-v5.9-tag1

Message

Geert Uytterhoeven July 3, 2020, 12:06 p.m. UTC
The following changes since commit b3a9e3b9622ae10064826dccb4f7a52bd88c7407:

  Linux 5.8-rc1 (2020-06-14 12:45:04 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-devel.git tags/renesas-arm-defconfig-for-v5.9-tag1

for you to fetch changes up to 1d21a0a32f93e24711b13af493ac5fe728a7be1f:

  arm64: defconfig: Enable additional support for Renesas platforms (2020-07-03 12:02:05 +0200)

----------------------------------------------------------------
Renesas ARM defconfig updates for v5.9

  - Refresh shmobile_defconfig for v5.8-rc1
  - Enable additional support for Renesas platforms to
    shmobile_defconfig, multi_v7_defconfig, and the arm64 defconfig.

----------------------------------------------------------------
Geert Uytterhoeven (9):
      ARM: shmobile: defconfig: Refresh for v5.8-rc1
      ARM: shmobile: defconfig: Enable DA9063 ONKEY support
      ARM: shmobile: defconfig: Enable CAT9554 support
      ARM: shmobile: defconfig: Enable RZ/A watchdog support
      ARM: shmobile: defconfig: Enable RZ/A1 CEU support
      ARM: shmobile: defconfig: Enable ADV7612 CEC support
      ARM: shmobile: defconfig: Enable RZ/A1H RTC support
      ARM: multi_v7_defconfig: Enable additional support for Renesas platforms
      arm64: defconfig: Enable additional support for Renesas platforms

 arch/arm/configs/multi_v7_defconfig |  6 ++++++
 arch/arm/configs/shmobile_defconfig | 13 ++++++++-----
 arch/arm64/configs/defconfig        |  4 ++++
 3 files changed, 18 insertions(+), 5 deletions(-)

Comments

Arnd Bergmann July 6, 2020, 2:36 p.m. UTC | #1
On Fri, Jul 3, 2020 at 2:07 PM Geert Uytterhoeven
<geert+renesas@glider.be> wrote:
> ----------------------------------------------------------------
> Renesas ARM defconfig updates for v5.9
>
>   - Refresh shmobile_defconfig for v5.8-rc1
>   - Enable additional support for Renesas platforms to
>     shmobile_defconfig, multi_v7_defconfig, and the arm64 defconfig.
>

As you write on one of the commits

   "All of the above are modular, except for Ethernet support (HDMI Audio
    support is an optional feature of the modular ADV7511 driver)."

I'm starting to see some demand for making more drivers loadable modules
that are traditionally built-in. Can you give a more specific reason why
this one (and I guess CONFIG_RAVB next to it) should not a loadable module?

        Arnd
Geert Uytterhoeven July 6, 2020, 2:45 p.m. UTC | #2
Hi Arnd,

On Mon, Jul 6, 2020 at 4:37 PM Arnd Bergmann <arnd@arndb.de> wrote:
> On Fri, Jul 3, 2020 at 2:07 PM Geert Uytterhoeven
> <geert+renesas@glider.be> wrote:
> > ----------------------------------------------------------------
> > Renesas ARM defconfig updates for v5.9
> >
> >   - Refresh shmobile_defconfig for v5.8-rc1
> >   - Enable additional support for Renesas platforms to
> >     shmobile_defconfig, multi_v7_defconfig, and the arm64 defconfig.
>
> As you write on one of the commits
>
>    "All of the above are modular, except for Ethernet support (HDMI Audio
>     support is an optional feature of the modular ADV7511 driver)."
>
> I'm starting to see some demand for making more drivers loadable modules
> that are traditionally built-in. Can you give a more specific reason why
> this one (and I guess CONFIG_RAVB next to it) should not a loadable module?

We tend to make Ethernet built-in, to ease testing using nfsroot.
If that is no longer desirable, I guess we'll have to improve our initramfs
skills ;-)

Gr{oetje,eeting}s,

                        Geert
Arnd Bergmann July 6, 2020, 3:40 p.m. UTC | #3
On Mon, Jul 6, 2020 at 4:45 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> On Mon, Jul 6, 2020 at 4:37 PM Arnd Bergmann <arnd@arndb.de> wrote:
> > On Fri, Jul 3, 2020 at 2:07 PM Geert Uytterhoeven
> > <geert+renesas@glider.be> wrote:
> > > ----------------------------------------------------------------
> > > Renesas ARM defconfig updates for v5.9
> > >
> > >   - Refresh shmobile_defconfig for v5.8-rc1
> > >   - Enable additional support for Renesas platforms to
> > >     shmobile_defconfig, multi_v7_defconfig, and the arm64 defconfig.
> >
> > As you write on one of the commits
> >
> >    "All of the above are modular, except for Ethernet support (HDMI Audio
> >     support is an optional feature of the modular ADV7511 driver)."
> >
> > I'm starting to see some demand for making more drivers loadable modules
> > that are traditionally built-in. Can you give a more specific reason why
> > this one (and I guess CONFIG_RAVB next to it) should not a loadable module?
>
> We tend to make Ethernet built-in, to ease testing using nfsroot.
> If that is no longer desirable, I guess we'll have to improve our initramfs
> skills ;-)

I'm not sure either. My feeling is that we should probably make all network
support loadable modules eventually, it seems wrong to single out yours
if your current workflow depends on the built-in driver today.

I'm pulling your branch for now, but I'll keep asking around what others
think, and we might change all drivers that can to be loadable
modules later on.

In particular, a number of chips that primarily run Android are now
changed so they can have even more fundamental drivers (pinctrl,
clk, ...) as loadable modules, and it seems like a good idea to
also support that in the defconfig.

       Arnd
Geert Uytterhoeven July 6, 2020, 3:48 p.m. UTC | #4
Hi Arnd,

On Mon, Jul 6, 2020 at 5:41 PM Arnd Bergmann <arnd@arndb.de> wrote:
> On Mon, Jul 6, 2020 at 4:45 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> > On Mon, Jul 6, 2020 at 4:37 PM Arnd Bergmann <arnd@arndb.de> wrote:
> > > On Fri, Jul 3, 2020 at 2:07 PM Geert Uytterhoeven
> > > <geert+renesas@glider.be> wrote:
> > > > ----------------------------------------------------------------
> > > > Renesas ARM defconfig updates for v5.9
> > > >
> > > >   - Refresh shmobile_defconfig for v5.8-rc1
> > > >   - Enable additional support for Renesas platforms to
> > > >     shmobile_defconfig, multi_v7_defconfig, and the arm64 defconfig.
> > >
> > > As you write on one of the commits
> > >
> > >    "All of the above are modular, except for Ethernet support (HDMI Audio
> > >     support is an optional feature of the modular ADV7511 driver)."
> > >
> > > I'm starting to see some demand for making more drivers loadable modules
> > > that are traditionally built-in. Can you give a more specific reason why
> > > this one (and I guess CONFIG_RAVB next to it) should not a loadable module?
> >
> > We tend to make Ethernet built-in, to ease testing using nfsroot.
> > If that is no longer desirable, I guess we'll have to improve our initramfs
> > skills ;-)
>
> I'm not sure either. My feeling is that we should probably make all network
> support loadable modules eventually, it seems wrong to single out yours
> if your current workflow depends on the built-in driver today.

Most non-USB Ethernet drivers seem to be built-in?

> I'm pulling your branch for now, but I'll keep asking around what others
> think, and we might change all drivers that can to be loadable
> modules later on.

Thanks!

> In particular, a number of chips that primarily run Android are now
> changed so they can have even more fundamental drivers (pinctrl,
> clk, ...) as loadable modules, and it seems like a good idea to
> also support that in the defconfig.

Aren't the Android devices usually using WiFi, and no nfsroot?

Gr{oetje,eeting}s,

                        Geert