[PATCH 3/5] staging: kpc2000: added designated initializers to two structs.

Jeremy Sowden jeremy at azazel.net
Wed May 15 11:19:13 UTC 2019


On 2019-05-15, at 14:11:53 +0300, Dan Carpenter wrote:
> On Wed, May 15, 2019 at 02:09:49PM +0300, Dan Carpenter wrote:
> > On Wed, May 15, 2019 at 11:34:52AM +0100, Jeremy Sowden wrote:
> > > diff --git a/drivers/staging/kpc2000/kpc2000/cell_probe.c
> > > b/drivers/staging/kpc2000/kpc2000/cell_probe.c
> > > index 30e6f176ddfa..9cb745f4323a 100644
> > > --- a/drivers/staging/kpc2000/kpc2000/cell_probe.c
> > > +++ b/drivers/staging/kpc2000/kpc2000/cell_probe.c
> > > @@ -94,7 +94,7 @@ void parse_core_table_entry(struct core_table_entry *cte, const u64 read_val, co
> > >  static int probe_core_basic(unsigned int core_num, struct kp2000_device *pcard,
> > >  			    char *name, const struct core_table_entry cte)
> > >  {
> > > -    struct mfd_cell  cell = {0};
> > > +    struct mfd_cell  cell = { .id = core_num, .name = name };
> > >      struct resource  resources[2];
> > >
> > >      struct kpc_core_device_platdata  core_pdata = {
> > > @@ -315,7 +315,7 @@ static int probe_core_uio(unsigned int core_num, struct kp2000_device *pcard,
> > >
> > >  static int  create_dma_engine_core(struct kp2000_device *pcard, size_t engine_regs_offset, int engine_num, int irq_num)
> > >  {
> > > -    struct mfd_cell  cell = {0};
> > > +    struct mfd_cell  cell = { .id = engine_num };
> > >      struct resource  resources[2];
> > >
> >
> > These changes make no sense because we just write over it later.
> >
> > Maybe you're going to fix it up later in the patch series, perhaps
> > but that's not how it's done.  Each patch should do "one thing", not
> > "half and thing and then half a thing later in the series possibly
> > (I am reviewing the patches in order so I don't know)".
> >
>
> I've finished reviewing the series and we never complete the other
> half of this patch.

Stupid mistake.  I've sent out v2.

Thanks for the review.

J.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://driverdev.linuxdriverproject.org/pipermail/driverdev-devel/attachments/20190515/b64cf144/attachment-0001.asc>


More information about the devel mailing list