[PATCH] arm64: dts: broadcom: bcm4908: describe PMB block

[Florian Fainelli]

On 1/12/2021 11:27 AM, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal at milecki.pl>
> 
> 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 at milecki.pl>
> ---
> 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 at c300 {
>  			compatible = "generic-ehci";
> @@ -213,6 +213,21 @@ phy12: phy at c {
>  				};
>  			};
>  		};
> +
> +		procmon: syscon at 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