From: Heiko Schocher <hs@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [RFC][DFU] Unification of dfu_alt_info alt settings description + command execution
Date: Fri, 19 Jul 2013 06:45:34 +0200 [thread overview]
Message-ID: <51E8C46E.5040103@denx.de> (raw)
In-Reply-To: <201307181710.05980.marex@denx.de>
Hello Marek,
Am 18.07.2013 17:10, schrieb Marek Vasut:
> Hi,
>
>> Dear Heiko Schocher,
>>
>> In message<51E77A1D.90403@denx.de> you wrote:
>>>> Try "nand write.trimffs" to write UBI images produced with ubinize .
>>>
>>> This solves not the erasecounter problem, or?
>>>
>>> For UBI we need something like this:
>>> http://www.linux-mtd.infradead.org/doc/ubi.html#L_flasher_algo
>>>
>>> But I am not an UBI expert. It is possible I overlook something
>>> obvious ...
>>
>> No, you don't. Devices managed by UBI should never be erased by
>> other, non-UBI-aware tools.
>
> I based my reply on the following commit in U-Boot and the fact that
> write.trimffs is used to flash UBI images. Maybe I was wrong?
>
> commit c9494866df835bcee68e17339aec1090faa704da
> Author: Ben Gardiner<bengardiner@nanometrics.ca>
> Date: Tue Jun 14 16:35:07 2011 -0400
>
> cmd_nand: add nand write.trimffs command
>
> Add another nand write. variant, trimffs. This command will request of
> nand_write_skip_bad() that all trailing all-0xff pages will be
> dropped from eraseblocks when they are written to flash as-per the
> reccommended behaviour of the UBI FAQ [1].
>
> The function that implements this timming is the drop_ffs() function
> by Artem Bityutskiy, ported from the mtd-utils tree.
>
> [1] http://www.linux-mtd.infradead.org/doc/ubi.html#L_flasher_algo
Yes, that sounds as a step in the right direction, but where are
the erasecounters handled, as described in [1] ?
And as this is a "ubi function" and not nand specific, the command
should start with "ubi ..." ... as we have a "ubi write ...", but
ubi write is only for ubi volumes ... i tend to say, we need a
"ubi format ..." similiar to ubiformat in the mtd utils [2] ...
[2] http://git.infradead.org/mtd-utils.git
ubiformat found in /ubi-utils/ubiformat.c
bye,
Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
next prev parent reply other threads:[~2013-07-19 4:45 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-16 15:35 [U-Boot] [RFC][DFU] Unification of dfu_alt_info alt settings description + command execution Lukasz Majewski
2013-07-16 21:27 ` Tormod Volden
2013-07-16 21:46 ` Lukasz Majewski
2013-07-17 10:26 ` Heiko Schocher
2013-07-17 14:34 ` Lukasz Majewski
2013-07-17 17:32 ` Tormod Volden
2013-07-18 5:36 ` Heiko Schocher
2013-07-18 7:13 ` Lukasz Majewski
2013-07-18 4:17 ` Marek Vasut
2013-07-18 5:16 ` Heiko Schocher
2013-07-18 8:09 ` Wolfgang Denk
2013-07-18 15:10 ` Marek Vasut
2013-07-19 4:45 ` Heiko Schocher [this message]
2013-07-19 13:55 ` Marek Vasut
2013-07-18 16:39 ` Tom Rini
2013-07-18 17:30 ` Michael Cashwell
2013-07-18 20:17 ` Lukasz Majewski
2013-07-18 22:33 ` Tom Rini
2013-08-23 10:07 ` Lukasz Majewski
2013-10-31 17:25 ` Lukasz Majewski
2013-10-31 20:32 ` Wolfgang Denk
2013-10-31 21:20 ` Lukasz Majewski
2013-10-31 23:11 ` Wolfgang Denk
2013-11-04 6:52 ` Lukasz Majewski
2013-11-01 6:15 ` Heiko Schocher
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=51E8C46E.5040103@denx.de \
--to=hs@denx.de \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox