[meta-freescale] [PATCH 1/2] linux-fslc: Update SRCREV

Discussion:

Andreas Müller

2018-09-05 11:44:05 UTC

The only change introduced is menuconfig fix.

Signed-off-by: Andreas Müller <***@gmail.com>
---
recipes-kernel/linux/linux-fslc_4.17.bb | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/recipes-kernel/linux/linux-fslc_4.17.bb b/recipes-kernel/linux/linux-fslc_4.17.bb
index 61c0c385..8426357c 100644
--- a/recipes-kernel/linux/linux-fslc_4.17.bb
+++ b/recipes-kernel/linux/linux-fslc_4.17.bb
@@ -14,6 +14,6 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=bbea815ee2795b2f4230826c0c6b8814"
PV = "4.17+git${SRCPV}"

SRCBRANCH = "4.17.x+fslc"
-SRCREV = "076d88a62d8a3ef0f393ea52fa89bb4ab1d720c5"
+SRCREV = "65375933f02aac8aa8e0e2da7827e2ae9dec0ff3"

COMPATIBLE_MACHINE = "(mxs|mx5|mx6|vf|use-mainline-bsp)"

--
2.14.4

--

Andreas Müller

2018-09-05 11:44:06 UTC

Permalink

Fixes:

| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[220]: Failed to apply ACL on /dev/v4l-subdev0: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[224]: Failed to apply ACL on /dev/v4l-subdev1: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[215]: Failed to apply ACL on /dev/v4l-subdev10: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[228]: Failed to apply ACL on /dev/v4l-subdev2: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[232]: Failed to apply ACL on /dev/v4l-subdev5: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[217]: Failed to apply ACL on /dev/v4l-subdev11: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[214]: Failed to apply ACL on /dev/dri/card1: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[216]: Failed to apply ACL on /dev/v4l-subdev8: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[226]: Failed to apply ACL on /dev/v4l-subdev9: Operation not supported

and nasty follow-ups: Starting weston from sddm as unpriviledged user fails
with some hints on missing access rights.

For a better understanding CONFIG_TMPFS=y was added which was set indirectly
by CONFIG_DEVTMPFS=y

Signed-off-by: Andreas Müller <***@gmail.com>
---
recipes-kernel/linux/linux-fslc/defconfig | 2 ++
1 file changed, 2 insertions(+)

diff --git a/recipes-kernel/linux/linux-fslc/defconfig b/recipes-kernel/linux/linux-fslc/defconfig
index 3b68a8af..84d59fc4 100644
--- a/recipes-kernel/linux/linux-fslc/defconfig
+++ b/recipes-kernel/linux/linux-fslc/defconfig
@@ -394,6 +394,8 @@ CONFIG_ZISOFS=y
CONFIG_UDF_FS=m
CONFIG_MSDOS_FS=m
CONFIG_VFAT_FS=y
+CONFIG_TMPFS=y
+CONFIG_TMPFS_POSIX_ACL=y
CONFIG_JFFS2_FS=y
CONFIG_UBIFS_FS=y
CONFIG_NFS_FS=y

--
2.14.4

--

Otavio Salvador

2018-09-05 23:24:08 UTC

Permalink

Post by Andreas MÃ¼ller
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[220]: Failed to apply ACL on /dev/v4l-subdev0: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[224]: Failed to apply ACL on /dev/v4l-subdev1: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[215]: Failed to apply ACL on /dev/v4l-subdev10: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[228]: Failed to apply ACL on /dev/v4l-subdev2: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[232]: Failed to apply ACL on /dev/v4l-subdev5: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[217]: Failed to apply ACL on /dev/v4l-subdev11: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[214]: Failed to apply ACL on /dev/dri/card1: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[216]: Failed to apply ACL on /dev/v4l-subdev8: Operation not supported
| Sep 04 09:05:10 imx6qdl-variscite-som systemd-udevd[226]: Failed to apply ACL on /dev/v4l-subdev9: Operation not supported
and nasty follow-ups: Starting weston from sddm as unpriviledged user fails
with some hints on missing access rights.
For a better understanding CONFIG_TMPFS=y was added which was set indirectly
by CONFIG_DEVTMPFS=y

I applied this to master. Could you send this as a change to our
imx_v6_v7_defconfig ? it would be good to send it upstream as well so
we avoid keeping this change for next releases.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854 Mobile: +1 (347) 903-9750
--

Andreas Müller

2018-09-06 09:39:36 UTC

Permalink

On Thu, Sep 6, 2018 at 1:24 AM, Otavio Salvador

Post by Otavio Salvador

I applied this to master. Could you send this as a change to our
imx_v6_v7_defconfig ? it would be good to send it upstream as well so
we avoid keeping this change for next releases.

Done

Just checked: Copying imx_v6_v7_defconfig to our defconfig fives the
following diff:

diff --git a/recipes-kernel/linux/linux-fslc/defconfig
b/recipes-kernel/linux/linux-fslc/defconfig
index 84d59fc4..8bee1f96 100644
--- a/recipes-kernel/linux/linux-fslc/defconfig
+++ b/recipes-kernel/linux/linux-fslc/defconfig
@@ -158,6 +158,7 @@ CONFIG_WL18XX=m
CONFIG_WLCORE_SDIO=m
# CONFIG_WILINK_PLATFORM_DATA is not set
CONFIG_INPUT_EVDEV=y
+CONFIG_INPUT_EVBUG=m
CONFIG_KEYBOARD_GPIO=y
CONFIG_KEYBOARD_SNVS_PWRKEY=y
CONFIG_KEYBOARD_IMX=y
@@ -293,6 +294,7 @@ CONFIG_USB_STORAGE=y
CONFIG_USB_CHIPIDEA=y
CONFIG_USB_CHIPIDEA_UDC=y
CONFIG_USB_CHIPIDEA_HOST=y
+CONFIG_USB_CHIPIDEA_ULPI=y
CONFIG_USB_SERIAL=m
CONFIG_USB_SERIAL_GENERIC=y
CONFIG_USB_SERIAL_FTDI_SIO=m
@@ -329,6 +331,7 @@ CONFIG_USB_GADGETFS=m
CONFIG_USB_FUNCTIONFS=m
CONFIG_USB_MASS_STORAGE=m
CONFIG_USB_G_SERIAL=m
+CONFIG_USB_ULPI_BUS=y
CONFIG_MMC=y
CONFIG_MMC_SDHCI=y
CONFIG_MMC_SDHCI_PLTFM=y
@@ -414,7 +417,7 @@ CONFIG_MAGIC_SYSRQ=y
# CONFIG_SCHED_DEBUG is not set
CONFIG_PROVE_LOCKING=y
# CONFIG_DEBUG_BUGVERBOSE is not set
-CONFIG_FUNCTION_TRACER=y
+# CONFIG_FTRACE is not set
# CONFIG_ARM_UNWIND is not set
CONFIG_SECURITYFS=y
CONFIG_CRYPTO_DEV_FSL_CAAM=y

Shouldn't we swap to use imx_v6_v7_defconfig shipped?

Andreas
--

Andreas Müller

2018-09-06 12:10:07 UTC

Permalink

Post by Andreas MÃ¼ller
On Thu, Sep 6, 2018 at 1:24 AM, Otavio Salvador

Post by Otavio Salvador

I applied this to master. Could you send this as a change to our
imx_v6_v7_defconfig ? it would be good to send it upstream as well so
we avoid keeping this change for next releases.

Done
Just checked: Copying imx_v6_v7_defconfig to our defconfig fives the
diff --git a/recipes-kernel/linux/linux-fslc/defconfig
b/recipes-kernel/linux/linux-fslc/defconfig
index 84d59fc4..8bee1f96 100644
--- a/recipes-kernel/linux/linux-fslc/defconfig
+++ b/recipes-kernel/linux/linux-fslc/defconfig
@@ -158,6 +158,7 @@ CONFIG_WL18XX=m
CONFIG_WLCORE_SDIO=m
# CONFIG_WILINK_PLATFORM_DATA is not set
CONFIG_INPUT_EVDEV=y
+CONFIG_INPUT_EVBUG=m

^ That is not the best idea for productive environment - it floods journal

Post by Andreas MÃ¼ller
CONFIG_KEYBOARD_GPIO=y
CONFIG_KEYBOARD_SNVS_PWRKEY=y
CONFIG_KEYBOARD_IMX=y
@@ -293,6 +294,7 @@ CONFIG_USB_STORAGE=y
CONFIG_USB_CHIPIDEA=y
CONFIG_USB_CHIPIDEA_UDC=y
CONFIG_USB_CHIPIDEA_HOST=y
+CONFIG_USB_CHIPIDEA_ULPI=y
CONFIG_USB_SERIAL=m
CONFIG_USB_SERIAL_GENERIC=y
CONFIG_USB_SERIAL_FTDI_SIO=m
@@ -329,6 +331,7 @@ CONFIG_USB_GADGETFS=m
CONFIG_USB_FUNCTIONFS=m
CONFIG_USB_MASS_STORAGE=m
CONFIG_USB_G_SERIAL=m
+CONFIG_USB_ULPI_BUS=y
CONFIG_MMC=y
CONFIG_MMC_SDHCI=y
CONFIG_MMC_SDHCI_PLTFM=y
@@ -414,7 +417,7 @@ CONFIG_MAGIC_SYSRQ=y
# CONFIG_SCHED_DEBUG is not set
CONFIG_PROVE_LOCKING=y
# CONFIG_DEBUG_BUGVERBOSE is not set
-CONFIG_FUNCTION_TRACER=y

^ That should be kept

Post by Andreas MÃ¼ller
+# CONFIG_FTRACE is not set
# CONFIG_ARM_UNWIND is not set
CONFIG_SECURITYFS=y
CONFIG_CRYPTO_DEV_FSL_CAAM=y
Shouldn't we swap to use imx_v6_v7_defconfig shipped?
Andreas

Suggestion: Since I have further issues with this kernel (seems not a
single firmware is loaded) - so I continue debug. As soon as I think
there is a better config I will

1. Send it to linux-fslc / imx_v6_v7_defconfig
2. Send a patch here to use imx_v6_v7_defconfig

OK?

Andreas
--

Otavio Salvador

2018-09-06 12:28:42 UTC

Permalink

On Thu, Sep 6, 2018 at 9:10 AM Andreas Müller <***@gmail.com> wrote:
...

Post by Andreas MÃ¼ller
Suggestion: Since I have further issues with this kernel (seems not a
single firmware is loaded) - so I continue debug. As soon as I think
there is a better config I will
1. Send it to linux-fslc / imx_v6_v7_defconfig
2. Send a patch here to use imx_v6_v7_defconfig
OK?

Please do. But let's use the defconfig in the layer as it makes easier
for users to customize it.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854 Mobile: +1 (347) 903-9750
--

Otavio Salvador

2018-09-06 12:27:22 UTC

Permalink

Post by Andreas MÃ¼ller
On Thu, Sep 6, 2018 at 1:24 AM, Otavio Salvador

Post by Otavio Salvador

I applied this to master. Could you send this as a change to our
imx_v6_v7_defconfig ? it would be good to send it upstream as well so
we avoid keeping this change for next releases.

This is not nice to have loaded automatically; it is very verbose.

Fine. Add it to our defconfig.

Post by Andreas MÃ¼ller
CONFIG_USB_SERIAL=m
CONFIG_USB_SERIAL_GENERIC=y
CONFIG_USB_SERIAL_FTDI_SIO=m
@@ -329,6 +331,7 @@ CONFIG_USB_GADGETFS=m
CONFIG_USB_FUNCTIONFS=m
CONFIG_USB_MASS_STORAGE=m
CONFIG_USB_G_SERIAL=m
+CONFIG_USB_ULPI_BUS=y

Fine. Add it to our defconfig.

Post by Andreas MÃ¼ller
CONFIG_MMC=y
CONFIG_MMC_SDHCI=y
CONFIG_MMC_SDHCI_PLTFM=y
@@ -414,7 +417,7 @@ CONFIG_MAGIC_SYSRQ=y
# CONFIG_SCHED_DEBUG is not set
CONFIG_PROVE_LOCKING=y
# CONFIG_DEBUG_BUGVERBOSE is not set
-CONFIG_FUNCTION_TRACER=y
+# CONFIG_FTRACE is not set

We need this to lttng to work.

Post by Andreas MÃ¼ller
# CONFIG_ARM_UNWIND is not set
CONFIG_SECURITYFS=y
CONFIG_CRYPTO_DEV_FSL_CAAM=y
Shouldn't we swap to use imx_v6_v7_defconfig shipped?

We can make more commits to it to simplify the sync.

--
Otavio Salvador O.S. Systems
http://www.ossystems.com.br http://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854 Mobile: +1 (347) 903-9750
--

Fabio Estevam

2018-09-06 14:06:10 UTC

Permalink

Hi Otavio,
.
On Thu, Sep 6, 2018 at 9:27 AM, Otavio Salvador

Post by Otavio Salvador

Fine. Add it to our defconfig.

Please don't add these two options.

These options ended up in 4.17 linux-stable tree by mistake.

i sent a revert for this change in the linux-stable tree as it causes
a reboot regression on mx6 wandboard.
--