On 04/09/2025 12:49, Krzysztof Kozlowski wrote: > On 04/09/2025 12:16, Pavan Kondeti wrote: >>>> + compatible: >>>> + allOf: >>>> + - const: gunyah-hypervisor >>>> + - const: simple-bus >>> >>> What? No. >>> >>> Don't create patches with AI. >>> >> I am next to Hrishabh when he is writing this patch. I can confirm he >> did not use AI :-) not sure what tool Krzysztof is using to catch > > My brain? > >> patches being written with AI, that tool needs some improvement for >> sure. > > Heh? Seriously, instead replying something like this think from how is > it possible to come with such syntax? > > It does not exist. NOWHERE. > > It had to be completely hallucinated by AI because I cannot imagine > coming with code which is completely different then EVERYTHING else. > There is no single code looking like that. > > >> >> I will let Hrishabh share why he put simple-bus here. > > > It is not about simple-bus! > And to clarify: it's not only about this part of the binding. Entire binding is terrible, does not meet any basic standards, does not follow basic principles of writing DTS. I cannot imagine this code passing internal review, so hallucinated AI is the most reasonable explanation. Sorry, if you send extremely poor code using patterns which do not exist, that;s either huge waste of community time or AI-based waste of community time. Best regards, Krzysztof
