mbox series

[SRU,focal/linux-raspi,0/2] LP: #1875454 - dtoverlay=uart4 breaks Raspberry Pi 4B boot

Message ID 20200624161035.25871-1-juergh@canonical.com
Headers show
Series LP: #1875454 - dtoverlay=uart4 breaks Raspberry Pi 4B boot | expand

Message

Juerg Haefliger June 24, 2020, 4:10 p.m. UTC 
U-boot still uses the old compatible "brcm,bcm2835-pl011" for the UARTs. The Pi
4B has additional UARTs 2-5 which need this compatible as well otherwise u-boot
won't load with something like 'dtoverlay=uart4'.

...Juerg


Juerg Haefliger (2):
  UBUNTU: SAUCE: arm: dts: Clarify the need for "brcm,bcm2835-<foo>"
  UBUNTU: SAUCE: arm: dts: Add 'brcm,bcm2835-pl011' for compatible for
    uart2-5

 arch/arm/boot/dts/bcm2711.dtsi | 10 +++++-----
 arch/arm/boot/dts/bcm283x.dtsi |  2 +-
 2 files changed, 6 insertions(+), 6 deletions(-)

Comments

Ian May June 24, 2020, 8:28 p.m. UTC | #1 
On 6/24/20 11:10 AM, Juerg Haefliger wrote:
> U-boot still uses the old compatible "brcm,bcm2835-pl011" for the UARTs. The Pi
> 4B has additional UARTs 2-5 which need this compatible as well otherwise u-boot
> won't load with something like 'dtoverlay=uart4'.
>
> ...Juerg
>
>
> Juerg Haefliger (2):
>   UBUNTU: SAUCE: arm: dts: Clarify the need for "brcm,bcm2835-<foo>"
>   UBUNTU: SAUCE: arm: dts: Add 'brcm,bcm2835-pl011' for compatible for
>     uart2-5
>
>  arch/arm/boot/dts/bcm2711.dtsi | 10 +++++-----
>  arch/arm/boot/dts/bcm283x.dtsi |  2 +-
>  2 files changed, 6 insertions(+), 6 deletions(-)
>
LGTM

Acked-by: Ian May <ian.may@canonical.com>
Kelsey Skunberg June 25, 2020, 7:11 a.m. UTC | #2 
On 2020-06-24 18:10:33 , Juerg Haefliger wrote:
> U-boot still uses the old compatible "brcm,bcm2835-pl011" for the UARTs. The Pi
> 4B has additional UARTs 2-5 which need this compatible as well otherwise u-boot
> won't load with something like 'dtoverlay=uart4'.
> 
> ...Juerg
> 
> 
> Juerg Haefliger (2):
>   UBUNTU: SAUCE: arm: dts: Clarify the need for "brcm,bcm2835-<foo>"
>   UBUNTU: SAUCE: arm: dts: Add 'brcm,bcm2835-pl011' for compatible for
>     uart2-5
> 
>  arch/arm/boot/dts/bcm2711.dtsi | 10 +++++-----
>  arch/arm/boot/dts/bcm283x.dtsi |  2 +-
>  2 files changed, 6 insertions(+), 6 deletions(-)
> 
> -- 
> 2.20.1
> 

Acked-by: Kelsey Skunberg <kelsey.skunberg@canonical.com>

> 
> -- 
> kernel-team mailing list
> kernel-team@lists.ubuntu.com
> https://lists.ubuntu.com/mailman/listinfo/kernel-team
Kleber Sacilotto de Souza July 3, 2020, 3:09 p.m. UTC | #3 
On 2020-06-24 18:10, Juerg Haefliger wrote:
> U-boot still uses the old compatible "brcm,bcm2835-pl011" for the UARTs. The Pi
> 4B has additional UARTs 2-5 which need this compatible as well otherwise u-boot
> won't load with something like 'dtoverlay=uart4'.
> 
> ...Juerg
> 
> 
> Juerg Haefliger (2):
>   UBUNTU: SAUCE: arm: dts: Clarify the need for "brcm,bcm2835-<foo>"
>   UBUNTU: SAUCE: arm: dts: Add 'brcm,bcm2835-pl011' for compatible for
>     uart2-5
> 
>  arch/arm/boot/dts/bcm2711.dtsi | 10 +++++-----
>  arch/arm/boot/dts/bcm283x.dtsi |  2 +-
>  2 files changed, 6 insertions(+), 6 deletions(-)
> 

Applied to focal/linux-raspi.

Thanks,
Kleber