On Tue, 24 Feb 2009 17:22:16 +0100, Wolfram Sang wrote: > Signed-off-by: Wolfram Sang <w.sang@xxxxxxxxxxxxxx> > --- > drivers/i2c/busses/i2c-pca-platform.c | 19 ++++++++++++------- > 1 files changed, 12 insertions(+), 7 deletions(-) > > diff --git a/drivers/i2c/busses/i2c-pca-platform.c b/drivers/i2c/busses/i2c-pca-platform.c > index df5e593..7b23891 100644 > --- a/drivers/i2c/busses/i2c-pca-platform.c > +++ b/drivers/i2c/busses/i2c-pca-platform.c > @@ -177,10 +177,20 @@ static int __devinit i2c_pca_pf_probe(struct platform_device *pdev) > (unsigned long) res->start); > i2c->adap.algo_data = &i2c->algo_data; > i2c->adap.dev.parent = &pdev->dev; > - i2c->adap.timeout = platform_data->timeout; > > - i2c->algo_data.i2c_clock = platform_data->i2c_clock_speed; > + if (platform_data) { > + i2c->adap.timeout = platform_data->timeout; > + i2c->algo_data.i2c_clock = platform_data->i2c_clock_speed; > + i2c->gpio = platform_data->gpio; > + } else { > + i2c->adap.timeout = HZ; > + i2c->algo_data.i2c_clock = 59000; > + i2c->gpio = -1; > + } > + > i2c->algo_data.data = i2c; > + i2c->algo_data.wait_for_completion = i2c_pca_pf_waitforcompletion; > + i2c->algo_data.reset_chip = i2c_pca_pf_dummyreset; > > switch (res->flags & IORESOURCE_MEM_TYPE_MASK) { > case IORESOURCE_MEM_32BIT: > @@ -198,11 +208,6 @@ static int __devinit i2c_pca_pf_probe(struct platform_device *pdev) > break; > } > > - i2c->algo_data.wait_for_completion = i2c_pca_pf_waitforcompletion; > - > - i2c->gpio = platform_data->gpio; > - i2c->algo_data.reset_chip = i2c_pca_pf_dummyreset; > - > /* Use gpio_is_valid() when in mainline */ > if (i2c->gpio > -1) { > ret = gpio_request(i2c->gpio, i2c->adap.name); Applied, thanks. -- Jean Delvare -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html