| Message ID | 20250408175146.979557-2-elder@riscstar.com |
|---|---|
| State | New |
| Headers | show |
| Series | serial: 8250_of: support an optional bus clock | expand |
On Tue, Apr 8, 2025 at 12:51 PM Alex Elder <elder@riscstar.com> wrote: > > The SpacemiT UART driver requires a bus clock to be enabled in addition > to the primary function clock. Add the option to specify two clocks > for an 8250-compatible UART, named "core" and "bus". This should be conditional on the compatible associated with 2 clocks. > > Signed-off-by: Alex Elder <elder@riscstar.com> > --- > Documentation/devicetree/bindings/serial/8250.yaml | 6 +++++- > 1 file changed, 5 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/serial/8250.yaml b/Documentation/devicetree/bindings/serial/8250.yaml > index dc0d52920575f..1adf935b7f36f 100644 > --- a/Documentation/devicetree/bindings/serial/8250.yaml > +++ b/Documentation/devicetree/bindings/serial/8250.yaml > @@ -135,7 +135,11 @@ properties: > clock-frequency: true > > clocks: > - maxItems: 1 > + oneOf: > + - maxItems: 1 > + - items: > + - const: core > + - const: bus 'clocks' does not take string values. You want: minItems: 1 items: - description: ... - description: ... Then an if/then schema with 'minItems: 2' for spacemit and 'maxItems: 1' for everyone else.
diff --git a/Documentation/devicetree/bindings/serial/8250.yaml b/Documentation/devicetree/bindings/serial/8250.yaml index dc0d52920575f..1adf935b7f36f 100644 --- a/Documentation/devicetree/bindings/serial/8250.yaml +++ b/Documentation/devicetree/bindings/serial/8250.yaml @@ -135,7 +135,11 @@ properties: clock-frequency: true clocks: - maxItems: 1 + oneOf: + - maxItems: 1 + - items: + - const: core + - const: bus resets: maxItems: 1
The SpacemiT UART driver requires a bus clock to be enabled in addition to the primary function clock. Add the option to specify two clocks for an 8250-compatible UART, named "core" and "bus". Signed-off-by: Alex Elder <elder@riscstar.com> --- Documentation/devicetree/bindings/serial/8250.yaml | 6 +++++- 1 file changed, 5 insertions(+), 1 deletion(-)