Hi Krzysztof, On 01/05/24 3:56 pm, Krzysztof Kozlowski wrote: > On 30/04/2024 14:24, MD Danish Anwar wrote: >> Add documentation for ti,pa-stats property which is syscon regmap for >> PA_STATS register. This will be used to dump statistics maintained by >> ICSSG firmware. >> >> Signed-off-by: MD Danish Anwar <danishanwar@xxxxxx> >> --- >> Documentation/devicetree/bindings/net/ti,icssg-prueth.yaml | 6 ++++++ >> 1 file changed, 6 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/net/ti,icssg-prueth.yaml b/Documentation/devicetree/bindings/net/ti,icssg-prueth.yaml >> index e253fa786092..abf372f7191b 100644 >> --- a/Documentation/devicetree/bindings/net/ti,icssg-prueth.yaml >> +++ b/Documentation/devicetree/bindings/net/ti,icssg-prueth.yaml >> @@ -55,6 +55,11 @@ properties: >> description: >> phandle to MII_RT module's syscon regmap >> >> + ti,pa-stats: >> + $ref: /schemas/types.yaml#/definitions/phandle >> + description: >> + phandle to PA_STATS module's syscon regmap > > One register needed? Then use phandle-array syntax - see examples. No this is not one register only. PA_STATS is a set of registers. My bad I should have mentioned *registers* in the commit description. Just like MII_RT and MII_G_RT modules, PA_STATS is also a set of registers where different statistics are dumped by ICSSG firmware. Moduling this as syscon will help the driver read/write those statistics using regmap_read() / write() just like the driver currently read / write registers from mii_rt and mii_g_rt regmaps. I have tried to describe this node *ti,pa-stats* the same way as "ti,mii-g-rt" and "ti,mii-rt" > Explain in description the purpose of this register in the context of > *this* device. > Sure will do that. > Best regards, > Krzysztof > -- Thanks and Regards, Danish
