On 04/09/2025 12:49, Krzysztof Kozlowski wrote:

On 04/09/2025 12:16, Pavan Kondeti wrote:

quoted

quoted

quoted

+  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?

quoted

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.

quoted

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