lkml.org 
[lkml]   [2020]   [Dec]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
SubjectRE: [PATCH v2 03/10] regulator: bd9571mwv: rid of using struct bd9571mwv
Date
Hi Geert-san, Matti-san,

> From: Geert Uytterhoeven, Sent: Wednesday, December 16, 2020 1:13 AM
> On Tue, Dec 15, 2020 at 5:02 PM Geert Uytterhoeven <geert@linux-m68k.org> wrote:
> > On Fri, Dec 11, 2020 at 3:03 PM Vaittinen, Matti
> > <Matti.Vaittinen@fi.rohmeurope.com> wrote:
> > > On Fri, 2020-12-11 at 20:27 +0900, Yoshihiro Shimoda wrote:
<snip>
> > > > --- a/drivers/regulator/bd9571mwv-regulator.c
> > > > +++ b/drivers/regulator/bd9571mwv-regulator.c
> > > > @@ -17,7 +17,7 @@
> > > > #include <linux/mfd/bd9571mwv.h>
> > > >
> > > > struct bd9571mwv_reg {
> > > > - struct bd9571mwv *bd;
> > > > + struct regmap *regmap;
> > >
> > > As a 'nit':
> > > I might consider adding the dev pointer here to avoid extra argument
> > > with all the bkup_mode functions below. (just pass this struct and
> > > mode). But that's only my preference - feel free to ignore this comment
> > > if patch is Ok to Mark, Marek & Others :)
> >
> > Struct regmap already contains a struct device pointer, but that's internal
> > to regmap.
> >
> > Perhaps adding a regmap_device() helper to retrieve the device pointer
> > might be worthwhile?
>
> -EEXISTS ;-)
>
> struct device *regmap_get_device(struct regmap *map)

Thank you for finding this. I'll fix this patch.

Best regards,
Yoshihiro Shimoda

\
 
 \ /
  Last update: 2020-12-16 03:16    [W:0.067 / U:0.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site