Like this?

# 'iris' and 'radeonsi' require LLVM, 'asahi' needs LLVM and additionally requires opencl (rust)

Personally I would vote for removal, because that seems self-explanatory enough to me. There is
no corresponding comment for vulkandrivers and the comment would also be outdated quite soon,
because mesa main branch has added a (by default) rusticl dependency to radeonsi as well    

On Fri, May 23 2025 at 11:39:30 +02:00:00, Koen Kooi <koen.kooi@oss.qualcomm.com> wrote:
Op 23 mei 2025, om 10:51 heeft Markus Volk <f_l_k@t-online.de> het volgende geschreven: [...] # radeonsi requires LLVM -GALLIUMDRIVERS_ASAHI = "${@bb.utils.contains('PACKAGECONFIG', 'asahi libclc', ',asahi', '', d)}" +GALLIUMDRIVERS_ASAHI = "${@bb.utils.contains('PACKAGECONFIG', 'asahi libclc opencl', ',asahi', '', d)}" GALLIUMDRIVERS_IRIS = "${@bb.utils.contains('PACKAGECONFIG', 'intel libclc', ',iris', '', d)}" GALLIUMDRIVERS_RADEONSI = "${@bb.utils.contains('PACKAGECONFIG', 'amd', ',radeonsi', '', d)}" GALLIUMDRIVERS_LLVMPIPE = ",llvmpipe"
Very pedantic nit-pick: the comment for these 4 lines only talks about radeon-si, I think we should either remove that comment or properly mention asahi, iris and radeonsi and their dependencies. regards, Koen