On Wed, Oct 07, 2015 at 03:12:34PM +0300, Sergei Shtylyov wrote:
> Hello.
>
> On 10/7/2015 11:16 AM, Geert Uytterhoeven wrote:
>
> >Since commit 4baadb9e05c68962 ("ARM: shmobile: r8a7778: remove obsolete
> >setup code"), Renesas R-Car SoCs are only supported in generic DT-only
> >ARM multi-platform builds. The driver doesn't need to use platform data
> >anymore, hence remove platform data configuration.
> >
> >Signed-off-by: Geert Uytterhoeven <[email protected]>
> >---
> >Commit 4baadb9e05c68962 is now in arm-soc/for-next.
> >---
> > drivers/i2c/busses/i2c-rcar.c | 19 +------------------
> > include/linux/i2c/i2c-rcar.h | 10 ----------
> > 2 files changed, 1 insertion(+), 28 deletions(-)
> > delete mode 100644 include/linux/i2c/i2c-rcar.h
> >
> >diff --git a/drivers/i2c/busses/i2c-rcar.c b/drivers/i2c/busses/i2c-rcar.c
> >index 3e125654bb28..cd38406db074 100644
> >--- a/drivers/i2c/busses/i2c-rcar.c
> >+++ b/drivers/i2c/busses/i2c-rcar.c
> [...]
> >@@ -654,14 +652,8 @@ static int rcar_i2c_probe(struct platform_device *pdev)
> >
> > bus_speed = 100000; /* default 100 kHz */
> > ret = of_property_read_u32(dev->of_node, "clock-frequency", &bus_speed);
>
> The result can be ignored now.Yup, my code checkers found this, too ;) Fixed it here. > > >- if (ret < 0 && pdata && pdata->bus_speed) > >- bus_speed = pdata->bus_speed; > > > >- if (pdev->dev.of_node) > >- priv->devtype = (long)of_match_device(rcar_i2c_dt_ids, > >- dev)->data; > >- else > >- priv->devtype = platform_get_device_id(pdev)->driver_data; > >+ priv->devtype = (long)of_match_device(rcar_i2c_dt_ids, dev)->data; I changed this to the enum type of devtype Other than that: Applied to for-next, thanks! > > > > ret = rcar_i2c_clock_calculate(priv, bus_speed, dev); > > if (ret < 0) > [...] > > MBR, Sergei >
signature.asc
Description: Digital signature
