mbox series

[0/5] Complete the migration of MTDPARTS_DEFAULT / MTDIDS_DEFAULT

Message ID 20221207082644.3515486-1-patrick.delaunay@foss.st.com
Headers show
Series Complete the migration of MTDPARTS_DEFAULT / MTDIDS_DEFAULT | expand

Message

Patrick Delaunay Dec. 7, 2022, 8:26 a.m. UTC 
Addition for previous commit a331017c237c ("Complete migration of
MTDPARTS_DEFAULT / MTDIDS_DEFAULT, include in environment")

Remove the remaining defines MTDPARTS_DEFAULT and MTDIDS_DEFAULT
in the configuration files (include/configs/*.h).

After this serie, the only remaining references of these 2 defines are
located in cmd/mtdparts.c and only for local purpose when
CONFIG_MTDIDS_DEFAULT or CONFIG_MTDPART_DEFAULT are not defined.

Patrick.


Patrick Delaunay (5):
  configs: am333x_guardian: move MTDIDS_DEFAULT in defconfif
  configs: x530: move MTDPART/MTDIDS_DEFAULT in defconfig
  configs: SBx81LIFXCAT: move MTDPART_DEFAULT in defconfig
  configs: SBx81LIFKW: move MTDPART_DEFAULT in defconfig
  configs: remove support of MTDIDS_DEFAULT/MTDPARTS_DEFAULT

 cmd/mtdparts.c                    | 5 +----
 configs/SBx81LIFKW_defconfig      | 1 +
 configs/SBx81LIFXCAT_defconfig    | 1 +
 configs/am335x_guardian_defconfig | 1 +
 configs/x530_defconfig            | 2 ++
 drivers/mtd/mtd_uboot.c           | 4 ----
 include/configs/SBx81LIFKW.h      | 1 -
 include/configs/SBx81LIFXCAT.h    | 1 -
 include/configs/am335x_guardian.h | 1 -
 include/configs/x530.h            | 2 --
 10 files changed, 6 insertions(+), 13 deletions(-)

Comments

Tom Rini Dec. 7, 2022, 2:40 p.m. UTC | #1 
On Wed, Dec 07, 2022 at 09:26:39AM +0100, Patrick Delaunay wrote:

> Addition for previous commit a331017c237c ("Complete migration of
> MTDPARTS_DEFAULT / MTDIDS_DEFAULT, include in environment")
> 
> Remove the remaining defines MTDPARTS_DEFAULT and MTDIDS_DEFAULT
> in the configuration files (include/configs/*.h).
> 
> After this serie, the only remaining references of these 2 defines are
> located in cmd/mtdparts.c and only for local purpose when
> CONFIG_MTDIDS_DEFAULT or CONFIG_MTDPART_DEFAULT are not defined.

Thanks for doing this! I suspect this means I can follow-up and remove
the defaults from cmd/mtparts.c later.
Chris Packham Dec. 11, 2022, 9:04 p.m. UTC | #2 
On 7/12/22 21:26, Patrick Delaunay wrote:
> Addition for previous commit a331017c237c ("Complete migration of
> MTDPARTS_DEFAULT / MTDIDS_DEFAULT, include in environment")
>
> Remove the remaining defines MTDPARTS_DEFAULT and MTDIDS_DEFAULT
> in the configuration files (include/configs/*.h).
>
> After this serie, the only remaining references of these 2 defines are
> located in cmd/mtdparts.c and only for local purpose when
> CONFIG_MTDIDS_DEFAULT or CONFIG_MTDPART_DEFAULT are not defined.
>
> Patrick.
>
>
> Patrick Delaunay (5):
>    configs: am333x_guardian: move MTDIDS_DEFAULT in defconfif
>    configs: x530: move MTDPART/MTDIDS_DEFAULT in defconfig
>    configs: SBx81LIFXCAT: move MTDPART_DEFAULT in defconfig
>    configs: SBx81LIFKW: move MTDPART_DEFAULT in defconfig
>    configs: remove support of MTDIDS_DEFAULT/MTDPARTS_DEFAULT

For x530, SBx81LIFXCAT and SBx81LIFKW

Reviewed-by: Chris Packham <chris.packham@alliedtelesis.co.nz>

>   cmd/mtdparts.c                    | 5 +----
>   configs/SBx81LIFKW_defconfig      | 1 +
>   configs/SBx81LIFXCAT_defconfig    | 1 +
>   configs/am335x_guardian_defconfig | 1 +
>   configs/x530_defconfig            | 2 ++
>   drivers/mtd/mtd_uboot.c           | 4 ----
>   include/configs/SBx81LIFKW.h      | 1 -
>   include/configs/SBx81LIFXCAT.h    | 1 -
>   include/configs/am335x_guardian.h | 1 -
>   include/configs/x530.h            | 2 --
>   10 files changed, 6 insertions(+), 13 deletions(-)
>