Re: 2.6.4-rc2-mm1: IPMI_SMB doesnt compile

[Adrian Bunk]

On Wed, Mar 10, 2004 at 10:01:07AM -0600, Corey Minyard wrote:
>...
> I have included a patch from Todd Davis at Intel that adds this function 
> to the I2C driver.  I believe Todd has been working on getting this in 
> through the I2C driver writers, although the patch is fairly non-intrusive.
> 
> However, I have no real way to test this patch.
>...

I can only confirm that it fixes the compilation...


The patch to i2c-core.c is strange:


> --- linux-v31/drivers/i2c/i2c-core.c	2004-02-19 19:31:07.000000000 -0600
> +++ linux/drivers/i2c/i2c-core.c	2004-03-10 09:48:08.000000000 -0600
> @@ -1256,6 +1256,12 @@
>  	return (func & adap_func) == func;
>  }
>  
> +int i2c_spin_delay;
> +void i2c_set_spin_delay(int val)
> +{
> +	i2c_spin_delay = val;
> +}
> +
>  EXPORT_SYMBOL(i2c_add_adapter);
>  EXPORT_SYMBOL(i2c_del_adapter);
>  EXPORT_SYMBOL(i2c_add_driver);
> @@ -1292,6 +1298,8 @@
>  
>  EXPORT_SYMBOL(i2c_get_functionality);
>  EXPORT_SYMBOL(i2c_check_functionality);
> +EXPORT_SYMBOL(i2c_set_spin_delay);
> +EXPORT_SYMBOL(i2c_spin_delay);
>  
>  MODULE_AUTHOR("Simon G. Vogl <simon@xxxxxxxxxxxxxxxxx>");
>  MODULE_DESCRIPTION("I2C-Bus main module");
>...


You can either add get/set functions and export them (more an OO 
paradigm) or export the variable.

If you export the variable, it's quite useless to add such a set 
function since everyone can set the variable directly.


cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/