]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
backlight: pcf50633-backlight: Remove unused driver
authorDr. David Alan Gilbert <linux@treblig.org>
Tue, 11 Mar 2025 01:49:57 +0000 (01:49 +0000)
committerLee Jones <lee@kernel.org>
Fri, 14 Mar 2025 11:36:33 +0000 (11:36 +0000)
The pcf50633 was used as part of the OpenMoko devices but
the support for its main chip was recently removed in:
commit 61b7f8920b17 ("ARM: s3c: remove all s3c24xx support")

See https://lore.kernel.org/all/Z8z236h4B5A6Ki3D@gallifrey/

Remove it.

Signed-off-by: "Dr. David Alan Gilbert" <linux@treblig.org>
Link: https://lore.kernel.org/r/20250311014959.743322-8-linux@treblig.org
Signed-off-by: Lee Jones <lee@kernel.org>
drivers/video/backlight/Kconfig
drivers/video/backlight/Makefile
drivers/video/backlight/pcf50633-backlight.c [deleted file]
include/linux/mfd/pcf50633/backlight.h [deleted file]
include/linux/mfd/pcf50633/core.h

index 6e40262e3110ee60c510b6ec44621b240d5a38e4..d9374d208ceebbf8b3c27976e9cb4d725939b942 100644 (file)
@@ -370,13 +370,6 @@ config BACKLIGHT_88PM860X
        help
          Say Y to enable the backlight driver for Marvell 88PM8606.
 
-config BACKLIGHT_PCF50633
-       tristate "Backlight driver for NXP PCF50633 MFD"
-       depends on MFD_PCF50633
-       help
-         If you have a backlight driven by a NXP PCF50633 MFD, say Y here to
-         enable its driver.
-
 config BACKLIGHT_AAT2870
        tristate "AnalogicTech AAT2870 Backlight"
        depends on MFD_AAT2870_CORE
index 156ff9461fb333d76c5219f7951fa72ec7d120e7..dfbb169bf6ea215704859f633b6c4a887f4ebacd 100644 (file)
@@ -50,7 +50,6 @@ obj-$(CONFIG_BACKLIGHT_MP3309C)               += mp3309c.o
 obj-$(CONFIG_BACKLIGHT_MT6370)         += mt6370-backlight.o
 obj-$(CONFIG_BACKLIGHT_OMAP1)          += omap1_bl.o
 obj-$(CONFIG_BACKLIGHT_PANDORA)                += pandora_bl.o
-obj-$(CONFIG_BACKLIGHT_PCF50633)       += pcf50633-backlight.o
 obj-$(CONFIG_BACKLIGHT_PWM)            += pwm_bl.o
 obj-$(CONFIG_BACKLIGHT_QCOM_WLED)      += qcom-wled.o
 obj-$(CONFIG_BACKLIGHT_RT4831)         += rt4831-backlight.o
diff --git a/drivers/video/backlight/pcf50633-backlight.c b/drivers/video/backlight/pcf50633-backlight.c
deleted file mode 100644 (file)
index 157be2f..0000000
+++ /dev/null
@@ -1,154 +0,0 @@
-// SPDX-License-Identifier: GPL-2.0-or-later
-/*
- *  Copyright (C) 2009-2010, Lars-Peter Clausen <lars@metafoo.de>
- *      PCF50633 backlight device driver
- */
-
-#include <linux/kernel.h>
-#include <linux/module.h>
-#include <linux/slab.h>
-#include <linux/platform_device.h>
-
-#include <linux/backlight.h>
-
-#include <linux/mfd/pcf50633/core.h>
-#include <linux/mfd/pcf50633/backlight.h>
-
-struct pcf50633_bl {
-       struct pcf50633 *pcf;
-       struct backlight_device *bl;
-
-       unsigned int brightness;
-       unsigned int brightness_limit;
-};
-
-/*
- * pcf50633_bl_set_brightness_limit
- *
- * Update the brightness limit for the pc50633 backlight. The actual brightness
- * will not go above the limit. This is useful to limit power drain for example
- * on low battery.
- *
- * @dev: Pointer to a pcf50633 device
- * @limit: The brightness limit. Valid values are 0-63
- */
-int pcf50633_bl_set_brightness_limit(struct pcf50633 *pcf, unsigned int limit)
-{
-       struct pcf50633_bl *pcf_bl = platform_get_drvdata(pcf->bl_pdev);
-
-       if (!pcf_bl)
-               return -ENODEV;
-
-       pcf_bl->brightness_limit = limit & 0x3f;
-       backlight_update_status(pcf_bl->bl);
-
-       return 0;
-}
-
-static int pcf50633_bl_update_status(struct backlight_device *bl)
-{
-       struct pcf50633_bl *pcf_bl = bl_get_data(bl);
-       unsigned int new_brightness;
-
-
-       if (bl->props.state & (BL_CORE_SUSPENDED | BL_CORE_FBBLANK) ||
-               bl->props.power != BACKLIGHT_POWER_ON)
-               new_brightness = 0;
-       else if (bl->props.brightness < pcf_bl->brightness_limit)
-               new_brightness = bl->props.brightness;
-       else
-               new_brightness = pcf_bl->brightness_limit;
-
-
-       if (pcf_bl->brightness == new_brightness)
-               return 0;
-
-       if (new_brightness) {
-               pcf50633_reg_write(pcf_bl->pcf, PCF50633_REG_LEDOUT,
-                                       new_brightness);
-               if (!pcf_bl->brightness)
-                       pcf50633_reg_write(pcf_bl->pcf, PCF50633_REG_LEDENA, 1);
-       } else {
-               pcf50633_reg_write(pcf_bl->pcf, PCF50633_REG_LEDENA, 0);
-       }
-
-       pcf_bl->brightness = new_brightness;
-
-       return 0;
-}
-
-static int pcf50633_bl_get_brightness(struct backlight_device *bl)
-{
-       struct pcf50633_bl *pcf_bl = bl_get_data(bl);
-
-       return pcf_bl->brightness;
-}
-
-static const struct backlight_ops pcf50633_bl_ops = {
-       .get_brightness = pcf50633_bl_get_brightness,
-       .update_status  = pcf50633_bl_update_status,
-       .options        = BL_CORE_SUSPENDRESUME,
-};
-
-static int pcf50633_bl_probe(struct platform_device *pdev)
-{
-       struct pcf50633_bl *pcf_bl;
-       struct device *parent = pdev->dev.parent;
-       struct pcf50633_platform_data *pcf50633_data = dev_get_platdata(parent);
-       struct pcf50633_bl_platform_data *pdata = pcf50633_data->backlight_data;
-       struct backlight_properties bl_props;
-
-       pcf_bl = devm_kzalloc(&pdev->dev, sizeof(*pcf_bl), GFP_KERNEL);
-       if (!pcf_bl)
-               return -ENOMEM;
-
-       memset(&bl_props, 0, sizeof(bl_props));
-       bl_props.type = BACKLIGHT_RAW;
-       bl_props.max_brightness = 0x3f;
-       bl_props.power = BACKLIGHT_POWER_ON;
-
-       if (pdata) {
-               bl_props.brightness = pdata->default_brightness;
-               pcf_bl->brightness_limit = pdata->default_brightness_limit;
-       } else {
-               bl_props.brightness = 0x3f;
-               pcf_bl->brightness_limit = 0x3f;
-       }
-
-       pcf_bl->pcf = dev_to_pcf50633(pdev->dev.parent);
-
-       pcf_bl->bl = devm_backlight_device_register(&pdev->dev, pdev->name,
-                                               &pdev->dev, pcf_bl,
-                                               &pcf50633_bl_ops, &bl_props);
-
-       if (IS_ERR(pcf_bl->bl))
-               return PTR_ERR(pcf_bl->bl);
-
-       platform_set_drvdata(pdev, pcf_bl);
-
-       pcf50633_reg_write(pcf_bl->pcf, PCF50633_REG_LEDDIM, pdata->ramp_time);
-
-       /*
-        * Should be different from bl_props.brightness, so we do not exit
-        * update_status early the first time it's called
-        */
-       pcf_bl->brightness = pcf_bl->bl->props.brightness + 1;
-
-       backlight_update_status(pcf_bl->bl);
-
-       return 0;
-}
-
-static struct platform_driver pcf50633_bl_driver = {
-       .probe =        pcf50633_bl_probe,
-       .driver = {
-               .name = "pcf50633-backlight",
-       },
-};
-
-module_platform_driver(pcf50633_bl_driver);
-
-MODULE_AUTHOR("Lars-Peter Clausen <lars@metafoo.de>");
-MODULE_DESCRIPTION("PCF50633 backlight driver");
-MODULE_LICENSE("GPL");
-MODULE_ALIAS("platform:pcf50633-backlight");
diff --git a/include/linux/mfd/pcf50633/backlight.h b/include/linux/mfd/pcf50633/backlight.h
deleted file mode 100644 (file)
index fd4a4f8..0000000
+++ /dev/null
@@ -1,42 +0,0 @@
-/* SPDX-License-Identifier: GPL-2.0-or-later */
-/*
- *  Copyright (C) 2009-2010, Lars-Peter Clausen <lars@metafoo.de>
- *      PCF50633 backlight device driver
- */
-
-#ifndef __LINUX_MFD_PCF50633_BACKLIGHT
-#define __LINUX_MFD_PCF50633_BACKLIGHT
-
-/*
-* @default_brightness: Backlight brightness is initialized to this value
-*
-* Brightness to be used after the driver has been probed.
-* Valid range 0-63.
-*
-* @default_brightness_limit: The actual brightness is limited by this value
-*
-* Brightness limit to be used after the driver has been probed. This is useful
-* when it is not known how much power is available for the backlight during
-* probe.
-* Valid range 0-63. Can be changed later with pcf50633_bl_set_brightness_limit.
-*
-* @ramp_time: Display ramp time when changing brightness
-*
-* When changing the backlights brightness the change is not instant, instead
-* it fades smooth from one state to another. This value specifies how long
-* the fade should take. The lower the value the higher the fade time.
-* Valid range 0-255
-*/
-struct pcf50633_bl_platform_data {
-       unsigned int    default_brightness;
-       unsigned int    default_brightness_limit;
-       uint8_t         ramp_time;
-};
-
-
-struct pcf50633;
-
-int pcf50633_bl_set_brightness_limit(struct pcf50633 *pcf, unsigned int limit);
-
-#endif
-
index 539f27f8bd89b0dd222c48bd28881201294a8b51..42d2b0e4884e838c00ff3648b68736d909a5611a 100644 (file)
@@ -15,7 +15,6 @@
 #include <linux/regulator/machine.h>
 #include <linux/pm.h>
 #include <linux/power_supply.h>
-#include <linux/mfd/pcf50633/backlight.h>
 
 struct pcf50633;
 struct regmap;
@@ -42,8 +41,6 @@ struct pcf50633_platform_data {
        void (*force_shutdown)(struct pcf50633 *);
 
        u8 resumers[5];
-
-       struct pcf50633_bl_platform_data *backlight_data;
 };
 
 struct pcf50633_irq {