RE: [EXT] Re: [PATCH 1/2 v7] dt-bindings: watchdog: marvell GTI system watchdog driver

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

 




> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
> Sent: Tuesday, May 9, 2023 1:38 PM
> To: Bharat Bhushan <bbhushan2@xxxxxxxxxxx>; wim@xxxxxxxxxxxxxxxxxx;
> linux@xxxxxxxxxxxx; robh+dt@xxxxxxxxxx; krzysztof.kozlowski+dt@xxxxxxxxxx;
> linux-watchdog@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; linux-
> kernel@xxxxxxxxxxxxxxx; Sunil Kovvuri Goutham <sgoutham@xxxxxxxxxxx>
> Subject: Re: [EXT] Re: [PATCH 1/2 v7] dt-bindings: watchdog: marvell GTI system
> watchdog driver
> 
> On 09/05/2023 09:26, Bharat Bhushan wrote:
> 
> 
> >>> +properties:
> >>> +  compatible:
> >>> +    oneOf:
> >>> +      - const: marvell,octeontx2-wdt
> >>
> >> Why is this alone? Judging by the enum below, octeontx2 is not specific.
> >>
> >>> +      - items:
> >>> +          - enum:
> >>> +              - marvell,octeontx2-95xx-wdt
> >>> +              - marvell,octeontx2-96xx-wdt
> >>> +              - marvell,octeontx2-98xx-wdt
> >>
> >> We don't allow wildcards in general
> >
> > Marvell have octeontx2 series of processor which have watchdog timer.
> > In 95xx,98xx,96xx are the processors in octeontx2 series of processor. So
> octeontx2-95xx is on soc, octeontx2-96xx is another and so on.
> 
> No, 95xx is not a processor. Otherwise please point me to exact product
> datasheet. Hint: I checked it.

Looks like 95xx data sheet is not public, will remove in that case.

> 
> >
> >>
> >>> +          - const: marvell,octeontx2-wdt
> >>> +      - const: marvell,cn10k-wdt
> >>
> >> Same question - why is this alone?
> > Same here, Marvell have cn10k series of processors and cn10kx and cnf10kx are
> the processor in this series.
> 
> I don't understand how does it explain my concern. This is alone because there
> are series of processors? How is that related?

Tried to make it look like other drivers. Let's keep it simple, we want to enable this only for below ones

properties:
  compatible:
    enum:
     - marvell,cn10k-wdt
     - marvell,octeontx2-wdt

Are you good with that?

Thanks
-Bharat

> 
> >
> > One of the difference between octeontx2 and cn10k series processor is
> number of timers available. Which within the available set of timers one of the
> timer is programmed to be watchdog timer.
> 
> Wrap your replies. It's difficult to read.
> 
> >
> > Can you please propose how you want these compatible to be defined?
> 
> https://urldefense.proofpoint.com/v2/url?u=https-
> 3A__elixir.bootlin.com_linux_v6.4-
> 2Drc1_source_Documentation_devicetree_bindings_soc_qcom_qcom-
> 2Ceud.yaml-
> 23L19&d=DwICaQ&c=nKjWec2b6R0mOyPaz7xtfQ&r=PAAlWswPe7d8gHlGbCLmy
> 2YezyK7O3Hv_t2heGnouBw&m=sbDC9A17UO1l_M7xW5546TQhAMxoejy6M_sv
> PitOn_9sOxb0ru3H7X9eEW00Gqna&s=tQrCoVSHNEOd9CkEeu6leJbmP0rtbL5Vd
> WlE9GQ-GTI&e=
> 
> Best regards,
> Krzysztof





[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