[v3] imx6q-sabresd: Add a qt5 defconfig variant

Introduce imx6q-sabresd_qt5_defconfig that supports the opensource
Etnaviv graphical stack.

This defconfig provides a way to quickly test some graphical applications,
such as kmscube, qt5, glmark2.

Currently kernel mainline exhibits issues when running cpufreq as ondemand
governor on mx6, so add a linux fragment that disables such option for the
time being.

Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
Changes since v2:
- Drop BR2_PACKAGE_MESA3D_DRI_DRIVER_SWRAST (Thomas)
- Bump to the latest kernel 4.11.6

 board/freescale/imx6q-sabresd/linux.fragment       |  1 +
 board/freescale/imx6q-sabresd/readme.txt           | 23 +++++++++
 .../imx6q-sabresd/rootfs_overlay/root/sabresd.json | 15 ++++++
 configs/imx6q-sabresd_qt5_defconfig                | 56 ++++++++++++++++++++++
 4 files changed, 95 insertions(+)
 create mode 100644 board/freescale/imx6q-sabresd/linux.fragment
 create mode 100644 board/freescale/imx6q-sabresd/rootfs_overlay/root/sabresd.json
 create mode 100644 configs/imx6q-sabresd_qt5_defconfig

Hi Thomas,

On Wed, Jun 21, 2017 at 5:49 PM, Fabio Estevam <festevam@gmail.com> wrote:
> Introduce imx6q-sabresd_qt5_defconfig that supports the opensource
> Etnaviv graphical stack.
>
> This defconfig provides a way to quickly test some graphical applications,
> such as kmscube, qt5, glmark2.
>
> Currently kernel mainline exhibits issues when running cpufreq as ondemand
> governor on mx6, so add a linux fragment that disables such option for the
> time being.
>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> ---
> Changes since v2:
> - Drop BR2_PACKAGE_MESA3D_DRI_DRIVER_SWRAST (Thomas)
> - Bump to the latest kernel 4.11.6

Any feedback, please?

On Tue, Jul 11, 2017 at 10:44 AM, Fabio Estevam <festevam@gmail.com> wrote:
> Hi Thomas,
>
> On Wed, Jun 21, 2017 at 5:49 PM, Fabio Estevam <festevam@gmail.com> wrote:
>> Introduce imx6q-sabresd_qt5_defconfig that supports the opensource
>> Etnaviv graphical stack.
>>
>> This defconfig provides a way to quickly test some graphical applications,
>> such as kmscube, qt5, glmark2.
>>
>> Currently kernel mainline exhibits issues when running cpufreq as ondemand
>> governor on mx6, so add a linux fragment that disables such option for the
>> time being.
>>
>> Signed-off-by: Fabio Estevam <festevam@gmail.com>
>> ---
>> Changes since v2:
>> - Drop BR2_PACKAGE_MESA3D_DRI_DRIVER_SWRAST (Thomas)
>> - Bump to the latest kernel 4.11.6
>
> Any feedback, please?

Ping, I have been trying to submit this patch for months, but no
progress so far.

On 21-06-17 22:49, Fabio Estevam wrote:
> Introduce imx6q-sabresd_qt5_defconfig that supports the opensource
> Etnaviv graphical stack.
> 
> This defconfig provides a way to quickly test some graphical applications,
> such as kmscube, qt5, glmark2.
> 
> Currently kernel mainline exhibits issues when running cpufreq as ondemand
> governor on mx6, so add a linux fragment that disables such option for the
> time being.
> 
> Signed-off-by: Fabio Estevam <festevam@gmail.com>

 Hi Fabio, I applied to next, with a lot of changes:

     - Add Fabio to DEVELOPERS
     - Rename linux config fragment to linux_qt5.fragment
     - Add comment to linux_qt5.fragment why it is needed
     - Add explicit CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE=y to it
     - Reorder defconfig entries to they match imx6q-sabresd_defconfig
     - Remove strace and host-uboot-tools, neither seems to be needed
     - Drop BR2_PACKAGE_QT5BASE_LICENSE_APPROVED, it no longer exists
     - Replace BR2_TARGET_ROOTFS_EXT2_BLOCKS with BR2_TARGET_ROOTFS_EXT2_SIZE
     - Regenerate .gitlab-ci.yml

 Could you re-test based on the next branch?

 Also, you set BR2_TOOLCHAIN_BUILDROOT_GLIBC=y but none of the packages selected
seem to need it. Is there a reason why it is needed? Could you either remove it
(after testing with uClibc-ng) or add a comment why it is needed and remove
BR2_TOOLCHAIN_BUILDROOT_WCHAR=y?

 Regards,
 Arnout

Hi Arnout,

On Thu, Aug 3, 2017 at 4:06 AM, Arnout Vandecappelle <arnout@mind.be> wrote:

>  Hi Fabio, I applied to next, with a lot of changes:
>
>      - Add Fabio to DEVELOPERS
>      - Rename linux config fragment to linux_qt5.fragment
>      - Add comment to linux_qt5.fragment why it is needed
>      - Add explicit CONFIG_CPU_FREQ_DEFAULT_GOV_PERFORMANCE=y to it
>      - Reorder defconfig entries to they match imx6q-sabresd_defconfig
>      - Remove strace and host-uboot-tools, neither seems to be needed
>      - Drop BR2_PACKAGE_QT5BASE_LICENSE_APPROVED, it no longer exists
>      - Replace BR2_TARGET_ROOTFS_EXT2_BLOCKS with BR2_TARGET_ROOTFS_EXT2_SIZE
>      - Regenerate .gitlab-ci.yml
>
>  Could you re-test based on the next branch?

Thanks for reworking the patch and applying it.

I am currently out of the office without hardware access to test it.

Will test it as soon as I am back to the office (week of August 21)

>
>  Also, you set BR2_TOOLCHAIN_BUILDROOT_GLIBC=y but none of the packages selected
> seem to need it. Is there a reason why it is needed? Could you either remove it
> (after testing with uClibc-ng) or add a comment why it is needed and remove
> BR2_TOOLCHAIN_BUILDROOT_WCHAR=y?

Sure, will do it.

Thanks