kernel: backport LEDs driver for BCMBCA devices
authorRafał Miłecki <rafal@milecki.pl>
Sun, 17 Jul 2022 13:10:01 +0000 (15:10 +0200)
committerRafał Miłecki <rafal@milecki.pl>
Mon, 18 Jul 2022 07:12:17 +0000 (09:12 +0200)
This includes BCM63xx and BCM4908 families.

Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
(cherry picked from commit d9ab1e56d8d16182bd292f393c012d7e6873ed89)

target/linux/bcm4908/config-5.4
target/linux/bcm53xx/config-5.4
target/linux/bcm63xx/config-5.4
target/linux/generic/backport-5.4/800-v5.20-0001-dt-bindings-leds-add-Broadcom-s-BCM63138-controller.patch [new file with mode: 0644]
target/linux/generic/backport-5.4/800-v5.20-0002-leds-bcm63138-add-support-for-BCM63138-controller.patch [new file with mode: 0644]

index 86bbfc2910c148124c531f739411d7086a8b3d17..4339cd6f9f314ff8ea6550c538a720101fdebf86 100644 (file)
@@ -128,6 +128,7 @@ CONFIG_IRQ_DOMAIN=y
 CONFIG_IRQ_DOMAIN_HIERARCHY=y
 CONFIG_IRQ_FORCED_THREADING=y
 CONFIG_IRQ_WORK=y
+CONFIG_LEDS_BCM63138=y
 CONFIG_LEDS_GPIO=y
 CONFIG_LIBFDT=y
 CONFIG_LOCK_DEBUGGING_SUPPORT=y
index f57bfd36b66ab2fdd2fde08091c24d4280397b5d..bee678bcf4fb4260e6ddaae4785e26543db2028d 100644 (file)
@@ -237,6 +237,7 @@ CONFIG_IRQ_DOMAIN=y
 CONFIG_IRQ_DOMAIN_HIERARCHY=y
 CONFIG_IRQ_FORCED_THREADING=y
 CONFIG_IRQ_WORK=y
+# CONFIG_LEDS_BCM63138 is not set
 CONFIG_LIBFDT=y
 CONFIG_LOCK_DEBUGGING_SUPPORT=y
 CONFIG_LOCK_SPIN_ON_OWNER=y
index 17311b85e31da59b1da0fb29863c92963fc5f3d1..f36a35a74628e3af7239096599a13a850fabcda0 100644 (file)
@@ -106,6 +106,7 @@ CONFIG_IRQ_MIPS_CPU=y
 CONFIG_IRQ_WORK=y
 CONFIG_KEXEC=y
 CONFIG_KEXEC_CORE=y
+# CONFIG_LEDS_BCM63138 is not set
 CONFIG_LEDS_BCM6328=y
 CONFIG_LEDS_BCM6358=y
 CONFIG_LEDS_GPIO=y
diff --git a/target/linux/generic/backport-5.4/800-v5.20-0001-dt-bindings-leds-add-Broadcom-s-BCM63138-controller.patch b/target/linux/generic/backport-5.4/800-v5.20-0001-dt-bindings-leds-add-Broadcom-s-BCM63138-controller.patch
new file mode 100644 (file)
index 0000000..b1072ce
--- /dev/null
@@ -0,0 +1,125 @@
+From 13344f8ce8a0d98aa7f5d69ce3b47393c73a343b Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
+Date: Mon, 27 Dec 2021 15:59:04 +0100
+Subject: [PATCH] dt-bindings: leds: add Broadcom's BCM63138 controller
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+Broadcom used 2 LEDs hardware blocks for their BCM63xx SoCs:
+1. Older one (BCM6318, BCM6328, BCM6362, BCM63268, BCM6838)
+2. Newer one (BCM6848, BCM6858, BCM63138, BCM63148, BCM63381, BCM68360)
+
+The newer one was also later also used on BCM4908 SoC.
+
+Old block is already documented in the leds-bcm6328.yaml. This binding
+documents the new one which uses different registers & programming. It's
+first used in BCM63138 thus the binding name.
+
+Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
+Reviewed-by: Rob Herring <robh@kernel.org>
+Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
+Signed-off-by: Pavel Machek <pavel@ucw.cz>
+---
+ .../bindings/leds/leds-bcm63138.yaml          | 95 +++++++++++++++++++
+ 1 file changed, 95 insertions(+)
+ create mode 100644 Documentation/devicetree/bindings/leds/leds-bcm63138.yaml
+
+--- /dev/null
++++ b/Documentation/devicetree/bindings/leds/leds-bcm63138.yaml
+@@ -0,0 +1,95 @@
++# SPDX-License-Identifier: GPL-2.0-only OR BSD-2-Clause
++%YAML 1.2
++---
++$id: http://devicetree.org/schemas/leds/leds-bcm63138.yaml#
++$schema: http://devicetree.org/meta-schemas/core.yaml#
++
++title: Broadcom's BCM63138 LEDs controller
++
++maintainers:
++  - Rafał Miłecki <rafal@milecki.pl>
++
++description: |
++  This LEDs controller was first used on BCM63138 and later reused on BCM4908,
++  BCM6848, BCM6858, BCM63138, BCM63148, BCM63381 and BCM68360 SoCs.
++
++  It supports up to 32 LEDs that can be connected parallelly or serially. It
++  also includes limited support for hardware blinking.
++
++  Binding serially connected LEDs isn't documented yet.
++
++properties:
++  compatible:
++    oneOf:
++      - items:
++          - enum:
++              - brcm,bcm4908-leds
++              - brcm,bcm6848-leds
++              - brcm,bcm6858-leds
++              - brcm,bcm63148-leds
++              - brcm,bcm63381-leds
++              - brcm,bcm68360-leds
++          - const: brcm,bcm63138-leds
++      - const: brcm,bcm63138-leds
++
++  reg:
++    maxItems: 1
++
++  "#address-cells":
++    const: 1
++
++  "#size-cells":
++    const: 0
++
++patternProperties:
++  "^led@[a-f0-9]+$":
++    type: object
++
++    $ref: common.yaml#
++
++    properties:
++      reg:
++        maxItems: 1
++        description: LED pin number
++
++      active-low:
++        type: boolean
++        description: Makes LED active low.
++
++    required:
++      - reg
++
++    unevaluatedProperties: false
++
++required:
++  - reg
++  - "#address-cells"
++  - "#size-cells"
++
++additionalProperties: false
++
++examples:
++  - |
++    #include <dt-bindings/leds/common.h>
++
++    leds@ff800800 {
++        compatible = "brcm,bcm4908-leds", "brcm,bcm63138-leds";
++        reg = <0xff800800 0xdc>;
++
++        #address-cells = <1>;
++        #size-cells = <0>;
++
++        led@0 {
++            reg = <0x0>;
++            function = LED_FUNCTION_POWER;
++            color = <LED_COLOR_ID_GREEN>;
++            default-state = "on";
++        };
++
++        led@3 {
++            reg = <0x3>;
++            function = LED_FUNCTION_STATUS;
++            color = <LED_COLOR_ID_GREEN>;
++            active-low;
++        };
++    };
diff --git a/target/linux/generic/backport-5.4/800-v5.20-0002-leds-bcm63138-add-support-for-BCM63138-controller.patch b/target/linux/generic/backport-5.4/800-v5.20-0002-leds-bcm63138-add-support-for-BCM63138-controller.patch
new file mode 100644 (file)
index 0000000..a2c3934
--- /dev/null
@@ -0,0 +1,371 @@
+From a0ba692072d89075d0a75c7ad9df31f2c1ee9a1c Mon Sep 17 00:00:00 2001
+From: =?UTF-8?q?Rafa=C5=82=20Mi=C5=82ecki?= <rafal@milecki.pl>
+Date: Mon, 27 Dec 2021 15:59:05 +0100
+Subject: [PATCH] leds: bcm63138: add support for BCM63138 controller
+MIME-Version: 1.0
+Content-Type: text/plain; charset=UTF-8
+Content-Transfer-Encoding: 8bit
+
+It's a new controller first introduced in BCM63138 SoC. Later it was
+also used in BCM4908, some BCM68xx and some BCM63xxx SoCs.
+
+Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
+Reviewed-by: Florian Fainelli <f.fainelli@gmail.com>
+Signed-off-by: Pavel Machek <pavel@ucw.cz>
+---
+ drivers/leds/blink/Kconfig         |  12 ++
+ drivers/leds/blink/Makefile        |   1 +
+ drivers/leds/blink/leds-bcm63138.c | 308 +++++++++++++++++++++++++++++
+ 3 files changed, 321 insertions(+)
+ create mode 100644 drivers/leds/blink/leds-bcm63138.c
+
+--- /dev/null
++++ b/drivers/leds/blink/Kconfig
+@@ -0,0 +1,11 @@
++config LEDS_BCM63138
++      tristate "LED Support for Broadcom BCM63138 SoC"
++      depends on LEDS_CLASS
++      depends on ARCH_BCM4908 || ARCH_BCM_5301X || BCM63XX || COMPILE_TEST
++      depends on HAS_IOMEM
++      depends on OF
++      default ARCH_BCM4908
++      help
++        This option enables support for LED controller that is part of
++        BCM63138 SoC. The same hardware block is known to be also used
++        in BCM4908, BCM6848, BCM6858, BCM63148, BCM63381 and BCM68360.
+--- /dev/null
++++ b/drivers/leds/blink/Makefile
+@@ -0,0 +1,2 @@
++# SPDX-License-Identifier: GPL-2.0
++obj-$(CONFIG_LEDS_BCM63138)   += leds-bcm63138.o
+--- /dev/null
++++ b/drivers/leds/blink/leds-bcm63138.c
+@@ -0,0 +1,308 @@
++// SPDX-License-Identifier: GPL-2.0-only
++/*
++ * Copyright (C) 2021 Rafał Miłecki <rafal@milecki.pl>
++ */
++#include <linux/delay.h>
++#include <linux/io.h>
++#include <linux/leds.h>
++#include <linux/module.h>
++#include <linux/of.h>
++#include <linux/pinctrl/consumer.h>
++#include <linux/platform_device.h>
++#include <linux/spinlock.h>
++
++#define BCM63138_MAX_LEDS                             32
++#define BCM63138_MAX_BRIGHTNESS                               9
++
++#define BCM63138_LED_BITS                             4                               /* how many bits control a single LED */
++#define BCM63138_LED_MASK                             ((1 << BCM63138_LED_BITS) - 1)  /* 0xf */
++#define BCM63138_LEDS_PER_REG                         (32 / BCM63138_LED_BITS)        /* 8 */
++
++#define BCM63138_GLB_CTRL                             0x00
++#define  BCM63138_GLB_CTRL_SERIAL_LED_DATA_PPOL               0x00000002
++#define  BCM63138_GLB_CTRL_SERIAL_LED_EN_POL          0x00000008
++#define BCM63138_MASK                                 0x04
++#define BCM63138_HW_LED_EN                            0x08
++#define BCM63138_SERIAL_LED_SHIFT_SEL                 0x0c
++#define BCM63138_FLASH_RATE_CTRL1                     0x10
++#define BCM63138_FLASH_RATE_CTRL2                     0x14
++#define BCM63138_FLASH_RATE_CTRL3                     0x18
++#define BCM63138_FLASH_RATE_CTRL4                     0x1c
++#define BCM63138_BRIGHT_CTRL1                         0x20
++#define BCM63138_BRIGHT_CTRL2                         0x24
++#define BCM63138_BRIGHT_CTRL3                         0x28
++#define BCM63138_BRIGHT_CTRL4                         0x2c
++#define BCM63138_POWER_LED_CFG                                0x30
++#define BCM63138_HW_POLARITY                          0xb4
++#define BCM63138_SW_DATA                              0xb8
++#define BCM63138_SW_POLARITY                          0xbc
++#define BCM63138_PARALLEL_LED_POLARITY                        0xc0
++#define BCM63138_SERIAL_LED_POLARITY                  0xc4
++#define BCM63138_HW_LED_STATUS                                0xc8
++#define BCM63138_FLASH_CTRL_STATUS                    0xcc
++#define BCM63138_FLASH_BRT_CTRL                               0xd0
++#define BCM63138_FLASH_P_LED_OUT_STATUS                       0xd4
++#define BCM63138_FLASH_S_LED_OUT_STATUS                       0xd8
++
++struct bcm63138_leds {
++      struct device *dev;
++      void __iomem *base;
++      spinlock_t lock;
++};
++
++struct bcm63138_led {
++      struct bcm63138_leds *leds;
++      struct led_classdev cdev;
++      u32 pin;
++      bool active_low;
++};
++
++/*
++ * I/O access
++ */
++
++static void bcm63138_leds_write(struct bcm63138_leds *leds, unsigned int reg,
++                              u32 data)
++{
++      writel(data, leds->base + reg);
++}
++
++static unsigned long bcm63138_leds_read(struct bcm63138_leds *leds,
++                                      unsigned int reg)
++{
++      return readl(leds->base + reg);
++}
++
++static void bcm63138_leds_update_bits(struct bcm63138_leds *leds,
++                                    unsigned int reg, u32 mask, u32 val)
++{
++      WARN_ON(val & ~mask);
++
++      bcm63138_leds_write(leds, reg, (bcm63138_leds_read(leds, reg) & ~mask) | (val & mask));
++}
++
++/*
++ * Helpers
++ */
++
++static void bcm63138_leds_set_flash_rate(struct bcm63138_leds *leds,
++                                       struct bcm63138_led *led,
++                                       u8 value)
++{
++      int reg_offset = (led->pin >> fls((BCM63138_LEDS_PER_REG - 1))) * 4;
++      int shift = (led->pin & (BCM63138_LEDS_PER_REG - 1)) * BCM63138_LED_BITS;
++
++      bcm63138_leds_update_bits(leds, BCM63138_FLASH_RATE_CTRL1 + reg_offset,
++                                BCM63138_LED_MASK << shift, value << shift);
++}
++
++static void bcm63138_leds_set_bright(struct bcm63138_leds *leds,
++                                   struct bcm63138_led *led,
++                                   u8 value)
++{
++      int reg_offset = (led->pin >> fls((BCM63138_LEDS_PER_REG - 1))) * 4;
++      int shift = (led->pin & (BCM63138_LEDS_PER_REG - 1)) * BCM63138_LED_BITS;
++
++      bcm63138_leds_update_bits(leds, BCM63138_BRIGHT_CTRL1 + reg_offset,
++                                BCM63138_LED_MASK << shift, value << shift);
++}
++
++static void bcm63138_leds_enable_led(struct bcm63138_leds *leds,
++                                   struct bcm63138_led *led,
++                                   enum led_brightness value)
++{
++      u32 bit = BIT(led->pin);
++
++      bcm63138_leds_update_bits(leds, BCM63138_SW_DATA, bit,
++                                value == LED_OFF ? 0 : bit);
++}
++
++/*
++ * API callbacks
++ */
++
++static void bcm63138_leds_brightness_set(struct led_classdev *led_cdev,
++                                       enum led_brightness value)
++{
++      struct bcm63138_led *led = container_of(led_cdev, struct bcm63138_led, cdev);
++      struct bcm63138_leds *leds = led->leds;
++      unsigned long flags;
++
++      spin_lock_irqsave(&leds->lock, flags);
++
++      bcm63138_leds_enable_led(leds, led, value);
++      if (!value)
++              bcm63138_leds_set_flash_rate(leds, led, 0);
++      else
++              bcm63138_leds_set_bright(leds, led, value);
++
++      spin_unlock_irqrestore(&leds->lock, flags);
++}
++
++static int bcm63138_leds_blink_set(struct led_classdev *led_cdev,
++                                 unsigned long *delay_on,
++                                 unsigned long *delay_off)
++{
++      struct bcm63138_led *led = container_of(led_cdev, struct bcm63138_led, cdev);
++      struct bcm63138_leds *leds = led->leds;
++      unsigned long flags;
++      u8 value;
++
++      if (!*delay_on && !*delay_off) {
++              *delay_on = 640;
++              *delay_off = 640;
++      }
++
++      if (*delay_on != *delay_off) {
++              dev_dbg(led_cdev->dev, "Blinking at unequal delays is not supported\n");
++              return -EINVAL;
++      }
++
++      switch (*delay_on) {
++      case 1152 ... 1408: /* 1280 ms ± 10% */
++              value = 0x7;
++              break;
++      case 576 ... 704: /* 640 ms ± 10% */
++              value = 0x6;
++              break;
++      case 288 ... 352: /* 320 ms ± 10% */
++              value = 0x5;
++              break;
++      case 126 ... 154: /* 140 ms ± 10% */
++              value = 0x4;
++              break;
++      case 59 ... 72: /* 65 ms ± 10% */
++              value = 0x3;
++              break;
++      default:
++              dev_dbg(led_cdev->dev, "Blinking delay value %lu is unsupported\n",
++                      *delay_on);
++              return -EINVAL;
++      }
++
++      spin_lock_irqsave(&leds->lock, flags);
++
++      bcm63138_leds_enable_led(leds, led, BCM63138_MAX_BRIGHTNESS);
++      bcm63138_leds_set_flash_rate(leds, led, value);
++
++      spin_unlock_irqrestore(&leds->lock, flags);
++
++      return 0;
++}
++
++/*
++ * LED driver
++ */
++
++static void bcm63138_leds_create_led(struct bcm63138_leds *leds,
++                                   struct device_node *np)
++{
++      struct led_init_data init_data = {
++              .fwnode = of_fwnode_handle(np),
++      };
++      struct device *dev = leds->dev;
++      struct bcm63138_led *led;
++      struct pinctrl *pinctrl;
++      u32 bit;
++      int err;
++
++      led = devm_kzalloc(dev, sizeof(*led), GFP_KERNEL);
++      if (!led) {
++              dev_err(dev, "Failed to alloc LED\n");
++              return;
++      }
++
++      led->leds = leds;
++
++      if (of_property_read_u32(np, "reg", &led->pin)) {
++              dev_err(dev, "Missing \"reg\" property in %pOF\n", np);
++              goto err_free;
++      }
++
++      if (led->pin >= BCM63138_MAX_LEDS) {
++              dev_err(dev, "Invalid \"reg\" value %d\n", led->pin);
++              goto err_free;
++      }
++
++      led->active_low = of_property_read_bool(np, "active-low");
++
++      led->cdev.max_brightness = BCM63138_MAX_BRIGHTNESS;
++      led->cdev.brightness_set = bcm63138_leds_brightness_set;
++      led->cdev.blink_set = bcm63138_leds_blink_set;
++
++      err = devm_led_classdev_register_ext(dev, &led->cdev, &init_data);
++      if (err) {
++              dev_err(dev, "Failed to register LED %pOF: %d\n", np, err);
++              goto err_free;
++      }
++
++      pinctrl = devm_pinctrl_get_select_default(led->cdev.dev);
++      if (IS_ERR(pinctrl) && PTR_ERR(pinctrl) != -ENODEV) {
++              dev_warn(led->cdev.dev, "Failed to select %pOF pinctrl: %ld\n",
++                       np, PTR_ERR(pinctrl));
++      }
++
++      bit = BIT(led->pin);
++      bcm63138_leds_update_bits(leds, BCM63138_PARALLEL_LED_POLARITY, bit,
++                                led->active_low ? 0 : bit);
++      bcm63138_leds_update_bits(leds, BCM63138_HW_LED_EN, bit, 0);
++      bcm63138_leds_set_flash_rate(leds, led, 0);
++      bcm63138_leds_enable_led(leds, led, led->cdev.brightness);
++
++      return;
++
++err_free:
++      devm_kfree(dev, led);
++}
++
++static int bcm63138_leds_probe(struct platform_device *pdev)
++{
++      struct device_node *np = dev_of_node(&pdev->dev);
++      struct device *dev = &pdev->dev;
++      struct bcm63138_leds *leds;
++      struct device_node *child;
++
++      leds = devm_kzalloc(dev, sizeof(*leds), GFP_KERNEL);
++      if (!leds)
++              return -ENOMEM;
++
++      leds->dev = dev;
++
++      leds->base = devm_platform_ioremap_resource(pdev, 0);
++      if (IS_ERR(leds->base))
++              return PTR_ERR(leds->base);
++
++      spin_lock_init(&leds->lock);
++
++      bcm63138_leds_write(leds, BCM63138_GLB_CTRL,
++                          BCM63138_GLB_CTRL_SERIAL_LED_DATA_PPOL |
++                          BCM63138_GLB_CTRL_SERIAL_LED_EN_POL);
++      bcm63138_leds_write(leds, BCM63138_HW_LED_EN, 0);
++      bcm63138_leds_write(leds, BCM63138_SERIAL_LED_POLARITY, 0);
++      bcm63138_leds_write(leds, BCM63138_PARALLEL_LED_POLARITY, 0);
++
++      for_each_available_child_of_node(np, child) {
++              bcm63138_leds_create_led(leds, child);
++      }
++
++      return 0;
++}
++
++static const struct of_device_id bcm63138_leds_of_match_table[] = {
++      { .compatible = "brcm,bcm63138-leds", },
++      { },
++};
++
++static struct platform_driver bcm63138_leds_driver = {
++      .probe = bcm63138_leds_probe,
++      .driver = {
++              .name = "leds-bcm63xxx",
++              .of_match_table = bcm63138_leds_of_match_table,
++      },
++};
++
++module_platform_driver(bcm63138_leds_driver);
++
++MODULE_AUTHOR("Rafał Miłecki");
++MODULE_LICENSE("GPL");
++MODULE_DEVICE_TABLE(of, bcm63138_leds_of_match_table);
+--- a/drivers/leds/Kconfig
++++ b/drivers/leds/Kconfig
+@@ -824,6 +824,8 @@ config LEDS_LM36274
+         Say Y to enable the LM36274 LED driver for TI LMU devices.
+         This supports the LED device LM36274.
++source "drivers/leds/blink/Kconfig"
++
+ comment "LED Triggers"
+ source "drivers/leds/trigger/Kconfig"
+--- a/drivers/leds/Makefile
++++ b/drivers/leds/Makefile
+@@ -95,3 +95,6 @@ obj-$(CONFIG_LEDS_USER)                      += uleds.o
+ # LED Triggers
+ obj-$(CONFIG_LEDS_TRIGGERS)           += trigger/
++
++# LED Blink
++obj-y                                 += blink/