Re: [Buildroot] [PATCH 0/2] Fix Rock 5B build, broken by gcc 12 warnings

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

Hello Kilian,

Thanks for your work. See below some comments/questions.

On Tue,  8 Aug 2023 07:01:20 +0200
Kilian Zinnecker via buildroot <buildroot@buildroot.org> wrote:

> Building the image for the Radxa Rock 5B currently breaks: The gcc
> version was recently changed to version 12 and gcc 12 seems to stop
> compiling, if certain warnings occur. See the broken build here:
> 
> https://gitlab.com/buildroot.org/buildroot/-/jobs/4795673759
> 
> This patch series adds two patches from the mainline Linux kernel,
> which fix two of the warnings. However, also custom code parts of the
> used custom kernel break due to warnings from gcc 12. The best
> solution would be to fix those issues as well. However, until they are
> fixed, the version of gcc used for the Rock 5B is set to the previous
> version, gcc 11.
> 
> Kilian Zinnecker (2):
>   configs/rock5b: Freeze gcc to version 11
>   board/radxa/rock5b: Add patches for gcc12 warnings

I'm not sure to understand the reasoning for the second patch. Does it
fix all gcc 12.x issues or not? If not, what is the motivation for this
second patch?

Also: gcc 11.x is currently the oldest version supported by Buildroot,
it means that as soon as gcc 14.x is released, we will drop support for
gcc 11.x. So clearly, the solution of sticking to gcc 11.x is not going
to work for a very long time.

Best regards,

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot