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

[Krzysztof Kozlowski]

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.

> 
>>
>>> +          - 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?

> 
> 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://elixir.bootlin.com/linux/v6.4-rc1/source/Documentation/devicetree/bindings/soc/qcom/qcom,eud.yaml#L19

Best regards,
Krzysztof