Re: [PATCH v3] dt-bindings: phy: rockchip-emmc-phy: Convert to dtschema

[Krzysztof Kozlowski <krzk@kernel.org>]

On 13/06/2024 11:44, Johan Jonker wrote:
>> ---
> 
> Add ack request from phy maintainer here.

Why? What do you mean for that? Why phy maintainer needs to ack patches
he is going to take?

> 
>> Changes in v3:
>>     - fix `reg` in example being too long
>>
>> Tested against `rockchip/rk3399-firefly.dtb`, `rockchip/rk3399-orangepi.dtb`
>> and `rockchip/rk3399-pinebook-pro.dtb`.
>>
>>  .../bindings/phy/rockchip,emmc-phy.yaml       | 79 +++++++++++++++++++
>>  .../bindings/phy/rockchip-emmc-phy.txt        | 43 ----------
>>  .../devicetree/bindings/soc/rockchip/grf.yaml |  2 +-
>>  3 files changed, 80 insertions(+), 44 deletions(-)
>>  create mode 100644 Documentation/devicetree/bindings/phy/rockchip,emmc-phy.yaml
>>  delete mode 100644 Documentation/devicetree/bindings/phy/rockchip-emmc-phy.txt
>>
>> diff --git a/Documentation/devicetree/bindings/phy/rockchip,emmc-phy.yaml b/Documentation/devicetree/bindings/phy/rockchip,emmc-phy.yaml
>> new file mode 100644
>> index 000000000000..85d74b343991
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/phy/rockchip,emmc-phy.yaml
>> @@ -0,0 +1,79 @@
> 
>> +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
> 
> You are converting an existing document, so GPL 2 only.

Which copyrightable part was copied? This comment is not correct in
general, because conversions are dual-licensed (there are exceptions,
but that's the generic rule).

Best regards,
Krzysztof