[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240627144538.mp7n2l7kupi7gt4l@straddle>
Date: Thu, 27 Jun 2024 09:45:38 -0500
From: Nishanth Menon <nm@...com>
To: MD Danish Anwar <danishanwar@...com>
CC: Suman Anna <s-anna@...com>, Conor Dooley <conor+dt@...nel.org>,
Krzysztof
Kozlowski <krzk+dt@...nel.org>,
Rob Herring <robh@...nel.org>,
Santosh
Shilimkar <ssantosh@...nel.org>, <devicetree@...r.kernel.org>,
<linux-arm-kernel@...ts.infradead.org>, <linux-kernel@...r.kernel.org>,
<srk@...com>, Vignesh Raghavendra <vigneshr@...com>,
Roger Quadros
<rogerq@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Subject: Re: [PATCH v3 2/2] dt-bindings: soc: ti: pruss: Add documentation
for PA_STATS support
On 21:03-20240625, MD Danish Anwar wrote:
> Add documentation for pa-stats node which is syscon regmap for
> PA_STATS registers. This will be used to dump statistics maintained by
> ICSSG firmware.
>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
> Signed-off-by: MD Danish Anwar <danishanwar@...com>
> ---
> .../devicetree/bindings/soc/ti/ti,pruss.yaml | 20 +++++++++++++++++++
> 1 file changed, 20 insertions(+)
>
> diff --git a/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml b/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml
> index c402cb2928e8..3cb1471cc6b6 100644
> --- a/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml
> +++ b/Documentation/devicetree/bindings/soc/ti/ti,pruss.yaml
> @@ -278,6 +278,26 @@ patternProperties:
>
> additionalProperties: false
>
> + ^pa-stats@[a-f0-9]+$:
> + description: |
> + PA-STATS sub-module represented as a SysCon. PA_STATS is a set of
> + registers where different statistics related to ICSSG, are dumped by
> + ICSSG firmware. This syscon sub-module will help the device to
> + access/read/write those statistics.
> +
> + type: object
> +
> + additionalProperties: false
> +
> + properties:
> + compatible:
> + items:
> + - const: ti,pruss-pa-st
> + - const: syscon
> +
> + reg:
> + maxItems: 1
> +
Is this a patch in a driver series? I do not see the driver part..
if this is a memory map, why doesn't bus model work?
> interrupt-controller@[a-f0-9]+$:
> description: |
> PRUSS INTC Node. Each PRUSS has a single interrupt controller instance
> --
> 2.34.1
>
--
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3 1A34 DDB5 849D 1736 249D
Powered by blists - more mailing lists