Message ID | CA+m5__+d8WbPe0wtZVw4VFV5L2qbLvD-TLzXP3FmXgdukT8vAg@mail.gmail.com |
---|---|
State | Accepted |
Delegated to: | Albert ARIBAUD |
Headers | show |
Hi Tom, On Fri, 16 Nov 2012 10:14:46 -0700, Tom Warren <twarren.nvidia@gmail.com> wrote: > Albert, > > Please pull u-boot-tegra/master into ARM/master. Thanks! > > ./MAKEALL -a arm completes w/o any new errors. > checkpatch is clean, also. I see three boards failing which do build on current u-boot-arm/master: palmld palmtc zipitz2 All have the same error: /home/albert/src/u-boot-arm/common/lcd.c:125: undefined reference to `flush_dcache_range' git bisect run ./MAKEALL zipitz2 traces the problem to commit 9c9e9b0d... "lcd: Add support for flushing LCD fb from dcache after update" from Simon Glass (already Cc:). Simon, can you look into this? Tom, can you provide a fixed pull request once the issue is fixed? Amicalement,
Albert, A fix was found and posted awhile back by Simon (195662 on PatchWork - it's assigned to you as delegate). Marek's last comment was 'Just apply this', so I suppose that's his version of an Acked-by. I don't carry that fix in my tree since it's not specifically Tegra-related. I can add it and resend a pull request, or you can apply it to ARM/master and I can rebase on that (though I don't think that'll change my pull request much, if at all). Let me know, Tom On Fri, Nov 16, 2012 at 2:27 PM, Albert ARIBAUD <albert.u.boot@aribaud.net> wrote: > Hi Tom, > > On Fri, 16 Nov 2012 10:14:46 -0700, Tom Warren > <twarren.nvidia@gmail.com> wrote: > >> Albert, >> >> Please pull u-boot-tegra/master into ARM/master. Thanks! >> >> ./MAKEALL -a arm completes w/o any new errors. >> checkpatch is clean, also. > > I see three boards failing which do build on current u-boot-arm/master: > > palmld palmtc zipitz2 > > All have the same error: > > /home/albert/src/u-boot-arm/common/lcd.c:125: undefined > reference to `flush_dcache_range' > > git bisect run ./MAKEALL zipitz2 traces the problem to commit > 9c9e9b0d... "lcd: Add support for flushing LCD fb from dcache after > update" from Simon Glass (already Cc:). > > Simon, can you look into this? > > Tom, can you provide a fixed pull request once the issue is fixed? > > Amicalement, > -- > Albert.
Hi Tom, On Fri, 16 Nov 2012 14:45:49 -0700, Tom Warren <twarren.nvidia@gmail.com> wrote: > Albert, > > A fix was found and posted awhile back by Simon (195662 on PatchWork - > it's assigned to you as delegate). Marek's last comment was 'Just > apply this', so I suppose that's his version of an Acked-by. I don't > carry that fix in my tree since it's not specifically Tegra-related. > I can add it and resend a pull request, or you can apply it to > ARM/master and I can rebase on that (though I don't think that'll > change my pull request much, if at all). > > Let me know, Ideally, I would like the tegra master branch to be reworked with the fix applied so that no commit fails building. > Tom Amicalement,