Re: [RFC v2 PATCH 01/10] dt-bindings: net: ti: Adds DUAL-EMAC mode support on PRU-ICSS2 for AM57xx SOCs

[Conor Dooley]

On Mon, Feb 03, 2025 at 05:59:55PM +0530, Basharath Hussain Khaja wrote:
> >> >> +    $ref: /schemas/types.yaml#/definitions/phandle
> >> >> +    description:
> >> >> +      phandle to Enhanced Capture (eCAP) event for ICSS
> >> > 
> >> > Why do you need phandles for these things, can they not be looked up by
> >> > compatible? (e.g. multiple devices on one SoC).
> >> > 
> >> 
> >> ecap is another peripheral similar to IEP in ICSSM/ICSSG. We have created a
> >> separate driver for possible reuse with ICSSG in future.
> > 
> > That's not an answer to my question.
> > 
> 
> We can use compatible if we have only one instance of a peripheral in the SOC. 
> On the AM57x SOC we have two identical ICSS instances(ICSS1 and ICSS2). So we 
> use phandles to differentiate between the two instances. Currently this patch 
> series adds support for ICSS2 instance on the AM57x SOC. Support for ICSS1 instance 
> will be added in subsequent patches.

Cool, that's an acceptance answer, thanks.

Attachment: signature.asc
Description: PGP signature