On Sat, Aug 31, 2024 at 09:18:43AM +0200, Lorenzo Bianconi wrote:
> clk-en7523 driver for EN7581 SoC is mapping all the scu memory region
> while it is configuring the chip-scu one via a syscon. Update the reg
> mapping definition for this device. This patch does not introduce any
> backward incompatibility since the dts for EN7581 SoC is not public yet.
What does "not public yet" mean? The bindings are public, so someone
could have built a dtb without whatever non-public dts you currently
have. Do you mean, has not yet made it into a released kernel?
Cheers,
Conor.
>
> Signed-off-by: Lorenzo Bianconi <lorenzo@xxxxxxxxxx>
> ---
> .../devicetree/bindings/clock/airoha,en7523-scu.yaml | 12 +++---------
> 1 file changed, 3 insertions(+), 9 deletions(-)
>
> diff --git a/Documentation/devicetree/bindings/clock/airoha,en7523-scu.yaml b/Documentation/devicetree/bindings/clock/airoha,en7523-scu.yaml
> index 84353fd09428..ca426c328535 100644
> --- a/Documentation/devicetree/bindings/clock/airoha,en7523-scu.yaml
> +++ b/Documentation/devicetree/bindings/clock/airoha,en7523-scu.yaml
> @@ -34,8 +34,8 @@ properties:
> - airoha,en7581-scu
>
> reg:
> - minItems: 2
> - maxItems: 4
> + minItems: 1
> + maxItems: 2
>
> "#clock-cells":
> description:
> @@ -75,9 +75,6 @@ allOf:
> reg:
> items:
> - description: scu base address
> - - description: misc scu base address
> - - description: reset base address
> - - description: pb scu base address
>
> additionalProperties: false
>
> @@ -98,10 +95,7 @@ examples:
>
> scuclk: clock-controller@1fa20000 {
> compatible = "airoha,en7581-scu";
> - reg = <0x0 0x1fa20000 0x0 0x400>,
> - <0x0 0x1fb00000 0x0 0x90>,
> - <0x0 0x1fb00830 0x0 0x8>,
> - <0x0 0x1fbe3400 0x0 0xfc>;
> + reg = <0x0 0x1fb00000 0x0 0x970>;
> #clock-cells = <1>;
> #reset-cells = <1>;
> };
>
> --
> 2.46.0
>
Attachment:
signature.asc
Description: PGP signature
