All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sascha Hauer <s.hauer@pengutronix.de>
To: Peter Korsgaard <jacmet@sunsite.dk>
Cc: barebox@lists.infradead.org
Subject: Re: [PATCH] commands/crc32: add compare 2 files crc
Date: Wed, 22 Sep 2010 18:08:34 +0200	[thread overview]
Message-ID: <20100922160834.GG23406@pengutronix.de> (raw)
In-Reply-To: <87d3s5by0y.fsf@macbook.be.48ers.dk>

On Wed, Sep 22, 2010 at 05:25:49PM +0200, Peter Korsgaard wrote:
> >>>>> "Sascha" == Sascha Hauer <s.hauer@pengutronix.de> writes:
> 
>  Sascha> On Wed, Sep 22, 2010 at 12:36:25PM +0200, Jean-Christophe PLAGNIOL-VILLARD wrote:
>  >> add -F options to compare to file crc
>  >> 
>  >> it's usefull to compare what you flash in a partition
> 
>  Sascha> Why don't you use memcmp?
> 
>  Sascha> memcmp -s uImage -d /dev/nand0.kernel.bb 0 0
> 
> It's sometimes interesting to be able to compare a checksum with a file
> on the host (E.G. output of cksum/md5sum/sha1sum/..)

crc32 can output the crc checksum of a file using the -f option. What
Jean-Christophe added is an option to calculate the checksum of two
files (one given with -f and the other with -F). I just fail to see why
this must be done in a single step instead of using a crc32 followed by
a memcmp.

Sascha

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

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

  reply	other threads:[~2010-09-22 16:08 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-09-22 10:36 [PATCH] commands/crc32: add compare 2 files crc Jean-Christophe PLAGNIOL-VILLARD
2010-09-22 12:36 ` Sascha Hauer
2010-09-22 13:12   ` Jean-Christophe PLAGNIOL-VILLARD
2010-09-22 15:25   ` Peter Korsgaard
2010-09-22 16:08     ` Sascha Hauer [this message]
2010-09-22 16:12       ` Jean-Christophe PLAGNIOL-VILLARD
2010-09-22 16:54         ` Sascha Hauer
2010-09-23 11:25           ` Jean-Christophe PLAGNIOL-VILLARD
2010-09-22 19:09       ` Peter Korsgaard

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=20100922160834.GG23406@pengutronix.de \
    --to=s.hauer@pengutronix.de \
    --cc=barebox@lists.infradead.org \
    --cc=jacmet@sunsite.dk \
    /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.