[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <175341132347.3513.7184287611040628050@lazor>
Date: Thu, 24 Jul 2025 19:42:03 -0700
From: Stephen Boyd <sboyd@...nel.org>
To: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>, Anup Patel <apatel@...tanamicro.com>, Bartosz Golaszewski <brgl@...ev.pl>, Conor Dooley <conor+dt@...nel.org>, Jassi Brar <jassisinghbrar@...il.com>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Linus Walleij <linus.walleij@...aro.org>, Michael Turquette <mturquette@...libre.com>, Mika Westerberg <mika.westerberg@...ux.intel.com>, Rafael J . Wysocki <rafael@...nel.org>, Rob Herring <robh@...nel.org>, Thomas Gleixner <tglx@...utronix.de>, Uwe Kleine-König <ukleinek@...nel.org>
Cc: Palmer Dabbelt <palmer@...belt.com>, Paul Walmsley <paul.walmsley@...ive.com>, Alexandre Ghiti <alex@...ti.fr>, Len Brown <lenb@...nel.org>, Sunil V L <sunilvl@...tanamicro.com>, Rahul Pathak <rpathak@...tanamicro.com>, Leyfoon Tan <leyfoon.tan@...rfivetech.com>, Atish Patra <atish.patra@...ux.dev>, Andrew Jones <ajones@...tanamicro.com>, Samuel Holland <samuel.holland@...ive.com>, Anup Patel <anup@...infault.org>, linux-clk@...r.kernel.org, devicetree@...r.kernel.org, linux-acpi@...r.kernel.org, linux-riscv@...ts.infradead.org, linux-kernel@...r.kernel.org, Anup Patel <apatel@...tanamicro.com>, Conor Dooley <conor.dooley@...rochip.com>
Subject: Re: [PATCH v8 08/24] dt-bindings: clock: Add RPMI clock service message proxy bindings
Quoting Anup Patel (2025-07-04 00:03:40)
> Add device tree bindings for the RPMI clock service group based
> message proxy implemented by the SBI implementation (machine mode
> firmware or hypervisor).
>
> The RPMI clock service group is defined by the RISC-V platform
> management interface (RPMI) specification.
>
> Reviewed-by: Conor Dooley <conor.dooley@...rochip.com>
> Signed-off-by: Anup Patel <apatel@...tanamicro.com>
[...]
> +additionalProperties: false
> +
> +examples:
> + - |
> + clock-controller {
Maybe the name should be 'clock-service' then? I don't understand SBI so
not sure why this is in DT to begin with. Is something consuming this
node? Or a driver is binding to it?
> + compatible = "riscv,rpmi-mpxy-clock";
> + mboxes = <&rpmi_shmem_mbox 0x8>;
> + riscv,sbi-mpxy-channel-id = <0x1000>;
> + };
Powered by blists - more mailing lists