Re: [PATCH 2/2] spl: get rid of SPL_LIBDISK_SUPPORT

2020-02-02 Thread Simon Glass
On Sat, 1 Feb 2020 at 12:38, Thomas Hebb  wrote:
>
> This option hasn't actually affected what's linked into the build since
> commit 91ff6865629c ("blk: Rework guard around part_init call"), which
> switched libdisk in SPL to depend on (CONFIG_SPL_FRAMEWORK &&
> CONFIG_PARTITIONS). After removing one straggling reference that seems
> to been authored before 91ff6865629c landed, there are absolutely no
> references to this in the code. Let's remove it.
>
> Signed-off-by: Thomas Hebb 
>
> ---
>
>  arch/arm/Kconfig |  1 -
>  arch/arm/mach-imx/mx6/Kconfig|  1 -
>  arch/arm/mach-mvebu/Kconfig  |  2 --
>  arch/arm/mach-omap2/Kconfig  |  3 ---
>  arch/arm/mach-omap2/am33xx/Kconfig   |  2 --
>  arch/arm/mach-stm32mp/Kconfig|  1 -
>  arch/arm/mach-zynq/Kconfig   |  3 ---
>  arch/arm/mach-zynqmp/Kconfig |  3 ---
>  common/spl/Kconfig   | 19 ++-
>  configs/am335x_baltos_defconfig  |  1 -
>  configs/am335x_guardian_defconfig|  1 -
>  configs/am335x_hs_evm_uart_defconfig |  1 -
>  configs/am335x_igep003x_defconfig|  1 -
>  configs/am335x_pdu001_defconfig  |  1 -
>  configs/am335x_shc_defconfig |  1 -
>  configs/am335x_shc_ict_defconfig |  1 -
>  configs/am335x_shc_netboot_defconfig |  1 -
>  configs/am335x_shc_sdboot_defconfig  |  1 -
>  configs/am335x_sl50_defconfig|  1 -
>  configs/am65x_evm_a53_defconfig  |  1 -
>  configs/am65x_evm_r5_defconfig   |  1 -
>  configs/am65x_hs_evm_a53_defconfig   |  1 -
>  configs/am65x_hs_evm_r5_defconfig|  1 -
>  configs/birdland_bav335a_defconfig   |  1 -
>  configs/birdland_bav335b_defconfig   |  1 -
>  configs/cgtqmx6eval_defconfig|  1 -
>  configs/chiliboard_defconfig |  1 -
>  configs/cm_t335_defconfig|  1 -
>  configs/cm_t43_defconfig |  1 -
>  configs/draco_defconfig  |  1 -
>  configs/etamin_defconfig |  1 -
>  configs/imx6qdl_icore_mipi_defconfig |  1 -
>  configs/imx6qdl_icore_mmc_defconfig  |  1 -
>  configs/imx6qdl_icore_rqs_defconfig  |  1 -
>  configs/imx6ul_geam_mmc_defconfig|  1 -
>  configs/imx6ul_isiot_emmc_defconfig  |  1 -
>  configs/j721e_evm_a72_defconfig  |  1 -
>  configs/j721e_evm_r5_defconfig   |  1 -
>  configs/j721e_hs_evm_a72_defconfig   |  1 -
>  configs/j721e_hs_evm_r5_defconfig|  1 -
>  configs/mx6cuboxi_defconfig  |  1 -
>  configs/mx6sabreauto_defconfig   |  1 -
>  configs/mx6sabresd_defconfig |  1 -
>  configs/mx6slevk_spl_defconfig   |  1 -
>  configs/mx6sxsabresd_spl_defconfig   |  1 -
>  configs/mx6ul_14x14_evk_defconfig|  1 -
>  configs/mx6ul_9x9_evk_defconfig  |  1 -
>  configs/novena_defconfig |  1 -
>  configs/opos6uldev_defconfig |  1 -
>  configs/pcm051_rev1_defconfig|  1 -
>  configs/pcm051_rev3_defconfig|  1 -
>  configs/pcm058_defconfig |  1 -
>  configs/pengwyn_defconfig|  1 -
>  configs/pepper_defconfig |  1 -
>  configs/pfla02_defconfig |  1 -
>  configs/phycore-am335x-r2-wega_defconfig |  1 -
>  configs/pico-dwarf-imx6ul_defconfig  |  1 -
>  configs/pico-hobbit-imx6ul_defconfig |  1 -
>  configs/pico-imx6_defconfig  |  1 -
>  configs/pico-imx6ul_defconfig|  1 -
>  configs/pico-pi-imx6ul_defconfig |  1 -
>  configs/picosam9g45_defconfig|  1 -
>  configs/platinum_picon_defconfig |  1 -
>  configs/platinum_titanium_defconfig  |  1 -
>  configs/pxm2_defconfig   |  1 -
>  configs/rastaban_defconfig   |  1 -
>  configs/riotboard_spl_defconfig  |  1 -
>  configs/rut_defconfig|  1 -
>  configs/sama5d27_som1_ek_mmc1_defconfig  |  1 -
>  configs/sama5d27_som1_ek_mmc_defconfig   |  1 -
>  configs/sama5d27_som1_ek_qspiflash_defconfig |  1 -
>  configs/sama5d27_wlsom1_ek_mmc_defconfig |  1 -
>  configs/sama5d2_icp_mmc_defconfig|  1 -
>  configs/sama5d2_xplained_emmc_defconfig  |  1 -
>  configs/sama5d2_xplained_mmc_defconfig   |  1 -
>  configs/sama5d2_xplained_qspiflash_defconfig |  1 -
>  configs/sama5d3_xplained_mmc_defconfig   |  1 -
>  configs/sama5d3xek_mmc_defconfig |  1 -
>  configs/sama5d4_xplained_mmc_defconfig   |  1 -
>  configs/sama5d4ek_mmc_defconfig  |  1 -
>  configs/sksimx6_defconfig|  1 -
>  configs/thuban_d

Re: [PATCH 2/2] spl: get rid of SPL_LIBDISK_SUPPORT

2020-02-02 Thread Felix Brack

On 01.02.20 20:38, Thomas Hebb wrote:

This option hasn't actually affected what's linked into the build since
commit 91ff6865629c ("blk: Rework guard around part_init call"), which
switched libdisk in SPL to depend on (CONFIG_SPL_FRAMEWORK &&
CONFIG_PARTITIONS). After removing one straggling reference that seems
to been authored before 91ff6865629c landed, there are absolutely no
references to this in the code. Let's remove it.

Signed-off-by: Thomas Hebb 

---



For the PDU001 board:
Tested-by: Felix Brack 


[PATCH 2/2] spl: get rid of SPL_LIBDISK_SUPPORT

2020-02-02 Thread Thomas Hebb
This option hasn't actually affected what's linked into the build since
commit 91ff6865629c ("blk: Rework guard around part_init call"), which
switched libdisk in SPL to depend on (CONFIG_SPL_FRAMEWORK &&
CONFIG_PARTITIONS). After removing one straggling reference that seems
to been authored before 91ff6865629c landed, there are absolutely no
references to this in the code. Let's remove it.

Signed-off-by: Thomas Hebb 

---

 arch/arm/Kconfig |  1 -
 arch/arm/mach-imx/mx6/Kconfig|  1 -
 arch/arm/mach-mvebu/Kconfig  |  2 --
 arch/arm/mach-omap2/Kconfig  |  3 ---
 arch/arm/mach-omap2/am33xx/Kconfig   |  2 --
 arch/arm/mach-stm32mp/Kconfig|  1 -
 arch/arm/mach-zynq/Kconfig   |  3 ---
 arch/arm/mach-zynqmp/Kconfig |  3 ---
 common/spl/Kconfig   | 19 ++-
 configs/am335x_baltos_defconfig  |  1 -
 configs/am335x_guardian_defconfig|  1 -
 configs/am335x_hs_evm_uart_defconfig |  1 -
 configs/am335x_igep003x_defconfig|  1 -
 configs/am335x_pdu001_defconfig  |  1 -
 configs/am335x_shc_defconfig |  1 -
 configs/am335x_shc_ict_defconfig |  1 -
 configs/am335x_shc_netboot_defconfig |  1 -
 configs/am335x_shc_sdboot_defconfig  |  1 -
 configs/am335x_sl50_defconfig|  1 -
 configs/am65x_evm_a53_defconfig  |  1 -
 configs/am65x_evm_r5_defconfig   |  1 -
 configs/am65x_hs_evm_a53_defconfig   |  1 -
 configs/am65x_hs_evm_r5_defconfig|  1 -
 configs/birdland_bav335a_defconfig   |  1 -
 configs/birdland_bav335b_defconfig   |  1 -
 configs/cgtqmx6eval_defconfig|  1 -
 configs/chiliboard_defconfig |  1 -
 configs/cm_t335_defconfig|  1 -
 configs/cm_t43_defconfig |  1 -
 configs/draco_defconfig  |  1 -
 configs/etamin_defconfig |  1 -
 configs/imx6qdl_icore_mipi_defconfig |  1 -
 configs/imx6qdl_icore_mmc_defconfig  |  1 -
 configs/imx6qdl_icore_rqs_defconfig  |  1 -
 configs/imx6ul_geam_mmc_defconfig|  1 -
 configs/imx6ul_isiot_emmc_defconfig  |  1 -
 configs/j721e_evm_a72_defconfig  |  1 -
 configs/j721e_evm_r5_defconfig   |  1 -
 configs/j721e_hs_evm_a72_defconfig   |  1 -
 configs/j721e_hs_evm_r5_defconfig|  1 -
 configs/mx6cuboxi_defconfig  |  1 -
 configs/mx6sabreauto_defconfig   |  1 -
 configs/mx6sabresd_defconfig |  1 -
 configs/mx6slevk_spl_defconfig   |  1 -
 configs/mx6sxsabresd_spl_defconfig   |  1 -
 configs/mx6ul_14x14_evk_defconfig|  1 -
 configs/mx6ul_9x9_evk_defconfig  |  1 -
 configs/novena_defconfig |  1 -
 configs/opos6uldev_defconfig |  1 -
 configs/pcm051_rev1_defconfig|  1 -
 configs/pcm051_rev3_defconfig|  1 -
 configs/pcm058_defconfig |  1 -
 configs/pengwyn_defconfig|  1 -
 configs/pepper_defconfig |  1 -
 configs/pfla02_defconfig |  1 -
 configs/phycore-am335x-r2-wega_defconfig |  1 -
 configs/pico-dwarf-imx6ul_defconfig  |  1 -
 configs/pico-hobbit-imx6ul_defconfig |  1 -
 configs/pico-imx6_defconfig  |  1 -
 configs/pico-imx6ul_defconfig|  1 -
 configs/pico-pi-imx6ul_defconfig |  1 -
 configs/picosam9g45_defconfig|  1 -
 configs/platinum_picon_defconfig |  1 -
 configs/platinum_titanium_defconfig  |  1 -
 configs/pxm2_defconfig   |  1 -
 configs/rastaban_defconfig   |  1 -
 configs/riotboard_spl_defconfig  |  1 -
 configs/rut_defconfig|  1 -
 configs/sama5d27_som1_ek_mmc1_defconfig  |  1 -
 configs/sama5d27_som1_ek_mmc_defconfig   |  1 -
 configs/sama5d27_som1_ek_qspiflash_defconfig |  1 -
 configs/sama5d27_wlsom1_ek_mmc_defconfig |  1 -
 configs/sama5d2_icp_mmc_defconfig|  1 -
 configs/sama5d2_xplained_emmc_defconfig  |  1 -
 configs/sama5d2_xplained_mmc_defconfig   |  1 -
 configs/sama5d2_xplained_qspiflash_defconfig |  1 -
 configs/sama5d3_xplained_mmc_defconfig   |  1 -
 configs/sama5d3xek_mmc_defconfig |  1 -
 configs/sama5d4_xplained_mmc_defconfig   |  1 -
 configs/sama5d4ek_mmc_defconfig  |  1 -
 configs/sksimx6_defconfig|  1 -
 configs/thuban_defconfig |  1 -
 configs/ti814x_evm_defconfig |  1 -
 configs/ti816x_evm_defconfig |  1 -
 configs/udoo_defconfig   |  1 -
 configs/udoo_neo_defconfig