[PATCH 2/4] i2c: i2c-ocores: Use reg-shift property

Peter Korsgaard jacmet at sunsite.dk
Fri Jun 8 21:41:36 EST 2012


>>>>> "J" == Jayachandran C <jayachandranc at netlogicmicro.com> writes:

 J> From: Ganesan Ramalingam <ganesanr at broadcom.com>
 J> Deprecate 'regstep' property and use the standard 'reg-shift' property
 J> for register offset shifts. 'regstep' will still be supported as an
 J> optional property, but will give a warning when used.

..
 
 J>  struct ocores_i2c_platform_data {
 J> -	u32 regstep;   /* distance between registers */
 J> -	u32 clock_khz; /* input clock in kHz */
 J> -	u8 num_devices; /* number of devices in the devices list */
 J> +	u32 reg_shift;		/* register offset shift value */
 J> +	u32 clock_khz;		/* input clock in kHz */
 J> +	u8 num_devices;		/* number of devices in the devices list */
 J>  	struct i2c_board_info const *devices; /* devices connected to the bus */
 J>  };

Why not just keep this change to the dt bindings, instead of risking
breaking stuff for platform drivers as well? There's no conceptual
reason why reg_shift is any better than regstep.

-- 
Bye, Peter Korsgaard


More information about the devicetree-discuss mailing list