We are deprecating calls which return NULL in favor of new variants which return an ERR_PTR. Only build tested.
Wolfram Sang (1): video: backlight: tosa_lcd: convert to use i2c_new_client_device()
drivers/video/backlight/tosa_lcd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Move away from the deprecated API and return the shiny new ERRPTR where useful.
Signed-off-by: Wolfram Sang wsa+renesas@sang-engineering.com --- drivers/video/backlight/tosa_lcd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/video/backlight/tosa_lcd.c b/drivers/video/backlight/tosa_lcd.c index e8ab583e5098..113116d3585c 100644 --- a/drivers/video/backlight/tosa_lcd.c +++ b/drivers/video/backlight/tosa_lcd.c @@ -107,7 +107,7 @@ static void tosa_lcd_tg_on(struct tosa_lcd_data *data) /* TG LCD GVSS */ tosa_tg_send(spi, TG_PINICTL, 0x0);
- if (!data->i2c) { + if (IS_ERR_OR_NULL(data->i2c)) { /* * after the pannel is powered up the first time, * we can access the i2c bus so probe for the DAC @@ -119,7 +119,7 @@ static void tosa_lcd_tg_on(struct tosa_lcd_data *data) .addr = DAC_BASE, .platform_data = data->spi, }; - data->i2c = i2c_new_device(adap, &info); + data->i2c = i2c_new_client_device(adap, &info); } }
On Thu, Mar 26, 2020 at 10:09:59PM +0100, Wolfram Sang wrote:
Move away from the deprecated API and return the shiny new ERRPTR where useful.
Signed-off-by: Wolfram Sang wsa+renesas@sang-engineering.com
Reviewed-by: Daniel Thompson daniel.thompson@linaro.org
drivers/video/backlight/tosa_lcd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/video/backlight/tosa_lcd.c b/drivers/video/backlight/tosa_lcd.c index e8ab583e5098..113116d3585c 100644 --- a/drivers/video/backlight/tosa_lcd.c +++ b/drivers/video/backlight/tosa_lcd.c @@ -107,7 +107,7 @@ static void tosa_lcd_tg_on(struct tosa_lcd_data *data) /* TG LCD GVSS */ tosa_tg_send(spi, TG_PINICTL, 0x0);
- if (!data->i2c) {
- if (IS_ERR_OR_NULL(data->i2c)) { /*
- after the pannel is powered up the first time,
- we can access the i2c bus so probe for the DAC
@@ -119,7 +119,7 @@ static void tosa_lcd_tg_on(struct tosa_lcd_data *data) .addr = DAC_BASE, .platform_data = data->spi, };
data->i2c = i2c_new_device(adap, &info);
data->i2c = i2c_new_client_device(adap, &info);}
-- 2.20.1
On Thu, 26 Mar 2020, Wolfram Sang wrote:
Move away from the deprecated API and return the shiny new ERRPTR where useful.
Signed-off-by: Wolfram Sang wsa+renesas@sang-engineering.com
drivers/video/backlight/tosa_lcd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Applied, thanks.
On Fri, Apr 17, 2020 at 11:14:46AM +0100, Lee Jones wrote:
On Thu, 26 Mar 2020, Wolfram Sang wrote:
Move away from the deprecated API and return the shiny new ERRPTR where useful.
Signed-off-by: Wolfram Sang wsa+renesas@sang-engineering.com
drivers/video/backlight/tosa_lcd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Applied, thanks.
Thanks! I don't see it in -next, though?
On Tue, 12 May 2020, Wolfram Sang wrote:
On Fri, Apr 17, 2020 at 11:14:46AM +0100, Lee Jones wrote:
On Thu, 26 Mar 2020, Wolfram Sang wrote:
Move away from the deprecated API and return the shiny new ERRPTR where useful.
Signed-off-by: Wolfram Sang wsa+renesas@sang-engineering.com
drivers/video/backlight/tosa_lcd.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
Applied, thanks.
Thanks! I don't see it in -next, though?
Just pushed. Should be there soon.
dri-devel@lists.freedesktop.org
-
Daniel Thompson -
Lee Jones -
Wolfram Sang