| Message ID | 20250119130105.2833517-2-benjamin.larsson@genexis.eu |
|---|---|
| State | New |
| Headers | show |
| Series | Airoha UART support | expand |
On Sun, Jan 19, 2025 at 02:01:04PM +0100, Benjamin Larsson wrote: > The Airoha SoC family have a mostly 16550-compatible UART > and High-Speed UART hardware with the exception of custom > baud rate settings register. > > Signed-off-by: Benjamin Larsson <benjamin.larsson@genexis.eu> > --- > Documentation/devicetree/bindings/serial/8250.yaml | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/Documentation/devicetree/bindings/serial/8250.yaml b/Documentation/devicetree/bindings/serial/8250.yaml > index 692aa05500fd..2fbb972e5460 100644 > --- a/Documentation/devicetree/bindings/serial/8250.yaml > +++ b/Documentation/devicetree/bindings/serial/8250.yaml > @@ -63,6 +63,8 @@ properties: > - const: mrvl,pxa-uart > - const: nuvoton,wpcm450-uart > - const: nuvoton,npcm750-uart > + - const: airoha,airoha-uart > + - const: airoha,airoha-hsuart I assume you placed it matching existing order (kind of alphabetical for compatibles with vendors)? Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
On 20/01/2025 16:46, Benjamin Larsson wrote: >>>>> diff --git a/Documentation/devicetree/bindings/serial/8250.yaml b/Documentation/devicetree/bindings/serial/8250.yaml >>>>> index 692aa05500fd..2fbb972e5460 100644 >>>>> --- a/Documentation/devicetree/bindings/serial/8250.yaml >>>>> +++ b/Documentation/devicetree/bindings/serial/8250.yaml >>>>> @@ -63,6 +63,8 @@ properties: >>>>> - const: mrvl,pxa-uart >>>>> - const: nuvoton,wpcm450-uart >>>>> - const: nuvoton,npcm750-uart >>>>> + - const: airoha,airoha-uart >>>>> + - const: airoha,airoha-hsuart >>>> I assume you placed it matching existing order (kind of alphabetical for >>>> compatibles with vendors)? >>>> >>>> Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> >>>> >>>> Best regards, >>>> Krzysztof >>>> >>> Hi, I placed it after nuvoton that was the most recent addition to the >>> serial8250_config uart_config[] table that was also added to the >> But wasn't nuvoton placed in specific, ordered place? > > Well not really as wpcm450-uart is placed before npcm750-uart in Uh, indeed. :/ > 8250.yaml. But I will move the entries to below ns16850 in the next > revision to keep the vendor list in alphabetical order. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/serial/8250.yaml b/Documentation/devicetree/bindings/serial/8250.yaml index 692aa05500fd..2fbb972e5460 100644 --- a/Documentation/devicetree/bindings/serial/8250.yaml +++ b/Documentation/devicetree/bindings/serial/8250.yaml @@ -63,6 +63,8 @@ properties: - const: mrvl,pxa-uart - const: nuvoton,wpcm450-uart - const: nuvoton,npcm750-uart + - const: airoha,airoha-uart + - const: airoha,airoha-hsuart - const: nvidia,tegra20-uart - const: nxp,lpc3220-uart - items:
The Airoha SoC family have a mostly 16550-compatible UART and High-Speed UART hardware with the exception of custom baud rate settings register. Signed-off-by: Benjamin Larsson <benjamin.larsson@genexis.eu> --- Documentation/devicetree/bindings/serial/8250.yaml | 2 ++ 1 file changed, 2 insertions(+)