mail archive of the barebox mailing list
 help / color / mirror / Atom feed
From: Trent Piepho <trent.piepho@igorinstitute.com>
To: Sascha Hauer <sha@pengutronix.de>
Cc: Barebox List <barebox@lists.infradead.org>, Yunus Bas <Y.Bas@phytec.de>
Subject: Re: [PATCH 3/3] imx-bbu-nand-fcb: Add command to help debug FCB issues
Date: Wed, 13 Oct 2021 03:14:01 -0700
Message-ID: <CAMHeXxO04HMwZ1nbFtm3n=SzaMD9ERBEgXXK-XyOdVvqju2DUg@mail.gmail.com> (raw)
In-Reply-To: <20211012082135.GN28453@pengutronix.de>

On Tue, Oct 12, 2021 at 1:21 AM Sascha Hauer <sha@pengutronix.de> wrote:
> On Mon, Oct 11, 2021 at 06:53:59PM -0700, Trent Piepho wrote:
> > Add new "fcb" command.  It can save a decoded copy of the FCB to a file,
> > do a hexdump of the decoded FCB, or display the FCB fields.  Or simply
> > read and validate the FCB.

> Not sure if we need to control this command in such a fine grained way.
> For me just extracting all possible FCBs including the firmware images,
> maybe printing consistency information would be enough. That's just a
> personal opinion though, feel free to override it.

Originally I was having issues creating correct FCBs, mostly due to
kobs-ng (I wonder how hard it would be to port barebox_update to
Linux?) and wanted to extract factory FCB and kobs-ng generated FCB.
But copying data from Barebox to Linux and looking at hexdumps was
very tedious.  Really, one wants to see fields of FCB decoded and
Barebox already had code that did this.

So the -i flag that prints out FCB fields and -o to save a copy were
what I wrote originally as debug aid, with -i the most useful to me.
I don't actually want to extract firmware images.  It was the FCB that
was the issue.  I didn't think it would be useful enough to other
people to bother sending it to the list.

But then later there was a thread from Yunas at Phytec about the
difficulty of doing a flash crc check on NAND when one does not know
where data will be due to possible bad blocks.  Extracting this
information from the FCB seemed like the correct way to do it and I
realized it would be easy to add into the command I had written.  So
that is why this feature is there.  And this was evidence that this
would be useful to someone besides myself.

I added hexdump because it seemed like someone might like it and it
was one line.  I could drop this part.

Extracting all possible FCBs has issues.  Number and location of FCBs
varies.  Pin strapping and possibly OTP memory fuses control what the
boot ROM does.  However, the boot ROM's search, what is actually in
flash, and what kobs-ng wants to write, can all be different.  I did
write this debug aid for a real problem!

If it did just write dump everything, then how would it work?  Some
details are not clear to me.

Where does the data go?  Assume /tmp?  Or argument to supply directory name?
It will need multiple files.  How to name?  Arguments for each
filename?  Seems too many arguments.  Or have a fixed filename
pattern?  FCB1, FCB2, firmware1.img, firmware2.img, etc.  Not really a
huge fan of hardcoded filenames.
What happens if the FCBs are not where Barebox thinks they are?  This
really does happen.
What if all the FCBs do not agree on the location/size of the firmware images?
Is it possible extra space used to dump firmware copies into /tmp,
then crc the copies matters vs doing it in place from flash?
What if someone wants to script something with firmware images other
than a checksum?  E.g., they want to erase firmware1 to test that
fallback to firmware2 works.  Or want to know how large the firmware
is.  I do not know of a way to get the size of a file in hush.

_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox


  reply	other threads:[~2021-10-13 11:30 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-10-12  1:53 [PATCH 1/3] imx-bbu-nand-fcb: Show additional information from decoded FCB Trent Piepho
2021-10-12  1:53 ` [PATCH 2/3] imx-bbu-nand-fcb: Save bootloader location into device parameters Trent Piepho
2021-10-12  8:11   ` Sascha Hauer
2021-10-12  1:53 ` [PATCH 3/3] imx-bbu-nand-fcb: Add command to help debug FCB issues Trent Piepho
2021-10-12  8:21   ` Sascha Hauer
2021-10-13 10:14     ` Trent Piepho [this message]
2021-10-14 12:09       ` Sascha Hauer

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='CAMHeXxO04HMwZ1nbFtm3n=SzaMD9ERBEgXXK-XyOdVvqju2DUg@mail.gmail.com' \
    --to=trent.piepho@igorinstitute.com \
    --cc=Y.Bas@phytec.de \
    --cc=barebox@lists.infradead.org \
    --cc=sha@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

mail archive of the barebox mailing list

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://lore.barebox.org/barebox/0 barebox/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 barebox barebox/ https://lore.barebox.org/barebox \
		barebox@lists.infradead.org
	public-inbox-index barebox

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git