On 10/27/22 11:22, Nathan Chancellor wrote:
Hi Rodrigo,
On Thu, Oct 27, 2022 at 10:29:33AM -0400, Rodrigo Siqueira wrote:
Nathan/Stephen,
Maybe I'm wrong, but I think you have access to some sort of CI that tests
multiple builds with different compiles and configs, right? Is it possible
to check this patch + amd-staging-drm-next in the CI to help us to
anticipate any compilation issue if we merge this change?
Yup, I have a build framework that I have developed for my
ClangBuiltLinux work that I was planning on putting this through to see
if there are any new warnings that show up in this code since it is
going to be built on a new architecture. I will report back on Ao's v2
(or v3 if it is available since Arnd had some comments on v2) when that
is done.
Or should we merge it and wait until it gets merged on the mainline? In case
of a problem, we can easily revert a small patch like this, right?
Sure, although if we can catch issues beforehand, that would be nice :)
Hi Nathan,
Thanks a lot for checking this. Let's fix any issue that your CI reports
before merging this patch.
Thanks
Siqueira
Cheers,
Nathan
