From mboxrd@z Thu Jan 1 00:00:00 1970 From: Stefan Roese Date: Tue, 22 Dec 2015 09:08:45 +0100 Subject: [U-Boot] [PATCH v2 4/8] axp: Fix debugging support in DDR3 write leveling In-Reply-To: <20151221232445.A92BA6121B@mail.nwl.cc> References: <1450740358-5014-1-git-send-email-phil@nwl.cc> <20151221232445.A92BA6121B@mail.nwl.cc> Message-ID: <5679050D.7070407@denx.de> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On 22.12.2015 00:25, Phil Sutter wrote: > If MV_DEBUG_WL is defined, DEBUG_WL_S and DEBUG_WL_D macros are missing. > In addition to that, get rid of debug output printing non-existent > counter variable. > > Signed-off-by: Phil Sutter > --- > drivers/ddr/marvell/axp/ddr3_write_leveling.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/ddr/marvell/axp/ddr3_write_leveling.c b/drivers/ddr/marvell/axp/ddr3_write_leveling.c > index df3a3df..da384f3 100644 > --- a/drivers/ddr/marvell/axp/ddr3_write_leveling.c > +++ b/drivers/ddr/marvell/axp/ddr3_write_leveling.c > @@ -22,6 +22,8 @@ > DEBUG_WL_FULL_S(s); DEBUG_WL_FULL_D(d, l); DEBUG_WL_FULL_S("\n") > > #ifdef MV_DEBUG_WL > +#define DEBUG_WL_S(s) puts(s) > +#define DEBUG_WL_D(d, l) printf("%x", d) > #define DEBUG_RL_S(s) \ > debug_cond(ddr3_get_log_level() >= MV_LOG_LEVEL_2, "%s", s) > #define DEBUG_RL_D(d, l) \ > @@ -1229,8 +1231,6 @@ static int ddr3_write_leveling_single_cs(u32 cs, u32 freq, int ratio_2to1, > DEBUG_WL_FULL_D((u32) phase, 1); > DEBUG_WL_FULL_S(", Delay = "); > DEBUG_WL_FULL_D((u32) delay, 1); > - DEBUG_WL_FULL_S(", Counter = "); > - DEBUG_WL_FULL_D((u32) i, 1); > DEBUG_WL_FULL_S("\n"); > > /* Drive DQS high for one cycle - All data PUPs */ > Reviewed-by: Stefan Roese Thanks, Stefan