On 1/12/2021 11:27 AM, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@xxxxxxxxxx>
>
> PMB (Power Management Bus) controls powering connected devices (e.g.
> PCIe, USB, SATA). In BCM4908 it's a part of the PROCMON block.
>
> Signed-off-by: Rafał Miłecki <rafal@xxxxxxxxxx>
> ---
> Florian: this patch is based on top of the
> [PATCH] arm64: dts: broadcom: bcm4908: describe internal switch
> one. Both modify "ranges".
> ---
> .../boot/dts/broadcom/bcm4908/bcm4908.dtsi | 17 ++++++++++++++++-
> 1 file changed, 16 insertions(+), 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi
> index 0b44f32fdef1..882953885bfc 100644
> --- a/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi
> +++ b/arch/arm64/boot/dts/broadcom/bcm4908/bcm4908.dtsi
> @@ -108,7 +108,7 @@ soc {
> compatible = "simple-bus";
> #address-cells = <1>;
> #size-cells = <1>;
> - ranges = <0x00 0x00 0x80000000 0xd0000>;
> + ranges = <0x00 0x00 0x80000000 0x281000>;
>
> usb@c300 {
> compatible = "generic-ehci";
> @@ -213,6 +213,21 @@ phy12: phy@c {
> };
> };
> };
> +
> + procmon: syscon@280000 {
> + compatible = "simple-mfd";
Likewise, I would use "simple-bus" here instead of "simple-mfd" even if
this does not create a functional difference.
--
Florian
