Re: [PATCH 1/3] dt-bindings: soc: renesas: add MFIS binding documentation

[Wolfram Sang <wsa+renesas@sang-engineering.com>]

[-- Attachment #1: Type: text/plain, Size: 1173 bytes --]


> > In case you mean this as unanswered questions to v1: This describes the
> > device specific second mbox cell. Like Tegra does it here (even with
> 
> I asked Linux ABI. Device specific numbers is not Linux ABI, because
> Linux is not device.

I have no idea what you want here. I see it like this: DT is OS
agnostic. So, the flags are to describe the hardware. The Linux driver
then does the proper things according to these flags. Other OS driver
might do other things.

> > shifts instead of plain numbers):
> > 
> > include/dt-bindings/mailbox/tegra186-hsp.h

And I still have no idea why my header is different than the Tegra one.

> > I really have no idea what is still missing?
> 
> It's not a binding, drop the header from the bindings.

So, I should add the header to the driver? Can do, but then I have to
remove the reference to the header from the binding doc. Namely, this
paragraph:

+      The first cell is the channel number as specified in the documentation
+      of the SoC. The second cell may specify flags as described in the file
+      <dt-bindings/soc/renesas,r8a78000-mfis.h>.

Not helpful for the user, or?


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]