Hi Haojian, On Mon, Jan 25, 2010 at 06:55:50AM -0500, Haojian Zhuang wrote: > +struct max8925_rtc_info { > + struct rtc_device *rtc_dev; > + struct max8925_chip *chip; > + struct i2c_client *i2c; I dont think you need this pointer here, right ? Cheers, Samuel. -- Intel Open Source Technology Centre http://oss.intel.com/