From mboxrd@z Thu Jan 1 00:00:00 1970 From: Doug Thompson Subject: Re: [PATCH] edac: make inst_grp_ops const Date: Tue, 31 Aug 2010 16:00:20 -0700 (PDT) Message-ID: <569266.95060.qm@web50108.mail.re2.yahoo.com> References: <20100831152402.22b5e897@nehalam> Mime-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: <20100831152402.22b5e897@nehalam> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: bluesmoke-devel-bounces@lists.sourceforge.net To: Doug Thompson , Mauro Carvalho Chehab , bluesmoke-devel@lists.sourceforge.net, Stephen Hemminger List-Id: edac.vger.kernel.org Ignore that prior ack by me, I thought it was an i7core_edac module, not co= re EDAC modules. I will take it myself and push it upstream doug t --- On Tue, 8/31/10, Stephen Hemminger wrote: > From: Stephen Hemminger > Subject: [PATCH] edac: make inst_grp_ops const > To: "Doug Thompson" , "Mauro Carvalho Chehab" = , bluesmoke-devel@lists.sourceforge.net > Date: Tuesday, August 31, 2010, 4:24 PM > This sysfs_ops should be const like > all the others in the kernel. > = > Signed-off-by: Stephen Hemminger > = > --- a/drivers/edac/edac_mc_sysfs.c=A0=A0=A0 > 2010-08-31 15:10:29.692381171 -0700 > +++ b/drivers/edac/edac_mc_sysfs.c=A0=A0=A0 > 2010-08-31 15:11:24.072859199 -0700 > @@ -764,7 +764,7 @@ static void > edac_inst_grp_release(struct > } > = > /* Intermediate show/store table */ > -static struct sysfs_ops inst_grp_ops =3D { > +static const struct sysfs_ops inst_grp_ops =3D { > =A0=A0=A0 .show =3D inst_grp_show, > =A0=A0=A0 .store =3D inst_grp_store > }; > = ---------------------------------------------------------------------------= --- This SF.net Dev2Dev email is sponsored by: Show off your parallel programming skills. Enter the Intel(R) Threading Challenge 2010. http://p.sf.net/sfu/intel-thread-sfd