[PATCH v5 2/4] dt-bindings: serial: amlogic,meson-uart: Add compatible string for T7
Krzysztof Kozlowski
krzysztof.kozlowski at linaro.org
Fri Jun 23 01:51:41 PDT 2023
On 23/06/2023 10:12, Lucas Tanure wrote:
> Amlogic T7 SoCs uses the same UART controller as S4 SoCs and G12A.
> There is no need for an extra compatible line in the driver, but
> add T7 compatible line for documentation.
>
> Signed-off-by: Lucas Tanure <tanure at linux.com>
> ---
> .../devicetree/bindings/serial/amlogic,meson-uart.yaml | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/serial/amlogic,meson-uart.yaml b/Documentation/devicetree/bindings/serial/amlogic,meson-uart.yaml
> index 01ec45b3b406..ad970c9ed1c7 100644
> --- a/Documentation/devicetree/bindings/serial/amlogic,meson-uart.yaml
> +++ b/Documentation/devicetree/bindings/serial/amlogic,meson-uart.yaml
> @@ -50,6 +50,10 @@ properties:
> items:
> - const: amlogic,meson-g12a-uart
> - const: amlogic,meson-gx-uart
> + - description: UART controller on T7 compatible SoCs
Your description is rather incorrect. This is UART on SoCs compatible
with S4, not with T7. Otherwise what do you expect to grow later when
adding more compatible devices? Just drop the description, it's kind of
obvious when done correctly (but can be misleading if done wrong).
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski at linaro.org>
Best regards,
Krzysztof
More information about the linux-amlogic
mailing list