[U-Boot] [PATCH 3/4] sunxi: Add default partition scheme

[Maxime Ripard]

On Thu, Nov 16, 2017 at 12:30:04PM -0500, Tom Rini wrote:
> On Thu, Nov 16, 2017 at 11:41:57AM +0000, Andre Przywara wrote:
> > Hi,
> > 
> > On 16/11/17 11:21, Maxime Ripard wrote:
> > > On Thu, Nov 16, 2017 at 10:30:38AM +0000, Andre Przywara wrote:
> > >> Hi,
> > >>
> > >> On 15/11/17 21:03, Alexander Graf wrote:
> > >>>
> > >>>
> > >>> On 15.11.17 11:11, Maxime Ripard wrote:
> > >>>> The partitions variable is especially useful to create a partition table
> > >>>> from U-Boot, either directly from the U-Boot shell, or through flashing
> > >>>> tools like fastboot and its oem format command.
> > >>>>
> > >>>> This is especially useful on devices with an eMMC you can't take out to
> > >>>> flash from another system, and booting a Linux system first to flash our
> > >>>> system then is not really practical.
> > >>>>
> > >>>> Signed-off-by: Maxime Ripard <maxime.ripard at free-electrons.com>
> > >>>> ---
> > >>>>  include/configs/sunxi-common.h | 7 +++++++
> > >>>>  1 file changed, 7 insertions(+)
> > >>>>
> > >>>> diff --git a/include/configs/sunxi-common.h b/include/configs/sunxi-common.h
> > >>>> index 4391a8cbc824..11da6ccfbf54 100644
> > >>>> --- a/include/configs/sunxi-common.h
> > >>>> +++ b/include/configs/sunxi-common.h
> > >>>> @@ -493,6 +493,12 @@ extern int soft_i2c_gpio_scl;
> > >>>>  #define SUNXI_MTDPARTS_DEFAULT
> > >>>>  #endif
> > >>>>  
> > >>>> +#define PARTS_DEFAULT \
> > >>>> +	"name=loader1,start=8k,size=32k;" \
> > >>>> +	"name=loader2,size=984k;" \
> > >>>> +	"name=boot,size=128M,bootable;" \
> > >>>> +	"name=system,size=-;"
> > >>>
> > >>> Is there a particular reason you're creating a boot and system
> > >>> partition? In a normal distro world, the distro installer will take care
> > >>> of creating ESP + root + swap + whatever for you - and they (or the user
> > >>> driving the installation) usually know best what they need :)
> 
> So, from another part of this thread, yes, this should not be called
> boot but be called esp so it's clearer.

That works for me.

Thanks!
Maxime

-- 
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 801 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20171117/3fb9a60a/attachment.sig>