From: Bo Shen <voice.shen@atmel.com>
To: Sascha Hauer <s.hauer@pengutronix.de>,
Barebox List <barebox@lists.infradead.org>
Subject: Re: mtd partition handling updates
Date: Fri, 13 Feb 2015 13:54:33 +0800 [thread overview]
Message-ID: <54DD9199.1090403@atmel.com> (raw)
In-Reply-To: <1423731274-9860-1-git-send-email-s.hauer@pengutronix.de>
Hi Sascha,
On 02/12/2015 04:54 PM, Sascha Hauer wrote:
> This series updates the mtd partition handling in barebox. It used
> to be too simple to create conflicting or inconsistent partitions on
> mtd devices. mtd partitions can be changed from board code, from the
> device tree, using addpart/delpart and with this series also with
> the newly created 'partitions' parameter for mtd devices. The
> informations are now kept in a consistent state, with each changed
> partition the partitions parameter is updated and when starting
> the kernel partitions in the device tree are also updated to the
> current partitioning.
> What's still missing is a way to store the partitioning somewhere
> in the environment, perhaps in a nonvolatile variable.
>
> Sascha
>
>
> The following changes since commit d7dc46d5d6f2c80d1719ef877df728457790ac45:
>
> defaultenv: ignore barebox_zero_env (2015-02-11 08:34:29 +0100)
>
> are available in the git repository at:
>
> git://git.pengutronix.de/git/barebox.git
>
> for you to fetch changes up to cd7feae00b43eeddcaeb229f861d0ddac93f91e2:
>
> defaultenv-2: mtdparts-add: Use new partition parameter (2015-02-12 09:33:44 +0100)
>
> ----------------------------------------------------------------
> Sascha Hauer (17):
> of: Add for_each_child_of_node_safe
> mtd: core: add error checks
> mtd: partitions: Use xstrdup
> mtd: partitions: Add error check
> mtd: Add partitions to list
> mtd: nand: remove automatically created bb devices
> move cmdline partition parsing code to separate file
> cmdlinepart: Change SIZE_REMAINING to loff_t
> cmdlinepart: make argument types safer
> cmdlinepart: add function to parse a cmdline partition string
> cmndlinepart: skip devname if partstr already contains it
> mtd: forbid conflicting mtd partitions
> mtd: Use flags parameter in mtd_add_partition
> mtd: Add a partitions parameter to mtd devices
> mtd: fixup device tree partitions
> defaultenv-2: mtdparts-add: remove unused variable
> defaultenv-2: mtdparts-add: Use new partition parameter
For the whole series, test OK with Doug's patch on sama5d4ek board.
Tested-by: Bo Shen <voice.shen@atmel.com>
> commands/partition.c | 104 +------------
> defaultenv/defaultenv-2-base/bin/mtdparts-add | 15 +-
> drivers/mtd/core.c | 213 +++++++++++++++++++++++++-
> drivers/mtd/nand/nand-bb.c | 16 +-
> drivers/mtd/partition.c | 14 +-
> include/cmdlinepart.h | 14 ++
> include/linux/mtd/mtd.h | 5 +
> include/nand.h | 7 +-
> include/of.h | 2 +
> lib/Makefile | 1 +
> lib/cmdlinepart.c | 124 +++++++++++++++
> 11 files changed, 393 insertions(+), 122 deletions(-)
> create mode 100644 include/cmdlinepart.h
> create mode 100644 lib/cmdlinepart.c
Best Regards,
Bo Shen
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
prev parent reply other threads:[~2015-02-13 6:01 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-12 8:54 mtd partition handling updates Sascha Hauer
2015-02-12 8:54 ` [PATCH 01/17] of: Add for_each_child_of_node_safe Sascha Hauer
2015-02-12 8:54 ` [PATCH 02/17] mtd: core: add error checks Sascha Hauer
2015-02-12 8:54 ` [PATCH 03/17] mtd: partitions: Use xstrdup Sascha Hauer
2015-02-12 8:54 ` [PATCH 04/17] mtd: partitions: Add error check Sascha Hauer
2015-02-12 8:54 ` [PATCH 05/17] mtd: Add partitions to list Sascha Hauer
2015-02-12 8:54 ` [PATCH 06/17] mtd: nand: remove automatically created bb devices Sascha Hauer
2015-02-12 8:54 ` [PATCH 07/17] move cmdline partition parsing code to separate file Sascha Hauer
2015-02-12 8:54 ` [PATCH 08/17] cmdlinepart: Change SIZE_REMAINING to loff_t Sascha Hauer
2015-02-12 8:54 ` [PATCH 09/17] cmdlinepart: make argument types safer Sascha Hauer
2015-02-12 8:54 ` [PATCH 10/17] cmdlinepart: add function to parse a cmdline partition string Sascha Hauer
2015-02-12 8:54 ` [PATCH 11/17] cmndlinepart: skip devname if partstr already contains it Sascha Hauer
2015-02-12 8:54 ` [PATCH 12/17] mtd: forbid conflicting mtd partitions Sascha Hauer
2015-02-12 8:54 ` [PATCH 13/17] mtd: Use flags parameter in mtd_add_partition Sascha Hauer
2015-02-12 8:54 ` [PATCH 14/17] mtd: Add a partitions parameter to mtd devices Sascha Hauer
2015-02-12 8:54 ` [PATCH 15/17] mtd: fixup device tree partitions Sascha Hauer
2015-02-12 8:54 ` [PATCH 16/17] defaultenv-2: mtdparts-add: remove unused variable Sascha Hauer
2015-02-12 8:54 ` [PATCH 17/17] defaultenv-2: mtdparts-add: Use new partition parameter Sascha Hauer
2015-02-13 5:54 ` Bo Shen [this message]
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=54DD9199.1090403@atmel.com \
--to=voice.shen@atmel.com \
--cc=barebox@lists.infradead.org \
--cc=s.hauer@pengutronix.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.