From: "Andreas Bießmann" <andreas.devel@googlemail.com>
To: u-boot@lists.denx.de
Subject: [U-Boot] [U-Boot, RFC] ext4fs: le32_to_cpu() used on a 16-bit field
Date: Sun, 21 Jul 2013 10:24:19 +0200 [thread overview]
Message-ID: <51EB9AB3.8090407@googlemail.com> (raw)
In-Reply-To: <CAPnjgZ2FA00KOxhJOau_dooE_X5Ddhx6-fGwyOz+skcurGo4aw@mail.gmail.com>
Hi all,
On 21.07.2013 06:32, Simon Glass wrote:
> +Tom
>
> On Fri, Jul 19, 2013 at 1:03 AM, Lukasz Majewski <l.majewski@samsung.com>wrote:
>
>> On Tue, 16 Jul 2013 08:14:35 +0000 (UTC) Rommel Custodio
>> sessyargc+uboot at gmail.com wrote,
>>
>> Hi Rommel,
>>
>>> Hi All,
>>>
>>> U-Boot 2013.07-rc3 [ELDK 5.2.1 / ELDK 5.3]
>>>
>>> Now I've started to use the new ext4 code. I need the "ext4write"
>>> command. Though there seems to be several problems with the ext2/ext4
>>> code.
>>>
>>> I am testing on an ml507 (PPC440, Big Endian).
>>> There are some cases where the a field is 16-bit but le32_to_cpu() is
>>> used. Some checks (ie eh_magic) fails to match even if I use a
>>> correctly ext4 formatted MMC/SD card.
>>>
>>> Does these seem right? Or am I mistaken?
>>
>> What kind of mailer program have you used to sent this e-mail?
>> I receive following errors:
>>
>> fatal: corrupt patch at line 111
>>
>> I cannot apply this patch with either git am -3 or git apply.
>>
>> Were you using git send-email?
>>
>
> It seems like we should try to fix this before the release?
+1
I can confirm that reading ext4 extended header on BE machine (avr32) is
broken. I'm currently on it to get it working by using Rommel's patch.
Regards
Andreas Bie?mann
next prev parent reply other threads:[~2013-07-21 8:24 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-07-16 8:14 [U-Boot] [U-Boot, RFC] ext4fs: le32_to_cpu() used on a 16-bit field Rommel Custodio
2013-07-16 8:33 ` Lukasz Majewski
[not found] ` <20130716204531.GA548@CrG.local>
2013-07-16 23:32 ` Rommel Custodio
2013-07-19 7:03 ` Lukasz Majewski
2013-07-20 8:03 ` [PATCH, RESEND] " Rommel G Custodio
2013-07-21 4:32 ` [U-Boot] [U-Boot, RFC] " Simon Glass
2013-07-21 8:24 ` Andreas Bießmann [this message]
2013-08-05 7:31 ` Pavel Machek
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=51EB9AB3.8090407@googlemail.com \
--to=andreas.devel@googlemail.com \
--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