On 5/11/2022 1:45 PM, Krzysztof Kozlowski wrote:
On 11/05/2022 09:53, J, KEERTHY wrote:
Any comments on this version?
You got Rob's review, so you don't need also mine.
If you want, then in general look okay, except the description for "reg"
you could split per items:
reg:
items:
- description: VTM cfg1 register space
- description: VTM cfg2 register space
- description: efuse register space
It's more obvious what items you expect.
Hi Krzysztof,
DTEX
Documentation/devicetree/bindings/thermal/ti,j72xx-thermal.example.dts
Documentation/devicetree/bindings/thermal/ti,j72xx-thermal.yaml: found
duplicate key "description" with value "VTM cfg2 register space"
(original value: "VTM cfg1 register space")
Documentation/devicetree/bindings/Makefile:26: recipe for target
'Documentation/devicetree/bindings/thermal/ti,j72xx-thermal.example.dts'
failed
make[1]: ***
[Documentation/devicetree/bindings/thermal/ti,j72xx-thermal.example.dts] Er
I believe multiple instances of description is failing. So can i keep
the description as is?
It looks weird. Can you share your code (pastebin/github/paste here)?
My bad. I had missed the '-' before description & hence it was erring
out. I fixed. I will post v7 of this patch alone.
Thanks,
Keerthy
Best regards,
Krzysztof
