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 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox