lkml.org 
[lkml]   [2019]   [Nov]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [Patch v2 12/20] media: ti-vpe: cal: Add DRA76x support
Benoit Parrot <bparrot@ti.com> wrote on Wed [2019-Nov-06 14:58:39 -0600]:
> Sakari Ailus <sakari.ailus@iki.fi> wrote on Wed [2019-Nov-06 10:57:09 +0200]:
> > Hi Benoit,
> >
> > On Mon, Nov 04, 2019 at 01:31:32PM -0600, Benoit Parrot wrote:
> >
> > ...
> >
> > > +static struct cal_data dra76x_cal_data = {
> >
> > const?
>
> Hmm, most likely.

Well, it turns out they cannot be constified because we need to be able to
update the register offset fir the regmap used for the syscon object.
So I'll leave them as is.

Benoit

>
> >
> > > + .csi2_phy_core = dra76x_cal_csi_phy,
> > > + .num_csi2_phy = ARRAY_SIZE(dra76x_cal_csi_phy),
> > > +
> >
> > This newline seems extra.
> >
> > > + .flags = 0,
> >
> > And flags will be zero in any case, as one more struct members are assigned
> > to.
>
> Is this guaranteed or compiler version dependent?
>
> >
> > > +};
> >
> > --
> > Regards,
> >
> > Sakari Ailus

\
 
 \ /
  Last update: 2019-11-12 14:26    [W:0.280 / U:0.096 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site