* [U-Boot] [PATCH 6/7] FAT: Simplify get_contents @ 2012-07-19 22:02 Benoît Thébaudeau 2012-07-20 13:21 ` [U-Boot] [PATCH v2 7/8] " Benoît Thébaudeau 0 siblings, 1 reply; 10+ messages in thread From: Benoît Thébaudeau @ 2012-07-19 22:02 UTC (permalink / raw) To: u-boot One call to get_cluster can be factorized with another, so avoid duplicating code. Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> Cc: Wolfgang Denk <wd@denx.de> --- .../fs/fat/fat.c | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git u-boot-66714b1.orig/fs/fat/fat.c u-boot-66714b1/fs/fat/fat.c index 4c80820..8ac8b85 100644 --- u-boot-66714b1.orig/fs/fat/fat.c +++ u-boot-66714b1/fs/fat/fat.c @@ -368,21 +368,9 @@ get_contents (fsdata *mydata, dir_entry *dentptr, __u8 *buffer, actsize += bytesperclust; } - /* actsize >= file size */ - actsize -= bytesperclust; - - /* get remaining clusters */ - if (get_cluster(mydata, curclust, buffer, (int)actsize) != 0) { - printf("Error reading cluster\n"); - return -1; - } - /* get remaining bytes */ - gotsize += (int)actsize; - filesize -= actsize; - buffer += actsize; actsize = filesize; - if (get_cluster(mydata, endclust, buffer, (int)actsize) != 0) { + if (get_cluster(mydata, curclust, buffer, (int)actsize) != 0) { printf("Error reading cluster\n"); return -1; } ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [U-Boot] [PATCH v2 7/8] FAT: Simplify get_contents 2012-07-19 22:02 [U-Boot] [PATCH 6/7] FAT: Simplify get_contents Benoît Thébaudeau @ 2012-07-20 13:21 ` Benoît Thébaudeau 2012-09-02 15:25 ` Wolfgang Denk 0 siblings, 1 reply; 10+ messages in thread From: Benoît Thébaudeau @ 2012-07-20 13:21 UTC (permalink / raw) To: u-boot One call to get_cluster can be factorized with another, so avoid duplicating code. Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> Cc: Wolfgang Denk <wd@denx.de> --- Changes for v2: - Patch renumbering because of the new v2 1/8. - Possible code style changes due to the new v2 1/8. .../fs/fat/fat.c | 14 +------------- 1 file changed, 1 insertion(+), 13 deletions(-) diff --git u-boot-66714b1.orig/fs/fat/fat.c u-boot-66714b1/fs/fat/fat.c index 101eb3a..19f6a8c 100644 --- u-boot-66714b1.orig/fs/fat/fat.c +++ u-boot-66714b1/fs/fat/fat.c @@ -367,21 +367,9 @@ get_contents(fsdata *mydata, dir_entry *dentptr, __u8 *buffer, actsize += bytesperclust; } - /* actsize >= file size */ - actsize -= bytesperclust; - - /* get remaining clusters */ - if (get_cluster(mydata, curclust, buffer, (int)actsize) != 0) { - printf("Error reading cluster\n"); - return -1; - } - /* get remaining bytes */ - gotsize += (int)actsize; - filesize -= actsize; - buffer += actsize; actsize = filesize; - if (get_cluster(mydata, endclust, buffer, (int)actsize) != 0) { + if (get_cluster(mydata, curclust, buffer, (int)actsize) != 0) { printf("Error reading cluster\n"); return -1; } ^ permalink raw reply related [flat|nested] 10+ messages in thread
* [U-Boot] [PATCH v2 7/8] FAT: Simplify get_contents 2012-07-20 13:21 ` [U-Boot] [PATCH v2 7/8] " Benoît Thébaudeau @ 2012-09-02 15:25 ` Wolfgang Denk 2012-09-04 20:50 ` Tom Rini 0 siblings, 1 reply; 10+ messages in thread From: Wolfgang Denk @ 2012-09-02 15:25 UTC (permalink / raw) To: u-boot Dear Beno?t Th?baudeau, In message <1663419836.332713.1342790497668.JavaMail.root@advansee.com> you wrote: > One call to get_cluster can be factorized with another, so avoid duplicatin> g > code. > > Signed-off-by: Beno?t Th?baudeau <benoit.thebaudeau@advansee.com> > Cc: Wolfgang Denk <wd@denx.de> > --- > Changes for v2: > - Patch renumbering because of the new v2 1/8. > - Possible code style changes due to the new v2 1/8. > > .../fs/fat/fat.c | 14 +------------- > 1 file changed, 1 insertion(+), 13 deletions(-) Applied, thanks. Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de There were meetings. There were always meetings. And they were dull, which is part of the reason they were meetings. Dull likes company. - Terry Pratchett, _Making_Money_ ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [PATCH v2 7/8] FAT: Simplify get_contents 2012-09-02 15:25 ` Wolfgang Denk @ 2012-09-04 20:50 ` Tom Rini 2012-09-04 22:07 ` Benoît Thébaudeau 2012-09-04 22:08 ` Wolfgang Denk 0 siblings, 2 replies; 10+ messages in thread From: Tom Rini @ 2012-09-04 20:50 UTC (permalink / raw) To: u-boot On Sun, Sep 02, 2012 at 05:25:20PM +0200, Wolfgang Denk wrote: > Dear Beno??t Th??baudeau, > > In message <1663419836.332713.1342790497668.JavaMail.root@advansee.com> you wrote: > > One call to get_cluster can be factorized with another, so avoid duplicatin> g > > code. > > > > Signed-off-by: Beno??t Th??baudeau <benoit.thebaudeau@advansee.com> > > Cc: Wolfgang Denk <wd@denx.de> > > --- > > Changes for v2: > > - Patch renumbering because of the new v2 1/8. > > - Possible code style changes due to the new v2 1/8. > > > > .../fs/fat/fat.c | 14 +------------- > > 1 file changed, 1 insertion(+), 13 deletions(-) > > Applied, thanks. OK, this change is NOT equivalent code. My platforms now hang thusly (with DEBUG set): reading u-boot.img VFAT Support enabled FAT16, fat_sect: 4, fatlength: 144 Rootdir begins at cluster: 0, sector: 292, offset: 24800 Data begins at: 316 Sector size: 512, cluster size: 4 FAT read sect=292, clust_size=4, DIRENTSPERBLOCK=16 Rootvfatname: |u-boot.ais| RootMismatch: |u-boot.ais|u-boot.ais| RootMismatch: |u-boot.ais|| RootMismatch: |mlo|| Rootvfatname: |u-boot.img| RootName: u-boot.img, start: 0xc2, size: 0x337d0 Filesize: 210896 bytes 64 bytes gc - clustnum: 194, startsect: 1092 Size: 210896, got: 64 This is all fine in full U-Boot. -- Tom ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [PATCH v2 7/8] FAT: Simplify get_contents 2012-09-04 20:50 ` Tom Rini @ 2012-09-04 22:07 ` Benoît Thébaudeau 2012-09-04 22:10 ` Tom Rini 2012-09-04 22:08 ` Wolfgang Denk 1 sibling, 1 reply; 10+ messages in thread From: Benoît Thébaudeau @ 2012-09-04 22:07 UTC (permalink / raw) To: u-boot Hi Tom, On Tuesday, September 4, 2012 10:50:34 PM, Tom Rini wrote: > On Sun, Sep 02, 2012 at 05:25:20PM +0200, Wolfgang Denk wrote: > > Dear Beno??t Th??baudeau, > > > > In message > > <1663419836.332713.1342790497668.JavaMail.root@advansee.com> you > > wrote: > > > One call to get_cluster can be factorized with another, so avoid > > > duplicatin> g > > > code. > > > > > > Signed-off-by: Beno??t Th??baudeau > > > <benoit.thebaudeau@advansee.com> > > > Cc: Wolfgang Denk <wd@denx.de> > > > --- > > > Changes for v2: > > > - Patch renumbering because of the new v2 1/8. > > > - Possible code style changes due to the new v2 1/8. > > > > > > .../fs/fat/fat.c | 14 > > > +------------- > > > 1 file changed, 1 insertion(+), 13 deletions(-) > > > > Applied, thanks. > > OK, this change is NOT equivalent code. My platforms now hang thusly > (with DEBUG set): > reading u-boot.img > VFAT Support enabled > FAT16, fat_sect: 4, fatlength: 144 > Rootdir begins at cluster: 0, sector: 292, offset: 24800 > Data begins at: 316 > Sector size: 512, cluster size: 4 > FAT read sect=292, clust_size=4, DIRENTSPERBLOCK=16 > Rootvfatname: |u-boot.ais| > RootMismatch: |u-boot.ais|u-boot.ais| > RootMismatch: |u-boot.ais|| > RootMismatch: |mlo|| > Rootvfatname: |u-boot.img| > RootName: u-boot.img, start: 0xc2, size: 0x337d0 > Filesize: 210896 bytes > 64 bytes > gc - clustnum: 194, startsect: 1092 > Size: 210896, got: 64 > > This is all fine in full U-Boot. OK. I'm looking into it. Can you give more details, like the type of storage (usb, mmc, etc.)? Do you have a command line and a disk image that could be used to duplicate the issue? Best regards, Beno?t ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [PATCH v2 7/8] FAT: Simplify get_contents 2012-09-04 22:07 ` Benoît Thébaudeau @ 2012-09-04 22:10 ` Tom Rini 2012-09-04 22:41 ` Tom Rini 2012-09-04 22:53 ` Benoît Thébaudeau 0 siblings, 2 replies; 10+ messages in thread From: Tom Rini @ 2012-09-04 22:10 UTC (permalink / raw) To: u-boot On 09/04/2012 03:07 PM, Beno?t Th?baudeau wrote: > Hi Tom, > > On Tuesday, September 4, 2012 10:50:34 PM, Tom Rini wrote: >> On Sun, Sep 02, 2012 at 05:25:20PM +0200, Wolfgang Denk wrote: >>> Dear Beno??t Th??baudeau, >>> >>> In message >>> <1663419836.332713.1342790497668.JavaMail.root@advansee.com> you >>> wrote: >>>> One call to get_cluster can be factorized with another, so avoid >>>> duplicatin> g >>>> code. >>>> >>>> Signed-off-by: Beno??t Th??baudeau >>>> <benoit.thebaudeau@advansee.com> >>>> Cc: Wolfgang Denk <wd@denx.de> >>>> --- >>>> Changes for v2: >>>> - Patch renumbering because of the new v2 1/8. >>>> - Possible code style changes due to the new v2 1/8. >>>> >>>> .../fs/fat/fat.c | 14 >>>> +------------- >>>> 1 file changed, 1 insertion(+), 13 deletions(-) >>> >>> Applied, thanks. >> >> OK, this change is NOT equivalent code. My platforms now hang thusly >> (with DEBUG set): >> reading u-boot.img >> VFAT Support enabled >> FAT16, fat_sect: 4, fatlength: 144 >> Rootdir begins at cluster: 0, sector: 292, offset: 24800 >> Data begins at: 316 >> Sector size: 512, cluster size: 4 >> FAT read sect=292, clust_size=4, DIRENTSPERBLOCK=16 >> Rootvfatname: |u-boot.ais| >> RootMismatch: |u-boot.ais|u-boot.ais| >> RootMismatch: |u-boot.ais|| >> RootMismatch: |mlo|| >> Rootvfatname: |u-boot.img| >> RootName: u-boot.img, start: 0xc2, size: 0x337d0 >> Filesize: 210896 bytes >> 64 bytes >> gc - clustnum: 194, startsect: 1092 >> Size: 210896, got: 64 >> >> This is all fine in full U-Boot. > > OK. I'm looking into it. > > Can you give more details, like the type of storage (usb, mmc, etc.)? Do you > have a command line and a disk image that could be used to duplicate the issue? It's an SD card. If you have any "OMAP" platform (beagleboard, beaglebone, pandaboard) or am35x/am37x or similar platforms SPL should hang like that. 72MB partition (or so) on either a 2 or 4GB card. Getting all the way up into U-Boot clears the problem away until power cycle. That last part makes me worried... -- Tom ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [PATCH v2 7/8] FAT: Simplify get_contents 2012-09-04 22:10 ` Tom Rini @ 2012-09-04 22:41 ` Tom Rini 2012-09-04 22:53 ` Benoît Thébaudeau 1 sibling, 0 replies; 10+ messages in thread From: Tom Rini @ 2012-09-04 22:41 UTC (permalink / raw) To: u-boot On 09/04/2012 03:10 PM, Tom Rini wrote: > On 09/04/2012 03:07 PM, Beno?t Th?baudeau wrote: >> Hi Tom, >> >> On Tuesday, September 4, 2012 10:50:34 PM, Tom Rini wrote: >>> On Sun, Sep 02, 2012 at 05:25:20PM +0200, Wolfgang Denk wrote: >>>> Dear Beno??t Th??baudeau, >>>> >>>> In message >>>> <1663419836.332713.1342790497668.JavaMail.root@advansee.com> you >>>> wrote: >>>>> One call to get_cluster can be factorized with another, so avoid >>>>> duplicatin> g >>>>> code. >>>>> >>>>> Signed-off-by: Beno??t Th??baudeau >>>>> <benoit.thebaudeau@advansee.com> >>>>> Cc: Wolfgang Denk <wd@denx.de> >>>>> --- >>>>> Changes for v2: >>>>> - Patch renumbering because of the new v2 1/8. >>>>> - Possible code style changes due to the new v2 1/8. >>>>> >>>>> .../fs/fat/fat.c | 14 >>>>> +------------- >>>>> 1 file changed, 1 insertion(+), 13 deletions(-) >>>> >>>> Applied, thanks. >>> >>> OK, this change is NOT equivalent code. My platforms now hang thusly >>> (with DEBUG set): >>> reading u-boot.img >>> VFAT Support enabled >>> FAT16, fat_sect: 4, fatlength: 144 >>> Rootdir begins at cluster: 0, sector: 292, offset: 24800 >>> Data begins at: 316 >>> Sector size: 512, cluster size: 4 >>> FAT read sect=292, clust_size=4, DIRENTSPERBLOCK=16 >>> Rootvfatname: |u-boot.ais| >>> RootMismatch: |u-boot.ais|u-boot.ais| >>> RootMismatch: |u-boot.ais|| >>> RootMismatch: |mlo|| >>> Rootvfatname: |u-boot.img| >>> RootName: u-boot.img, start: 0xc2, size: 0x337d0 >>> Filesize: 210896 bytes >>> 64 bytes >>> gc - clustnum: 194, startsect: 1092 >>> Size: 210896, got: 64 >>> >>> This is all fine in full U-Boot. >> >> OK. I'm looking into it. >> >> Can you give more details, like the type of storage (usb, mmc, etc.)? Do you >> have a command line and a disk image that could be used to duplicate the issue? > > It's an SD card. If you have any "OMAP" platform (beagleboard, > beaglebone, pandaboard) or am35x/am37x or similar platforms SPL should > hang like that. 72MB partition (or so) on either a 2 or 4GB card. > Getting all the way up into U-Boot clears the problem away until power > cycle. That last part makes me worried... OK, this is somehow a 'me' problem it seems. I don't see it on a beagleboard and rebuilding things gives me a different failure now, so something is up. Digging more now, sorry for the noise. -- Tom ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [PATCH v2 7/8] FAT: Simplify get_contents 2012-09-04 22:10 ` Tom Rini 2012-09-04 22:41 ` Tom Rini @ 2012-09-04 22:53 ` Benoît Thébaudeau 2012-09-05 0:03 ` Tom Rini 1 sibling, 1 reply; 10+ messages in thread From: Benoît Thébaudeau @ 2012-09-04 22:53 UTC (permalink / raw) To: u-boot On Wednesday, September 5, 2012 12:10:41 AM, Tom Rini wrote: > On 09/04/2012 03:07 PM, Beno?t Th?baudeau wrote: > > On Tuesday, September 4, 2012 10:50:34 PM, Tom Rini wrote: > >> On Sun, Sep 02, 2012 at 05:25:20PM +0200, Wolfgang Denk wrote: > >>> Dear Beno??t Th??baudeau, > >>> > >>> In message > >>> <1663419836.332713.1342790497668.JavaMail.root@advansee.com> you > >>> wrote: > >>>> One call to get_cluster can be factorized with another, so avoid > >>>> duplicatin> g > >>>> code. > >>>> > >>>> Signed-off-by: Beno??t Th??baudeau > >>>> <benoit.thebaudeau@advansee.com> > >>>> Cc: Wolfgang Denk <wd@denx.de> > >>>> --- > >>>> Changes for v2: > >>>> - Patch renumbering because of the new v2 1/8. > >>>> - Possible code style changes due to the new v2 1/8. > >>>> > >>>> .../fs/fat/fat.c | 14 > >>>> +------------- > >>>> 1 file changed, 1 insertion(+), 13 deletions(-) > >>> > >>> Applied, thanks. > >> > >> OK, this change is NOT equivalent code. My platforms now hang > >> thusly > >> (with DEBUG set): > >> reading u-boot.img > >> VFAT Support enabled > >> FAT16, fat_sect: 4, fatlength: 144 > >> Rootdir begins at cluster: 0, sector: 292, offset: 24800 > >> Data begins at: 316 > >> Sector size: 512, cluster size: 4 > >> FAT read sect=292, clust_size=4, DIRENTSPERBLOCK=16 > >> Rootvfatname: |u-boot.ais| > >> RootMismatch: |u-boot.ais|u-boot.ais| > >> RootMismatch: |u-boot.ais|| > >> RootMismatch: |mlo|| > >> Rootvfatname: |u-boot.img| > >> RootName: u-boot.img, start: 0xc2, size: 0x337d0 > >> Filesize: 210896 bytes > >> 64 bytes > >> gc - clustnum: 194, startsect: 1092 > >> Size: 210896, got: 64 > >> > >> This is all fine in full U-Boot. > > > > OK. I'm looking into it. > > > > Can you give more details, like the type of storage (usb, mmc, > > etc.)? Do you > > have a command line and a disk image that could be used to > > duplicate the issue? > > It's an SD card. If you have any "OMAP" platform (beagleboard, > beaglebone, pandaboard) or am35x/am37x or similar platforms SPL > should > hang like that. Unfortunately, I don't have these platforms. > 72MB partition (or so) on either a 2 or 4GB card. > Getting all the way up into U-Boot clears the problem away until > power > cycle. That last part makes me worried... What do you mean by "Getting all the way up into U-Boot"? Is it that your SPL can be aborted and give a command prompt? The only difference that this specific patch (7/8) introduces in your use case (i.e. a request to read the first 64 bytes of a file of 210896 bytes) is that it removes a spurious call to get_cluster() with a size of 0, which should be transparent. Can you confirm that your tests bisect to this patch and not to another one in this series? Can you try to apply locally 5/8 and 8/8 to see if it makes a difference, just in case there would be some dependencies? Is it OK to send e-mails with attachments on the ML? If so, I can post the full fat.c that I currently use so that you can test with it. I have used it for a couple of years without any issue. Also, why do you read only 64 bytes from this file? According to your debug trace, this read was successful, and then no error is shown. So, if it hangs, it might be that the read data is corrupted, or that the image is not fully read because of this 64-byte limit. Best regards, Beno?t ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [PATCH v2 7/8] FAT: Simplify get_contents 2012-09-04 22:53 ` Benoît Thébaudeau @ 2012-09-05 0:03 ` Tom Rini 0 siblings, 0 replies; 10+ messages in thread From: Tom Rini @ 2012-09-05 0:03 UTC (permalink / raw) To: u-boot On 09/04/2012 03:53 PM, Beno?t Th?baudeau wrote: > On Wednesday, September 5, 2012 12:10:41 AM, Tom Rini wrote: >> On 09/04/2012 03:07 PM, Beno?t Th?baudeau wrote: >>> On Tuesday, September 4, 2012 10:50:34 PM, Tom Rini wrote: >>>> On Sun, Sep 02, 2012 at 05:25:20PM +0200, Wolfgang Denk wrote: >>>>> Dear Beno??t Th??baudeau, >>>>> >>>>> In message >>>>> <1663419836.332713.1342790497668.JavaMail.root@advansee.com> you >>>>> wrote: >>>>>> One call to get_cluster can be factorized with another, so avoid >>>>>> duplicatin> g >>>>>> code. >>>>>> >>>>>> Signed-off-by: Beno??t Th??baudeau >>>>>> <benoit.thebaudeau@advansee.com> >>>>>> Cc: Wolfgang Denk <wd@denx.de> >>>>>> --- >>>>>> Changes for v2: >>>>>> - Patch renumbering because of the new v2 1/8. >>>>>> - Possible code style changes due to the new v2 1/8. >>>>>> >>>>>> .../fs/fat/fat.c | 14 >>>>>> +------------- >>>>>> 1 file changed, 1 insertion(+), 13 deletions(-) >>>>> >>>>> Applied, thanks. >>>> >>>> OK, this change is NOT equivalent code. My platforms now hang >>>> thusly >>>> (with DEBUG set): >>>> reading u-boot.img >>>> VFAT Support enabled >>>> FAT16, fat_sect: 4, fatlength: 144 >>>> Rootdir begins at cluster: 0, sector: 292, offset: 24800 >>>> Data begins at: 316 >>>> Sector size: 512, cluster size: 4 >>>> FAT read sect=292, clust_size=4, DIRENTSPERBLOCK=16 >>>> Rootvfatname: |u-boot.ais| >>>> RootMismatch: |u-boot.ais|u-boot.ais| >>>> RootMismatch: |u-boot.ais|| >>>> RootMismatch: |mlo|| >>>> Rootvfatname: |u-boot.img| >>>> RootName: u-boot.img, start: 0xc2, size: 0x337d0 >>>> Filesize: 210896 bytes >>>> 64 bytes >>>> gc - clustnum: 194, startsect: 1092 >>>> Size: 210896, got: 64 >>>> >>>> This is all fine in full U-Boot. >>> >>> OK. I'm looking into it. >>> >>> Can you give more details, like the type of storage (usb, mmc, >>> etc.)? Do you >>> have a command line and a disk image that could be used to >>> duplicate the issue? >> >> It's an SD card. If you have any "OMAP" platform (beagleboard, >> beaglebone, pandaboard) or am35x/am37x or similar platforms SPL >> should >> hang like that. > > Unfortunately, I don't have these platforms. > >> 72MB partition (or so) on either a 2 or 4GB card. >> Getting all the way up into U-Boot clears the problem away until >> power >> cycle. That last part makes me worried... > > What do you mean by "Getting all the way up into U-Boot"? Is it that your SPL > can be aborted and give a command prompt? > > The only difference that this specific patch (7/8) introduces in your use case > (i.e. a request to read the first 64 bytes of a file of 210896 bytes) is that it > removes a spurious call to get_cluster() with a size of 0, which should be > transparent. > > Can you confirm that your tests bisect to this patch and not to another one in > this series? > > Can you try to apply locally 5/8 and 8/8 to see if it makes a difference, just > in case there would be some dependencies? > > Is it OK to send e-mails with attachments on the ML? If so, I can post the full > fat.c that I currently use so that you can test with it. I have used it for a > couple of years without any issue. > > Also, why do you read only 64 bytes from this file? According to your debug > trace, this read was successful, and then no error is shown. So, if it hangs, it > might be that the read data is corrupted, or that the image is not fully read > because of this 64-byte limit. I've bisected this twice and I've come to a conclusion. It is this patch that makes my problem show up, but it's not this patch at fault. What I'm going to go with is that the "am33xx: Remove redundant timer config" patch that I forgot in my last round of pull requests needs to come in as with that set of patches applied locally to top of tree the platform is fine again. -- Tom ^ permalink raw reply [flat|nested] 10+ messages in thread
* [U-Boot] [PATCH v2 7/8] FAT: Simplify get_contents 2012-09-04 20:50 ` Tom Rini 2012-09-04 22:07 ` Benoît Thébaudeau @ 2012-09-04 22:08 ` Wolfgang Denk 1 sibling, 0 replies; 10+ messages in thread From: Wolfgang Denk @ 2012-09-04 22:08 UTC (permalink / raw) To: u-boot Dear Tom, In message <20120904205034.GC23991@bill-the-cat> you wrote: > > OK, this change is NOT equivalent code. My platforms now hang thusly argh... :-( > This is all fine in full U-Boot. What shall we do? Revert? Fix? Best regards, Wolfgang Denk -- DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de Why is an average signature file longer than an average Perl script?? ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2012-09-05 0:03 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2012-07-19 22:02 [U-Boot] [PATCH 6/7] FAT: Simplify get_contents Benoît Thébaudeau 2012-07-20 13:21 ` [U-Boot] [PATCH v2 7/8] " Benoît Thébaudeau 2012-09-02 15:25 ` Wolfgang Denk 2012-09-04 20:50 ` Tom Rini 2012-09-04 22:07 ` Benoît Thébaudeau 2012-09-04 22:10 ` Tom Rini 2012-09-04 22:41 ` Tom Rini 2012-09-04 22:53 ` Benoît Thébaudeau 2012-09-05 0:03 ` Tom Rini 2012-09-04 22:08 ` Wolfgang Denk
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox