mbox series

[v2,0/9] Fixes for SPI boot on SanCloud BBE Lite

Message ID 20221114124243.3719037-1-paul.barker@sancloud.com
Headers show
Series Fixes for SPI boot on SanCloud BBE Lite | expand

Message

Paul Barker Nov. 14, 2022, 12:42 p.m. UTC
These changes fix SPI boot on the SanCloud BBE Lite and other AM335x
boards.

They also put us in a good place to enable SPL_OF_CONTROL for eMMC/SD
card boot on AM335x boards. Note that if SPL_OF_CONTROL is enabled, some
SPL features need to be dropped due to the limited amount of sram
available. In my testing I dropped NAND, NET & USB support.

This series has been build tested for all targets using buildman and no
errors were observed. It has been boot tested on the SanCloud BBE Lite
with the following configurations:

    1) SD card boot using am335x_evm_defconfig with no config changes.

    2) SD card boot using am335x_evm_defconfig with the following config
       changes:

        CONFIG_SPL_OF_CONTROL=y
        CONFIG_SPL_OF_TRANSLATE=y
        CONFIG_SPL_MUSB_NEW=n
        CONFIG_SPL_NAND_DRIVERS=n
        CONFIG_SPL_NAND_ECC=n
        CONFIG_SPL_NAND_BASE=n
        CONFIG_SPL_NET=n
        CONFIG_SPL_USB_GADGET=n

    3) SPI boot using am335x_evm_spiboot_defconfig with the following
       config changes:

        CONFIG_DEFAULT_DEVICE_TREE="am335x-sancloud-bbe-lite"
        CONFIG_OF_LIST="am335x-sancloud-bbe-lite"

As discussed in my previous message [1], I can't see how to construct a
test case to accompany the first patch which fixes an SPL crash during
the pre-relocation lists_bind_drivers() call due to exhaustion of the
available memory. We can discuss this further though if anyone has any
ideas on how to write an appropriate test case.

[1]: https://lore.kernel.org/u-boot/d8182df9-1a09-81b9-dfbe-ebca4bad432f@sancloud.com/

Paul Barker (9):
  dm: core: Fix iteration over driver_info records
  bus: TI sysc driver requires DM
  bus: Optionally include TI sysc driver in SPL/TPL
  am335x-evm: Enable required dtb nodes in SPL
  am335x-evm: Fix spiboot configuration
  am335x-evm: Support STMicro/Micron SPI flash
  am335x-sancloud-bbe-lite: SPI flash is JEDEC compatible
  am335x-sancloud-bbe: Add -u-boot.dtsi files
  MAINTAINERS: Adopt SanCloud boards

 MAINTAINERS                                   |  6 +++
 arch/arm/dts/am335x-evm-u-boot.dtsi           | 30 ++++++++++++-
 .../dts/am335x-sancloud-bbe-lite-u-boot.dtsi  | 44 +++++++++++++++++++
 arch/arm/dts/am335x-sancloud-bbe-lite.dts     |  2 +-
 arch/arm/dts/am335x-sancloud-bbe-u-boot.dtsi  |  6 +++
 configs/am335x_evm_defconfig                  |  1 +
 configs/am335x_evm_spiboot_defconfig          |  5 +++
 drivers/Makefile                              |  2 +-
 drivers/bus/Kconfig                           |  9 +++-
 drivers/bus/Makefile                          |  5 ++-
 drivers/core/lists.c                          |  6 +--
 11 files changed, 107 insertions(+), 9 deletions(-)
 create mode 100644 arch/arm/dts/am335x-sancloud-bbe-lite-u-boot.dtsi
 create mode 100644 arch/arm/dts/am335x-sancloud-bbe-u-boot.dtsi


base-commit: 0cbeed4f6648e0e4966475e3544280a69ecb59d3

Comments

Paul Barker Dec. 1, 2022, 9:18 a.m. UTC | #1
On 14/11/2022 12:42, Paul Barker wrote:
> These changes fix SPI boot on the SanCloud BBE Lite and other AM335x
> boards.
> 
> They also put us in a good place to enable SPL_OF_CONTROL for eMMC/SD
> card boot on AM335x boards. Note that if SPL_OF_CONTROL is enabled, some
> SPL features need to be dropped due to the limited amount of sram
> available. In my testing I dropped NAND, NET & USB support.
> 
> This series has been build tested for all targets using buildman and no
> errors were observed. It has been boot tested on the SanCloud BBE Lite
> with the following configurations:
> 
>     1) SD card boot using am335x_evm_defconfig with no config changes.
> 
>     2) SD card boot using am335x_evm_defconfig with the following config
>        changes:
> 
>         CONFIG_SPL_OF_CONTROL=y
>         CONFIG_SPL_OF_TRANSLATE=y
>         CONFIG_SPL_MUSB_NEW=n
>         CONFIG_SPL_NAND_DRIVERS=n
>         CONFIG_SPL_NAND_ECC=n
>         CONFIG_SPL_NAND_BASE=n
>         CONFIG_SPL_NET=n
>         CONFIG_SPL_USB_GADGET=n
> 
>     3) SPI boot using am335x_evm_spiboot_defconfig with the following
>        config changes:
> 
>         CONFIG_DEFAULT_DEVICE_TREE="am335x-sancloud-bbe-lite"
>         CONFIG_OF_LIST="am335x-sancloud-bbe-lite"
> 
> As discussed in my previous message [1], I can't see how to construct a
> test case to accompany the first patch which fixes an SPL crash during
> the pre-relocation lists_bind_drivers() call due to exhaustion of the
> available memory. We can discuss this further though if anyone has any
> ideas on how to write an appropriate test case.
> 
> [1]: https://lore.kernel.org/u-boot/d8182df9-1a09-81b9-dfbe-ebca4bad432f@sancloud.com/
> 
> Paul Barker (9):
>   dm: core: Fix iteration over driver_info records
>   bus: TI sysc driver requires DM
>   bus: Optionally include TI sysc driver in SPL/TPL
>   am335x-evm: Enable required dtb nodes in SPL
>   am335x-evm: Fix spiboot configuration
>   am335x-evm: Support STMicro/Micron SPI flash
>   am335x-sancloud-bbe-lite: SPI flash is JEDEC compatible
>   am335x-sancloud-bbe: Add -u-boot.dtsi files
>   MAINTAINERS: Adopt SanCloud boards
> 
>  MAINTAINERS                                   |  6 +++
>  arch/arm/dts/am335x-evm-u-boot.dtsi           | 30 ++++++++++++-
>  .../dts/am335x-sancloud-bbe-lite-u-boot.dtsi  | 44 +++++++++++++++++++
>  arch/arm/dts/am335x-sancloud-bbe-lite.dts     |  2 +-
>  arch/arm/dts/am335x-sancloud-bbe-u-boot.dtsi  |  6 +++
>  configs/am335x_evm_defconfig                  |  1 +
>  configs/am335x_evm_spiboot_defconfig          |  5 +++
>  drivers/Makefile                              |  2 +-
>  drivers/bus/Kconfig                           |  9 +++-
>  drivers/bus/Makefile                          |  5 ++-
>  drivers/core/lists.c                          |  6 +--
>  11 files changed, 107 insertions(+), 9 deletions(-)
>  create mode 100644 arch/arm/dts/am335x-sancloud-bbe-lite-u-boot.dtsi
>  create mode 100644 arch/arm/dts/am335x-sancloud-bbe-u-boot.dtsi
> 
> 
> base-commit: 0cbeed4f6648e0e4966475e3544280a69ecb59d3

A gentle ping on this patch series.