lkml.org 
[lkml]   [2020]   [Apr]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRE: [RESEND PATCH v2 1/2] mfd: da9063: Fix revision handling to correctly select reg tables
Date
On 20 April 2020 08:32, Lee Jones wrote:

> >
> > > > > > + return -EINVAL;
> > > > >
> > > > > Do you want to fail silently here?
> > > >
> > > > Well an error message is printed in the calling code, so didn't feel like it
> > > > was necessary to have additional debug here. Felt like bloat.
> > >
> > > As a user, I would prefer a more specific reason.
> > >
> > > Thus, I would provide an error message here and omit the generic one.
> >
> > I can update although I'll of course then need to do similar messages for the
> > other error legs of this function. FWIW, as this is only being called once in
> > the same file this error leg of code currently can never occur.
>
> As a tiny improvement, it's not a deal breaker. If it's too much
> work, you can either submit a subsequent patch or omit it completely.

Taken care of in the v3 patch set I submitted Friday. Hopefully that covers
everything. :)
\
 
 \ /
  Last update: 2020-04-20 10:15    [W:0.063 / U:0.140 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site