Re: [PATCH V6] UBI: Extend UBI layer debug/messaging capabilities

From: Joe Perches
Date: Tue Nov 11 2014 - 03:34:55 EST


On Tue, 2014-11-11 at 10:24 +0200, Tanya Brokhman wrote:
> On 11/10/2014 7:57 PM, Joe Perches wrote:
> > On Mon, 2014-11-10 at 18:37 +0100, Richard Weinberger wrote:
> >> Am 03.11.2014 um 14:58 schrieb Tanya Brokhman:
> >>> If there is more then one UBI device mounted, there is no way to
> >>> distinguish between messages from different UBI devices.
> >>> Add device number to all ubi layer message types.
> >
> > Adding "error" and "warning" to the message logs is
> > duplicative to the KERN_<LEVEL> logging information.
> >
> >>> Changes from V5:
> >>> - Added ptr verification @ ubi_err/ubi_msg/ubi_warn
> >>> Removed extra printing of ubi number
> >>> Removed new messages.
> >
> > Did you all ever look at what I posted?
>
> I did. Its not my place to re-post your change in my patch. I personally
> prefer it the way I've done it but it's just a matter of opinion and
> personal preference.

Don't forget object code size.


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/