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 :) Cheers, Nathan
