On 26/03/2024 09:58, Kanak Shilledar wrote:
> From: Kanak Shilledar <kanakshilledar111@xxxxxxxxxxxxxx>
>
> Convert the Actions Semi Owl UART to newer DT schema.
> Created DT schema based on the .txt file which had
> `compatible`, `reg` and `interrupts` as the required properties.
> This binding is used by Actions S500 and S700 SoC.
Please mention changes from pure conversion. You added clocks.
>
> Signed-off-by: Kanak Shilledar <kanakshilledar111@xxxxxxxxxxxxxx>
> ---
> Changes in v2
> - Added Clock property to fix the warning of `dts/actions/s700-cubieboard7.dtb:
> serial@e0126000: Unevaluated properties are not allowed
> ('clocks' was unexpected)` thrown by `make dtbs_check`
> - Changed the compatible identifier for having both the compatible names
> earlier was throwing `owl-s500-cubieboard6.dtb: serial@b0120000: compatible:
> ['actions,s500-uart', 'actions,owl-uart'] is too long` warning.
> - Updated commit message
>
Do not attach (thread) your patchsets to some other threads (unrelated
or older versions). This buries them deep in the mailbox and might
interfere with applying entire sets.
>
> .../bindings/serial/actions,owl-uart.txt | 16 -------
> .../bindings/serial/actions,owl-uart.yaml | 46 +++++++++++++++++++
> 2 files changed, 46 insertions(+), 16 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/serial/actions,owl-uart.txt
> create mode 100644 Documentation/devicetree/bindings/serial/actions,owl-uart.yaml
>
...
> +
> +required:
> + - compatible
> + - reg
> + - interrupts
> +
> +unevaluatedProperties: false
> +
> +examples:
> + - |
> + #include <dt-bindings/interrupt-controller/arm-gic.h>
> + serial@b0126000 {
> + compatible = "actions,s500-uart", "actions,owl-uart";
> + reg = <0xb0126000 0x1000>;
> + interrupts = <GIC_SPI 32 IRQ_TYPE_LEVEL_HIGH>;
Please extend the example and make it complete by adding clocks (based
on DTS).
With two changes above:
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
---
This is an automated instruction, just in case, because many review tags
are being ignored. If you know the process, you can skip it (please do
not feel offended by me posting it here - no bad intentions intended).
If you do not know the process, here is a short explanation:
Please add Acked-by/Reviewed-by/Tested-by tags when posting new
versions, under or above your Signed-off-by tag. Tag is "received", when
provided in a message replied to you on the mailing list. Tools like b4
can help here. However, there's no need to repost patches *only* to add
the tags. The upstream maintainer will do that for tags received on the
version they apply.
https://elixir.bootlin.com/linux/v6.5-rc3/source/Documentation/process/submitting-patches.rst#L577
Best regards,
Krzysztof
