[PATCH] [OneNAND] Standardising prints in onenand_base.c

Artem Bityutskiy dedekind1 at gmail.com
Mon Oct 5 03:11:50 EDT 2009


On Mon, 2009-10-05 at 10:04 +0300, Artem Bityutskiy wrote:
> On Thu, 2009-10-01 at 13:08 +0530, Amul Kumar Saha wrote:
> > Note: Reposting with corrected Signed-Off
> > 
> > This patch resolves all the prints present in onenand_base.c
> > Primarily, it replaces the hard-coded function names in the prints,
> > and makes use of __func__.
> > 
> > Signed-off-by: Amul Kumar Saha <amul.saha at samsung.com>
> > ---
> >  onenand_base.c |  172 +++++++++++++++++++++++++++++++++++----------------------
> >  1 file changed, 106 insertions(+), 66 deletions(-)
> 
> David decided to ignore the discussion and pushed your first patch
> instead of this one already.

Oh, David actually took the right patch, sorry. You decided to re-send
the old patch and did not address the nitpicks I pointed, e.g., there is
still:

DEBUG(MTD_DEBUG_LEVEL3, "onenand_erase: start = 0x%012llx, len = %llu
\n", (unsigned long long) instr->addr, (unsigned long long) instr->len);

Well, not big deal, but would be nice to have a follow up patch.

-- 
Best Regards,
Artem Bityutskiy (Артём Битюцкий)




More information about the linux-mtd mailing list