Re: [PATCH 2/4] clk/Renesas-MSTP: Delete an error message for a failed memory allocation

Wolfram Sang
2016-09-15T19:41:22Z

[-- Attachment #1: Type: text/plain, Size: 868 bytes --]

On Thu, Sep 15, 2016 at 10:13:17PM +0300, Laurent Pinchart wrote:
> On Thursday 15 Sep 2016 21:07:17 Geert Uytterhoeven wrote:
> > On Wed, Sep 14, 2016 at 10:01 PM, SF Markus Elfring wrote:
> > > From: Markus Elfring <elfring@users.sourceforge.net>
> > > Date: Wed, 14 Sep 2016 21:17:18 +0200
> > >
> > > Omit an extra message for a memory allocation failure in this function.
> > >
> > > Link:
> > > http://events.linuxfoundation.org/sites/events/files/slides/LCJ16-Refacto
> > > r_Strings-WSang_0.pdf
> > >
> > > Signed-off-by: Markus Elfring <elfring@users.sourceforge.net>
> >
> > Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
>
> There are two other memory allocation failures printed by drivers in the same
> directory that, you'll get my ack if you extend this patch to remove the three
> messages in one go.

I agree.

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]