Re: [PATCH v2 1/2] dt-bindings: arm: mediatek: Add MT8186 Starmie Chromebooks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Nov 26, 2024 at 2:34 AM Conor Dooley <conor@xxxxxxxxxx> wrote:
>
> On Mon, Nov 25, 2024 at 08:21:28AM +0000, Wojciech Macek wrote:
> > Add an entry for the MT8186 based Starmie Chromebooks, also known as the
> > ASUS Chromebook Enterprise CM30 Detachable (CM3001). The device is
> > a tablet style chromebook.
> >
> > Signed-off-by: Wojciech Macek <wmacek@xxxxxxxxxxxx>
> > ---
> >
> > Changelog v2-v1:
> >  - Fixed items/const bidings description in mediatek.yaml
> >
> >  Documentation/devicetree/bindings/arm/mediatek.yaml | 13 +++++++++++++
> >  1 file changed, 13 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/arm/mediatek.yaml b/Documentation/devicetree/bindings/arm/mediatek.yaml
> > index 1d4bb50fcd8d9..6191a5320c148 100644
> > --- a/Documentation/devicetree/bindings/arm/mediatek.yaml
> > +++ b/Documentation/devicetree/bindings/arm/mediatek.yaml
> > @@ -263,6 +263,19 @@ properties:
> >            - const: google,steelix-sku196608
> >            - const: google,steelix
> >            - const: mediatek,mt8186
> > +      - description: Google Starmie (ASUS Chromebook Enterprise CM30 (CM3001))
> > +        items:
> > +          - const: google,starmie-sku0
> > +          - const: google,starmie-sku2
> > +          - const: google,starmie-sku3
>
> Compatible order here is back to making little sense. Why is the oldest
> SKU, in both cases, appearing first?

FWIW the SKU number has no particular ordering. They were released together.
SKU denotes different configurations (memory, storage, touchscreen and other
component options). OOTH revisions would actually have chronological meaning.


ChenYu

> > +          - const: google,starmie
> > +          - const: mediatek,mt8186
> > +      - description: Google Starmie (ASUS Chromebook Enterprise CM30 (CM3001))
> > +        items:
> > +          - const: google,starmie-sku1
> > +          - const: google,starmie-sku4
> > +          - const: google,starmie
> > +          - const: mediatek,mt8186
> >        - description: Google Steelix (Lenovo 300e Yoga Chromebook Gen 4)
> >          items:
> >            - enum:
> > --
> > 2.47.0.371.ga323438b13-goog
> >





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux