On 01/21/2014 09:34 AM, York Sun wrote: > On 01/21/2014 09:29 AM, Scott Wood wrote: >> On Tue, 2014-01-21 at 10:14 +0100, Wolfgang Denk wrote: >>> Dear York, >>> >>> In message <f1d691e4-180a-4a2d-be07-812547d46...@freescale.com> you wrote: >>>> >>>>> On second thought, I also think we should avoid solutions where the >>>>> BE/LE test has to be done for each and every I/O accessor call again >>>>> and again. We should rather do this just once, and for example set >>>>> function pointers as needed (hoping that this driver will only be >>>>> needed after relocation, so we have writable data segment). >>>> >>>> I like the idea of setting it just once, but I don't see how to >>>> implement it. A pointer is probably not the solution, because we do need >>>> some drivers before relocation. >>> >>> "some drivers before relocation" - how many which are these? > > IFC, DDR, I2C (only 32-bit controller is concerned), GUT > >>> >>> Also, is it really necessary to make the decision about endianess at >>> runtime? We don't have multi-board support in U-Boot yet, so when you >>> build an image you know exactly which SoC you are building for, so you >>> should be able to make the selection at compile time? >> >> It is done at compile time in this patch. >> > > No. It is not necessary to do it at run time. It would be easier to use a > switch > to decide at compiling time. It does involve many changes to implement the > wrapper. >
Acked-by: York Sun <york...@freescale.com> Do we all agree on this patch? York _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
