Re: [PATCH v2 8/9] hwmon: (it87) Manage device specific features with table

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Mon, 29 Oct 2012 06:53:47 -0700, Guenter Roeck wrote:
> On Mon, Oct 29, 2012 at 01:39:58PM +0100, Jean Delvare wrote:
> > On Sun, 28 Oct 2012 11:20:00 -0700, Guenter Roeck wrote:
> > > diff --git a/drivers/hwmon/it87.c b/drivers/hwmon/it87.c
> > > index 340c4ea..f3f3e79 100644
> > > --- a/drivers/hwmon/it87.c
> > > +++ b/drivers/hwmon/it87.c
> > > @@ -228,6 +228,59 @@ static const u8 IT87_REG_TEMP_OFFSET[]	= { 0x56, 0x57, 0x59 };
> > >  #define IT87_REG_AUTO_TEMP(nr, i) (0x60 + (nr) * 8 + (i))
> > >  #define IT87_REG_AUTO_PWM(nr, i)  (0x65 + (nr) * 8 + (i))
> > >  
> > > +struct it87_devices {
> > > +	const char * const name;
> > 
> > What is the second const good for?
>
> 	const char (equivalent to char const)	The content is const
> 	* const					The pointer is const

I know that. My question was: what is the point of making "name" a
const pointer here? I don't think it adds any value.

-- 
Jean Delvare

_______________________________________________
lm-sensors mailing list
lm-sensors@xxxxxxxxxxxxxx
http://lists.lm-sensors.org/mailman/listinfo/lm-sensors


[Index of Archives]     [Linux Kernel]     [Linux Hardware Monitoring]     [Linux USB Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Yosemite Backpacking]

  Powered by Linux