Dear Stefano Babic,
On 30/03/2012 17:28, Marek Vasut wrote:
However, Albert has sent a report
http://www.mail-archive.com/u-boot@lists.denx.de/msg80566.html
a none of these boards was broken. But I see now that other boards are affected (the mx28evk does not compile due to missing CONFIG_APBH_DMA).
Fabio, can you fix please? This is trivial.
Albert, are these patches part of your pull-request to Wolfgang ?
I believe the pullRQ isn't cooked yet. The fix for this issue right now would be to merge a patch that implements blank dcache-management functions for arm1136 -- like is in AG's patch. So I'm all for merging AG's patch into AA's tree.
I am testing Anatolij's patch on mx35pdk.
TFTP from server 192.168.2.14; our IP address is 192.168.2.97 Filename 'mx35pdk/uImage'. Load address: 0x80800000 Loading: Misaligned cache operation [8fe726e8, 8fe72728]
However, data is correctly loaded. I will check mmc on the "flea" board.
You get this from FEC? Will have to recheck, this is weird, will recheck once I get back to my m28.
It's a good thing this stirred a wave of response including patches. We now know very well which boards are maintained ;-)
Also, once any such breaking patch lands into mainline, we'll know in _less_than_24_hours_ that something got broken. (this is handled by DENX CI machine).
Well, this is a good thing - my worries are about that patches for imx were already merged, and
Finally, we can't really run physical (HW) tests indeed, but did we ever run physical tests with each and every patch?
Not every patch, but a patchset that can have influence on several SOCs, yes.
Agreed.
(and to conclude this -- these patches were tested on M28 and MX6Q-board)
mmmh...I suppose the following patches must be merged, too (I had merged into u-boot-next, really):
Author: Eric Nelson eric.nelson@boundarydevices.com Date: Sun Mar 4 11:47:37 2012 +0000
i.MX6: define CACHELINE_SIZEand also even if not mandatory: commit 1b2150b0770d8d019a41993d8692e4a29bf70a9e Author: Eric Nelson eric.nelson@boundarydevices.com Date: Sun Mar 4 11:47:38 2012 +0000
i.MX6: implement enable_caches() disabled by default until drivers are fixed Signed-off-by: Eric Nelson <eric.nelson@boundarydevices.com> Acked-by: Marek Vasut <marex@denx.de>
They weren't already? Hm.
Best regards, Stefano Babic
Best regards, Marek Vasut