[U-Boot] [PATCH v2] disk: convert to Kconfig

[Tom Rini]

On Fri, Dec 02, 2016 at 05:23:16PM +0000, Patrick DELAUNAY wrote:
> Hi Tom
> 
> > 
> > > This converts the following to Kconfig:
> > >            CONFIG_PARTITIONS
> > >            CONFIG_MAC_PARTITION
> > >            CONFIG_DOS_PARTITION
> > >            CONFIG_ISO_PARTITION
> > >            CONFIG_AMIGA_PARTITION
> > >            CONFIG_EFI_PARTITION
> > >            CONFIG_PARTITION_UUIDS
> > >            CONFIG_PARTITION_TYPE_GUID
> > [snip]
> > >  606 files changed, 996 insertions(+), 583 deletions(-)
> > 
> > The insertions to deletions ratio is still high, and I think we can do a little
> > better.  Some of this is due to platforms that I fix with
> > https://patchwork.ozlabs.org/patch/700546/ and will have the various types
> > selected now.  But I think we can still do a little better with defaults too:
> 
> I agree that DISTRO_DEFAULT should reduce the update ratio....
> I wait the distro update and propose my patch on the top of  https://patchwork.ozlabs.org/patch/700546/ ?
> or I continue submission as it ?

I'm testing 700546 right now so please wait for me to push and merge
that and then do on top of it.

> > [snip]
> > > +config ISO_PARTITION
> > > +	bool "Enable ISO partition table"
> > > +	depends on PARTITIONS
> > 
> > This should be default y for x86, along with DOS and MAC.
> > 
> > > +config SPL_ISO_PARTITION
> > > +	bool "Enable ISO partition table for SPL"
> > > +	depends on SPL && PARTITIONS
> > 
> > Here and elsewhere, SPL_xxx_PARTITION should be default y if
> > xxx_PARTITION
> > 
> > > +config PARTITION_UUIDS
> > > +	bool "Enable support of UUID for partition"
> > > +	help
> > > +	  Activate the configuration of UUID for partition
> > 
> > I think a lot of these will go away once CMD_GPT and CMD_PART are
> > converted to Kconfig, but I don't want to chicken-and-egg this nor have too
> > many things in flight at once, so lets put that off until after this series.
> > Thanks!
> 
> Yes dependency need to be reviewed when all this command will moved to Kconfig
> 
> CMD_GPT => select EFI and PARTITION_UUIDS
> CMD_PART => select PARTITION_UUIDS
> 
> I will propose v3 with reduced ratio (I hope) but it is more complicated depends rules :
> 
> config MAC_PARTITION
> 	default y if PPC || x86 || SPARC || SH
> 
> config SPL_MAC_PARTITION
> 	default y if MAC_PARTITION
> 
> config DOS_PARTITION
> 	default y if PPC || x86 || CMD_FAT || USB_STORAGE
> 
> config SPL_DOS_PARTITION
> 	default y if DOS_PARTITION
> 
> config ISO_PARTITION
> 	default y if M68K || PPC || SPARC || MIPS
> 
> config SPL_ISO_PARTITION
> 	default y if ISO_PARTITION
> 
> config SPL_AMIGA_PARTITION
> 	default y if AMIGA_PARTITION
> 
> config SPL_EFI_PARTITION
> 	default y if EFI_PARTITION

Thanks!

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20161202/de80604c/attachment.sig>