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

From: Yinbo Zhu
Date: Wed Sep 28 2022 - 23:57:32 EST




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

Best regards,
Krzysztof
Hi Krzysztof,

I have a function test for these three patch, that it's okay.
for binding patch I have a compile test, as follow, it is okay.
root@m-pc:/home/m/workspace/test/code/upstream# make DT_CHECKER_FLAGS=-m dt_binding_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/thermal/ls2k-thermal.yaml

BRs,
Yinbo Zhu.