[PATCH 1/2] dracut: Add drivers/clk by default on arm

[PATCH 1/2] dracut: Add drivers/clk by default on arm

[Nicolas Chauvet]

This will bundle clock drivers into the initramfs on arm

Tested on ti dm8148-t410 where adpll is needed on 4.6+ kernel
This will avoid to rely on (maybe broken) bootloader clocks.

Theses modules are also usually loaded early. Having them bundled into
the initramfs will avoid lot of deferred probes and others delay.

Signed-off-by: Nicolas Chauvet <kwizart-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
 modules.d/90kernel-modules/module-setup.sh | 1 +
 1 file changed, 1 insertion(+)

diff --git a/modules.d/90kernel-modules/module-setup.sh b/modules.d/90kernel-modules/module-setup.sh
index c30715d..7904c02 100755
--- a/modules.d/90kernel-modules/module-setup.sh
+++ b/modules.d/90kernel-modules/module-setup.sh
@@ -26,6 +26,7 @@ installkernel() {
         if [[ "$(uname -p)" == arm* ]]; then
             # arm specific modules
             instmods \
+                "=drivers/clk" \
                 "=drivers/i2c/busses" \
                 "=drivers/regulator" \
                 "=drivers/rtc" \
-- 
2.4.11

[PATCH 2/2] dracut: Add drivers/tty/serial on arm

[Nicolas Chauvet]

This is needed when serial drivers are compiled as module on arm
Current candidates are 8250_omap or msn_serial

Signed-off-by: Nicolas Chauvet <kwizart-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
 modules.d/90kernel-modules/module-setup.sh | 1 +
 1 file changed, 1 insertion(+)

diff --git a/modules.d/90kernel-modules/module-setup.sh b/modules.d/90kernel-modules/module-setup.sh
index 7904c02..215f468 100755
--- a/modules.d/90kernel-modules/module-setup.sh
+++ b/modules.d/90kernel-modules/module-setup.sh
@@ -30,6 +30,7 @@ installkernel() {
                 "=drivers/i2c/busses" \
                 "=drivers/regulator" \
                 "=drivers/rtc" \
+                "=drivers/tty/serial" \
                 "=drivers/usb/host" \
                 "=drivers/usb/phy" \
                 ${NULL}
-- 
2.4.11

Re: [PATCH 2/2] dracut: Add drivers/tty/serial on arm

[Peter Robinson]

On Thu, May 12, 2016 at 4:28 PM, Nicolas Chauvet <kwizart-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
> This is needed when serial drivers are compiled as module on arm
> Current candidates are 8250_omap or msn_serial

Neither of these are used for serial consoles so why do they need to
be in the initrd?

Peter

> Signed-off-by: Nicolas Chauvet <kwizart-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
>  modules.d/90kernel-modules/module-setup.sh | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/modules.d/90kernel-modules/module-setup.sh b/modules.d/90kernel-modules/module-setup.sh
> index 7904c02..215f468 100755
> --- a/modules.d/90kernel-modules/module-setup.sh
> +++ b/modules.d/90kernel-modules/module-setup.sh
> @@ -30,6 +30,7 @@ installkernel() {
>                  "=drivers/i2c/busses" \
>                  "=drivers/regulator" \
>                  "=drivers/rtc" \
> +                "=drivers/tty/serial" \
>                  "=drivers/usb/host" \
>                  "=drivers/usb/phy" \
>                  ${NULL}
> --
> 2.4.11
>
> --
> To unsubscribe from this list: send the line "unsubscribe initramfs" in
> the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 2/2] dracut: Add drivers/tty/serial on arm

[Nicolas Chauvet]

2016-05-12 17:28 GMT+02:00 Nicolas Chauvet <kwizart-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>:
> This is needed when serial drivers are compiled as module on arm
> Current candidates are 8250_omap or msn_serial
>
> Signed-off-by: Nicolas Chauvet <kwizart-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
>  modules.d/90kernel-modules/module-setup.sh | 1 +
>  1 file changed, 1 insertion(+)
>
> diff --git a/modules.d/90kernel-modules/module-setup.sh b/modules.d/90kernel-modules/module-setup.sh
> index 7904c02..215f468 100755
> --- a/modules.d/90kernel-modules/module-setup.sh
> +++ b/modules.d/90kernel-modules/module-setup.sh
> @@ -30,6 +30,7 @@ installkernel() {
>                  "=drivers/i2c/busses" \
>                  "=drivers/regulator" \
>                  "=drivers/rtc" \
> +                "=drivers/tty/serial" \
>                  "=drivers/usb/host" \
>                  "=drivers/usb/phy" \
>                  ${NULL}
> --
@peter, thx for review, and sorry for the late answear.


I've got the following confirmation by the upstream msn_serial maintainer:

<sboyd> nchauvet: if the tty driver is built as a module the console
part can't be used
<sboyd> you can run a getty though
<sboyd> I've done that to test the module loading of that driver and
it works fine

It means that bundling the tty_serial as a module into the initramfs
would requires to have a getty.
Given that there is currently no need to have a getty into the
initrafms I expect this can wait for the rootfs to be available.
So I discard this patch.

FYI the reason why I've started to experiment building the msm_serial
as a module is because there is a a spurious message related to msm
init() even on unrelated hardware (so before probe())
[    3.135866] msm_serial: driver initialized


Is there any issue with the other patch ? (it's unrelated to this one).

-- 
-

Nicolas (kwizart)

Re: [PATCH 1/2] dracut: Add drivers/clk by default on arm

[Harald Hoyer]

Am 12.05.2016 um 17:28 schrieb Nicolas Chauvet:
> This will bundle clock drivers into the initramfs on arm
> 
> Tested on ti dm8148-t410 where adpll is needed on 4.6+ kernel
> This will avoid to rely on (maybe broken) bootloader clocks.
> 
> Theses modules are also usually loaded early. Having them bundled into
> the initramfs will avoid lot of deferred probes and others delay.
> 
> Signed-off-by: Nicolas Chauvet <kwizart-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
> ---
>  modules.d/90kernel-modules/module-setup.sh | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/modules.d/90kernel-modules/module-setup.sh b/modules.d/90kernel-modules/module-setup.sh
> index c30715d..7904c02 100755
> --- a/modules.d/90kernel-modules/module-setup.sh
> +++ b/modules.d/90kernel-modules/module-setup.sh
> @@ -26,6 +26,7 @@ installkernel() {
>          if [[ "$(uname -p)" == arm* ]]; then
>              # arm specific modules
>              instmods \
> +                "=drivers/clk" \
>                  "=drivers/i2c/busses" \
>                  "=drivers/regulator" \
>                  "=drivers/rtc" \
> 

Thanks! Pushed.