Re: [PATCH v5 2/3] dt-bindings: thermal: add loongson2k thermal binding

From: Yinbo Zhu
Date: Thu Sep 29 2022 - 03:07:42 EST




在 2022/9/29 下午3:00, Krzysztof Kozlowski 写道:
On 29/09/2022 05:57, Yinbo Zhu wrote:


在 2022/9/29 上午11:42, Yinbo Zhu 写道:


在 2022/9/28 下午10:18, Krzysztof Kozlowski 写道:
On 28/09/2022 10:37, Krzysztof Kozlowski wrote:
On 28/09/2022 10:37, Yinbo Zhu wrote:
Add the loongson2k thermal binding with DT schema format using
json-schema.

Signed-off-by: Yinbo Zhu <zhuyinbo@xxxxxxxxxxx>

Please add Acked-by/Reviewed-by tags when posting new versions. However,
there's no need to repost patches *only* to add the tags. The upstream
maintainer will do that for acks received on the version they apply.

https://elixir.bootlin.com/linux/v5.17/source/Documentation/process/submitting-patches.rst#L540


If a tag was not added on purpose, please state why and what changed.

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>

... and please test your patches before sending :(
You said is refer that "reg: [[0, 534779136], [0, 48]] is too long" ?
Need fix that warning, right?

Yes. You said you tested it but then sent with an error... so it's not
really a testing.
sorry, I did do some testing. but I think It is okay that can generate a dtb without reporting an error when compile yaml file, in fact, I ignore the warning, I will fix it in v6.

Best regards,
Krzysztof