On Fri, Mar 01, 2013 at 05:06:48PM +0100, Wolfgang Denk wrote: > Dear Jagannadha Sutradharudu Teki, > > In message <10597224-d520-4a3f-8185-5de018ee5...@tx2ehsmhs026.ehs.local> you > wrote: > > This patch provides a support to decode the mtest start > > and end values from fdt config node. > > > > Signed-off-by: Jagannadha Sutradharudu Teki <jaga...@xilinx.com> > > Tested-by: Jagannadha Sutradharudu Teki <jaga...@xilinx.com> > > --- > > common/cmd_mem.c | 6 ++++-- > > common/main.c | 18 ++++++++++++++++++ > > 2 files changed, 22 insertions(+), 2 deletions(-) > > You are adding code here which may bot be user and/or wanted by the > majority of boards, so please make it configureable (and document the > new config option).
In addition, if we're going to start whacking around in here, can we just make this function bail if not passed a start and end address? I'm a little torn since on the one hand this is a known function, but on the other it's not something one would expect to be used outside of testing environments. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot
