Re: [meta-freescale] [PATCH 2/2] linux-fslc/defconfig: Add CONFIG_TMPFS_POSIX_ACL=y

2018-09-06 Thread Fabio Estevam
Hi Otavio,
.
On Thu, Sep 6, 2018 at 9:27 AM, Otavio Salvador
 wrote:

>>  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
>
> Fine. Add it to our defconfig.
>
>>  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.

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.
-- 
___
meta-freescale mailing list
meta-freescale@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-freescale


Re: [meta-freescale] [PATCH 2/2] linux-fslc/defconfig: Add CONFIG_TMPFS_POSIX_ACL=y

2018-09-06 Thread Otavio Salvador
On Thu, Sep 6, 2018 at 9:10 AM Andreas Müller  wrote:
...
> 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.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750
-- 
___
meta-freescale mailing list
meta-freescale@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-freescale


Re: [meta-freescale] [PATCH 2/2] linux-fslc/defconfig: Add CONFIG_TMPFS_POSIX_ACL=y

2018-09-06 Thread Otavio Salvador
On Thu, Sep 6, 2018 at 6:40 AM Andreas Müller  wrote:
>
> On Thu, Sep 6, 2018 at 1:24 AM, Otavio Salvador
>  wrote:
> > On Wed, Sep 5, 2018 at 8:45 AM Andreas Müller  
> > wrote:
> >>
> >> 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 
> >
> > 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

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

>  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

Fine. Add it to our defconfig.

>  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.

>  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.

>  # 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.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750
-- 
___
meta-freescale mailing list
meta-freescale@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-freescale


Re: [meta-freescale] [PATCH 2/2] linux-fslc/defconfig: Add CONFIG_TMPFS_POSIX_ACL=y

2018-09-06 Thread Andreas Müller
On Thu, Sep 6, 2018 at 11:39 AM, Andreas Müller  wrote:
> On Thu, Sep 6, 2018 at 1:24 AM, Otavio Salvador
>  wrote:
>> On Wed, Sep 5, 2018 at 8:45 AM Andreas Müller  
>> wrote:
>>>
>>> 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 
>>
>> 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
^ That is not the best idea for productive environment - it floods journal
>  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
> +# 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
-- 
___
meta-freescale mailing list
meta-freescale@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-freescale


Re: [meta-freescale] [PATCH 2/2] linux-fslc/defconfig: Add CONFIG_TMPFS_POSIX_ACL=y

2018-09-06 Thread Andreas Müller
On Thu, Sep 6, 2018 at 1:24 AM, Otavio Salvador
 wrote:
> On Wed, Sep 5, 2018 at 8:45 AM Andreas Müller  wrote:
>>
>> 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 
>
> 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
-- 
___
meta-freescale mailing list
meta-freescale@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-freescale


Re: [meta-freescale] [PATCH 2/2] linux-fslc/defconfig: Add CONFIG_TMPFS_POSIX_ACL=y

2018-09-05 Thread Otavio Salvador
On Wed, Sep 5, 2018 at 8:45 AM Andreas Müller  wrote:
>
> 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 

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.brhttp://code.ossystems.com.br
Mobile: +55 (53) 9 9981-7854  Mobile: +1 (347) 903-9750
-- 
___
meta-freescale mailing list
meta-freescale@yoctoproject.org
https://lists.yoctoproject.org/listinfo/meta-freescale