On Tue, Mar 08, 2022 at 12:51:25PM +0530, Kuldeep Singh wrote: > Pl022 clock-names can be one of following: > ['apb_pclk'] or ['sspclk', 'apb_pclk'] > > The current schema refers to second case only. Make necessary changes to > incorporate both the cases and resolve below dtc warning: > clock-names: 'apb_pclk' is not one of ['sspclk'] > > Signed-off-by: Kuldeep Singh <singh.kuldeep87k@xxxxxxxxx> > --- > v2: > - Reword commit message > - Drop SSPCLK > > Documentation/devicetree/bindings/spi/spi-pl022.yaml | 10 ++++++---- > 1 file changed, 6 insertions(+), 4 deletions(-) > > diff --git a/Documentation/devicetree/bindings/spi/spi-pl022.yaml b/Documentation/devicetree/bindings/spi/spi-pl022.yaml > index 7d36e15db5b3..6cfab948624e 100644 > --- a/Documentation/devicetree/bindings/spi/spi-pl022.yaml > +++ b/Documentation/devicetree/bindings/spi/spi-pl022.yaml > @@ -38,11 +38,13 @@ properties: > maxItems: 2 > > clock-names: > - items: > + oneOf: > - enum: > - - SSPCLK > - - sspclk > - - const: apb_pclk > + - apb_pclk Are you going to make the driver work with no SPI clock? Because this change is saying that it is valid to have the APB bus clock and no SPI clock. Rob > + - items: > + - enum: > + - sspclk > + - const: apb_pclk > > pl022,autosuspend-delay: > description: delay in ms following transfer completion before the > -- > 2.25.1 > >
