backlight: hx8357: Utilise temporary variable for struct device
authorAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Thu, 1 Feb 2024 14:47:45 +0000 (16:47 +0200)
committerLee Jones <lee@kernel.org>
Thu, 7 Mar 2024 09:03:28 +0000 (09:03 +0000)
We have a temporary variable to keep pointer to struct device.
Utilise it inside the ->probe() implementation.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
Reviewed-by: Javier Martinez Canillas <javierm@redhat.com>
Link: https://lore.kernel.org/r/20240201144951.294215-5-andriy.shevchenko@linux.intel.com
Signed-off-by: Lee Jones <lee@kernel.org>
drivers/video/backlight/hx8357.c

index 70a6275..339d912 100644 (file)
@@ -574,7 +574,7 @@ static int hx8357_probe(struct spi_device *spi)
        hx8357_init_fn init_fn;
        int i, ret;
 
-       lcd = devm_kzalloc(&spi->dev, sizeof(*lcd), GFP_KERNEL);
+       lcd = devm_kzalloc(dev, sizeof(*lcd), GFP_KERNEL);
        if (!lcd)
                return -ENOMEM;
 
@@ -604,8 +604,7 @@ static int hx8357_probe(struct spi_device *spi)
                        gpiod_set_consumer_name(lcd->im_pins->desc[i], "im_pins");
        }
 
-       lcdev = devm_lcd_device_register(&spi->dev, "mxsfb", &spi->dev, lcd,
-                                       &hx8357_ops);
+       lcdev = devm_lcd_device_register(dev, "mxsfb", dev, lcd, &hx8357_ops);
        if (IS_ERR(lcdev)) {
                ret = PTR_ERR(lcdev);
                return ret;
@@ -618,7 +617,7 @@ static int hx8357_probe(struct spi_device *spi)
        if (ret)
                return dev_err_probe(dev, ret, "Couldn't initialize panel\n");
 
-       dev_info(&spi->dev, "Panel probed\n");
+       dev_info(dev, "Panel probed\n");
 
        return 0;
 }