Re: [PATCH v2 2/8] dt-bindings: counter: Add new ti,am62-eqep compatible

From: Judith Mendez
Date: Wed May 29 2024 - 13:21:03 EST


Hi Conor,

On 5/25/24 10:23 AM, Conor Dooley wrote:
On Fri, May 24, 2024 at 04:30:32PM -0500, Judith Mendez wrote:
On 5/24/24 1:38 PM, Conor Dooley wrote:
On Thu, May 23, 2024 at 06:15:10PM -0500, Judith Mendez wrote:
Add new compatible ti,am62-eqep for TI K3 devices. If a device
uses this compatible, require power-domains property.

Since there is only one functional and interface clock for eqep,
clock-names is not really required. The clock-name also changed
for TI K3 SoCs so make clock-names optional for the new compatible
since there is only one clock that is routed to the IP.

Really the clock should be named after the function it has in the IP
block - it looks like "sysoutclk" is more likely the name of the clock
routed to the IP rather than the role it has?

It is the name of the clock, though id like to keep sysclkout for
backwards compatibility, even though the name is confusing.

FWIW, I was not suggesting that it be renamed.

Understood, thanks.


Thanks,
Conor.