All of lore.kernel.org
 help / color / mirror / Atom feed
From: Chin Liang See <clsee@altera.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH 1/6] arm: socfpga: cyclone5-socdk: Enabling mtd partitioning layout
Date: Mon, 14 Dec 2015 08:43:58 +0800	[thread overview]
Message-ID: <1450053838.1855.13.camel@altera.com> (raw)
In-Reply-To: <201512140122.29291.marex@denx.de>

On Mon, 2015-12-14 at 01:22 +0100, Marek Vasut wrote:
> On Monday, December 14, 2015 at 01:11:27 AM, Chin Liang See wrote:
> > On Sun, 2015-12-13 at 16:42 +0100, Marek Vasut wrote:
> > > On Sunday, December 13, 2015 at 02:03:02 PM, Chin Liang See
> > > wrote:
> > > > On Sun, 2015-12-13 at 04:14 +0100, Marek Vasut wrote:
> > > > > On Sunday, December 13, 2015 at 01:49:06 AM, Chin Liang See
> > > > > 
> > > > > wrote:
> > > > > > On Sun, 2015-12-13 at 01:01 +0100, Marek Vasut wrote:
> > > > > > > On Sunday, December 13, 2015 at 12:59:48 AM, Chin Liang
> > > > > > > See
> > > > > > > 
> > > > > > > wrote:
> > > > > > > > On Sat, 2015-12-12 at 16:36 +0100, Marek Vasut wrote:
> > > > > > > > > On Saturday, December 12, 2015 at 07:30:46 AM, Chin
> > > > > > > > > Liang
> > > > > > > > > See
> > > > > > > > 
> > > > > > > > > wrote:
> > > > > > > > [...]
> > > > > > > > 
> > > > > > > > > > > Can you share the final layout before you roll
> > > > > > > > > > > out
> > > > > > > > > > > patches ?
> > > > > > > > > > 
> > > > > > > > > > Sure, plan to do so but need to away from desk just
> > > > > > > > > > now.
> > > > > > > > > > 
> > > > > > > > > > Here is the old layout
> > > > > > > > > > 256k(spl)
> > > > > > > > > > 64k(env)
> > > > > > > > > > 64k(dtb)
> > > > > > > > > > 256k(boot)
> > > > > > > > > > 16m(kernel)
> > > > > > > > > > 16m(rootfs)
> > > > > > > > > > 
> > > > > > > > > > The new one would like this
> > > > > > > > > > 256k(spl)
> > > > > > > > > 
> > > > > > > > > I'd say you should just call this u-boot, see above
> > > > > > > > > for
> > > > > > > > > the
> > > > > > > > > rationale.
> > > > > > > > > 
> > > > > > > > > > 256k(env)
> > > > > > > > > > 15872k(boot)
> > > > > > > > > > 16m(rootfs)
> > > > > > > > > > 
> > > > > > > > > > The boot partition can be used as ubi part or raw
> > > > > > > > > > partition.
> > > > > > > > > > It contains the linux dtb, u-boot and linux images.
> > > > > > > > > 
> > > > > > > > > Is that an UBIFS partition ? If so, why don't you
> > > > > > > > > just
> > > > > > > > > use
> > > > > > > > > two
> > > > > > > > > UBI
> > > > > > > > > volumes ?
> > > > > > > > 
> > > > > > > > For backward compatibility, it can be raw if user want
> > > > > > > > to
> > > > > > > > stick
> > > > > > > > with
> > > > > > > > old way.
> > > > > > > 
> > > > > > > If you're breaking the partitioning layout anyway, you
> > > > > > > don't
> > > > > > > have
> > > > > > > to
> > > > > > > care about the "old way", right ?
> > > > > > 
> > > > > > Actually this partition can be used as raw partition if
> > > > > > user
> > > > > > don't
> > > > > > want
> > > > > > to store zimage and dtb as raw binary.
> > > > > 
> > > > > You should never store raw zImage/dtb in a flash on a
> > > > > production
> > > > > system.
> > > > > This is real bad and can result in a corruption in the future
> > > > > when
> > > > > the
> > > > > system is in the field for a long time. I'd suggest to just
> > > > > use
> > > > > two
> > > > > UBI
> > > > > volumes, one for fitImage and the other for rootfs.
> > > > 
> > > > Yup, we are trying to move the raw away by having UBI enabled.
> > > > I
> > > > presume when you refer 2 UBI volumes, I presume its still
> > > > single
> > > > UBI
> > > > partition?
> > > 
> > > Yeah, one UBI partition with two UBI volumes -- boot and root.
> > 
> > Ok I grasp you correctly. In this case, I would suggest to opt for
> > 2
> > partitions instead of 2 volumes. User can just use sf erase rootfs
> > instead of checking for volume offset. I suspect the ubi part will
> > take
> > longer time if we are having one partition only. I shall send out
> > the
> > new revision if this works for you :)
> 
> Oh, I see what you're concerned about. On the other hand, you are not
> supposed 
> to erase UBI partition, since that'd nuke the UBI erase counters and
> UBI would
> loose track of how many times each block was erased, which helps with
> wear 
> leveling.
> 
> If you're adamant about two UBI partitions , you can try the trick
> where you can
> create overlapping mtd partitions, that way you'd have both options
> available.
> (it is actually a legal construction, you can check the kernel MTD
> docs).

After grabbing a coffee and rethinking, we can have 2 MTD_PARTITION
where one for backward compatiblity (supporting raw) while another for
UBI. 

MTDPARTS_RAW
256k(spl),
256k(env),
15872k(boot),
-(rootfs)

MTDPARTS_UBI
256k(spl),
256k(env),
-(UBI)

We should make UBI as default then. If user care about backward compatiblity, they shall use the MTDPARTS_RAW then.

Thanks
Chin Liang

  reply	other threads:[~2015-12-14  0:43 UTC|newest]

Thread overview: 69+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-12-11  9:15 [U-Boot] [PATCH 1/6] arm: socfpga: cyclone5-socdk: Enabling mtd partitioning layout Chin Liang See
2015-12-11  9:15 ` [U-Boot] [PATCH 2/6] arm: socfpga: arria5-socdk: " Chin Liang See
2015-12-11  9:15 ` [U-Boot] [PATCH 3/6] arm: socfpga: de0-nano-soc: " Chin Liang See
2015-12-11 14:21   ` Marek Vasut
2015-12-11 15:43     ` Dinh Nguyen
2015-12-11 17:36       ` Marek Vasut
2015-12-12  0:01         ` Chin Liang See
2015-12-12 21:10   ` Pavel Machek
2015-12-12 21:45     ` Marek Vasut
2015-12-13  0:01       ` Chin Liang See
2015-12-13  0:04     ` Chin Liang See
2015-12-11  9:15 ` [U-Boot] [PATCH 4/6] arm: socfpga: mcvevk: " Chin Liang See
2015-12-11 14:21   ` Marek Vasut
2015-12-11  9:15 ` [U-Boot] [PATCH 5/6] arm: socfpga: sockit: " Chin Liang See
2015-12-11 14:23   ` Marek Vasut
2015-12-11  9:15 ` [U-Boot] [PATCH 6/6] arm: socfpga: socrates: " Chin Liang See
2015-12-11 14:23   ` Marek Vasut
2015-12-11 14:21 ` [U-Boot] [PATCH 1/6] arm: socfpga: cyclone5-socdk: " Marek Vasut
2015-12-11 23:59   ` Chin Liang See
2015-12-12  3:20     ` Marek Vasut
2015-12-12  6:30       ` Chin Liang See
2015-12-12 15:36         ` Marek Vasut
2015-12-12 23:59           ` Chin Liang See
2015-12-13  0:01             ` Marek Vasut
2015-12-13  0:49               ` Chin Liang See
2015-12-13  3:14                 ` Marek Vasut
2015-12-13 13:03                   ` Chin Liang See
2015-12-13 15:42                     ` Marek Vasut
2015-12-14  0:11                       ` Chin Liang See
2015-12-14  0:22                         ` Marek Vasut
2015-12-14  0:43                           ` Chin Liang See [this message]
2015-12-14  0:58                             ` Marek Vasut
2015-12-14  1:22                               ` Chin Liang See
2015-12-14  1:25                                 ` Marek Vasut
2015-12-14 15:22                                   ` Chin Liang See
2015-12-15  0:32                                     ` Marek Vasut
2015-12-15  1:09                                       ` Chin Liang See
2015-12-15  1:15                                         ` Marek Vasut
2015-12-15 10:09                                           ` Chin Liang See
2015-12-15 18:16                                             ` Marek Vasut
2015-12-18  9:39                                               ` Chin Liang See
2015-12-18 13:10                                                 ` Marek Vasut
2015-12-22 15:49                                                   ` Chin Liang See
2015-12-22 15:53                                                     ` Marek Vasut
2015-12-22 16:00                                                       ` Chin Liang See
2015-12-22 20:10                                                         ` Marek Vasut
2015-12-23  0:24                                                           ` Chin Liang See
2015-12-23  0:48                                                             ` Marek Vasut
2015-12-23  1:38                                                               ` Chin Liang See
2015-12-23 19:02                                                           ` Pavel Machek
2015-12-23 22:57                                                             ` Chin Liang See
2015-12-23 23:16                                                               ` Marek Vasut
2015-12-23 23:36                                                                 ` Chin Liang See
2015-12-24  1:30                                                                   ` Marek Vasut
2015-12-28  8:51                                                                     ` Chin Liang See
2015-12-28 12:39                                                                       ` Marek Vasut
2015-12-29 21:06                                                                         ` Chin Liang See
2015-12-29 21:20                                                                           ` Marek Vasut
2015-12-23 23:18                                                             ` Marek Vasut
2015-12-14  7:54                               ` Pavel Machek
2015-12-14 11:09                                 ` Marek Vasut
2015-12-14 11:26                                   ` Pavel Machek
2015-12-14 11:31                                     ` Marek Vasut
2015-12-14 11:51                                       ` Pavel Machek
2015-12-14 12:20                                         ` Marek Vasut
2015-12-14 12:53                                           ` Pavel Machek
2015-12-14 13:34                                             ` Marek Vasut
2015-12-14 15:31                                               ` Chin Liang See
2015-12-13  0:49               ` Chin Liang See

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1450053838.1855.13.camel@altera.com \
    --to=clsee@altera.com \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.