Re: [PATCH] edac: make inst_grp_ops const

[Doug Thompson <norsk5@yahoo.com>]

Ignore that prior ack by me, I thought it was an i7core_edac module, not core EDAC modules.

I will take it myself and push it upstream

doug t


--- On Tue, 8/31/10, Stephen Hemminger <shemminger@vyatta.com> wrote:

> From: Stephen Hemminger <shemminger@vyatta.com>
> Subject: [PATCH] edac: make inst_grp_ops const
> To: "Doug Thompson" <dougthompson@xmission.com>, "Mauro Carvalho Chehab" <mchehab@redhat.com>, 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 <shemminger@vyatta.com>
> 
> --- a/drivers/edac/edac_mc_sysfs.c   
> 2010-08-31 15:10:29.692381171 -0700
> +++ b/drivers/edac/edac_mc_sysfs.c   
> 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 = {
> +static const struct sysfs_ops inst_grp_ops = {
>      .show = inst_grp_show,
>      .store = 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