[PATCH V3] Add support for generic BCM SoC chipsets

Stephen Warren swarren at wwwdotorg.org
Fri Nov 16 23:10:06 EST 2012


On 11/13/2012 02:51 PM, Christian Daudt wrote:
> In order to start upstreaming Broadcom SoC support, create
> a starting hierarchy, arch and dts files.
> The first support SoC family that is planned is the
> BCM281XX (BCM11130/11140/11351/28145/28155) family of dual A9 mobile
> SoC cores.
> This code is just the skeleton code for get the machine upstreamed. It
> has been made MULTIPLATFORM compatible.

Sorry for the slow review...

> diff --git a/Documentation/devicetree/bindings/arm/bcm/bcm281xx.txt b/Documentation/devicetree/bindings/arm/bcm/bcm281xx.txt

> +Required root node property:
> +
> +compatible = "bcm,bcm281xx";

Hmm. I still tend to think we should list the specific SoC rather than
the family here, but I guess if you're sure there won't be any SW
compatibility between the different SoCs in the series, this is fine.

> diff --git a/arch/arm/boot/dts/bcm281xx.dtsi b/arch/arm/boot/dts/bcm281xx.dtsi

> +	uart at 3e000000 {
> +		compatible = "snps,dw-apb-uart", "bcm,bcm281xx-sw-apb-uart";

The most specific value should be first, so swap those two entries.

> diff --git a/arch/arm/mach-bcm/Kconfig b/arch/arm/mach-bcm/Kconfig

> +	help
> +	  This enables support for system based on Broadcom SoCs.
> +	  It currently supports the 'BCM281XX' family, which includes
> +	  BCM11130, BCM1140, BCM11351, BCM28145 and

Should that be 11140 rather than 1140?

Aside from that,
Reviewed-by: Stephen Warren <swarren at wwwdotorg.org>



More information about the linux-arm-kernel mailing list