On Fri, Aug 23, 2024 at 09:05:27PM +0530, Karthikeyan Krishnasamy wrote: > Add devicetree binding documentation for Relfor Saib > board which uses Rockchip RV1109 SoC > > Signed-off-by: Karthikeyan Krishnasamy <karthikeyan@xxxxxxxxxxx> > --- > Documentation/devicetree/bindings/arm/rockchip.yaml | 6 ++++++ > 1 file changed, 6 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/rockchip.yaml b/Documentation/devicetree/bindings/arm/rockchip.yaml > index 1ef09fbfdfaf..29f7e09ae443 100644 > --- a/Documentation/devicetree/bindings/arm/rockchip.yaml > +++ b/Documentation/devicetree/bindings/arm/rockchip.yaml > @@ -848,6 +848,12 @@ properties: > - radxa,zero-3w > - const: rockchip,rk3566 > > + - description: Relfor SAIB board > + items: > + - const: relfor,saib > + - enum: > + - rockchip,rv1109 This does not make sense to me. Why do you have an enum for the SoC model, implying that this SAIB board would have more than one possible SoC? I'd expect to see - const: rockvhip,rv1109 > + > - description: Rikomagic MK808 v1 > items: > - const: rikomagic,mk808 > -- > 2.39.2 >
Attachment:
signature.asc
Description: PGP signature
