| Message ID | 20200703120642.5128-2-geert+renesas@glider.be |
|---|---|
| State | New |
| Headers | show |
| Series | Renesas ARM SoC updates for v5.9 | expand |
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
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
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
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
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(-)