[Buildroot] [PATCH 1/2] cramfs: change to new site location
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Fri Apr 6 15:13:31 UTC 2018
Hello,
On Fri, 6 Apr 2018 15:02:55 +0000, Chris Brandt wrote:
> > It is missing the new case where the compressed block length is stored
> > as an u16 at the beginning of the block. That's pretty much the only use
> > of u16 in the code so easy to locate.
>
> So then in that case, you would need a bswap_32() and a bswap_16() macro.
>
> However...that code is currently inside a "} else if (0) {" block, so it
> looks like it will never get executed anyway.
I don't see where this else if (0) block is. Where do you see that ?
Matthew: the current patch in Buildroot also fixes cramfsck, but not
your contribution on the new version of the cramfs tools.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list