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

Conor Dooley
2025-02-04T18:19:48Z

[-- Attachment #1: Type: text/plain, Size: 1013 bytes --]

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 #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]