On Thu, Sep 04, 2025 at 12:59:14PM +0200, Krzysztof Kozlowski wrote:

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

quoted

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!

Ok, I see what we messed up there around compatible property when adding
simple-bus. Sorry about this.

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.

Point taken.

The intention is absolutely not to waste the time of reviewers or
maintainers. We will make sure to apply the lesson learned here.
Thanks again for your time.

Thanks,
Pavan