mvebu: 6.12: refresh patches
Drop upstreamed patches: 0001-v6.16-pinctrl-armada-37xx-use-correct-OUTPUT_VAL-register-.patch 0002-v6.16-pinctrl-armada-37xx-set-GPIO-output-value-before-set.patch 820-v6.11-01-dt-bindings-firmware-add-cznic-turris-omnia-mcu-bind.patch 820-v6.11-02-platform-cznic-Add-preliminary-support-for-Turris-Om.patch 820-v6.11-03-platform-cznic-turris-omnia-mcu-Add-support-for-MCU-.patch 820-v6.11-04-platform-cznic-turris-omnia-mcu-Add-support-for-powe.patch 820-v6.11-05-platform-cznic-turris-omnia-mcu-Add-support-for-MCU-.patch 820-v6.11-06-platform-cznic-turris-omnia-mcu-Add-support-for-MCU-.patch 820-v6.11-07-ARM-dts-turris-omnia-Add-MCU-system-controller-node.patch 820-v6.11-08-ARM-dts-turris-omnia-Add-GPIO-key-node-for-front-but.patch 820-v6.11-09-platform-cznic-turris-omnia-mcu-Depend-on-OF.patch 820-v6.11-10-platform-cznic-turris-omnia-mcu-Depend-on-WATCHDOG.patch 820-v6.11-11-platform-cznic-turris-omnia-mcu-fix-Kconfig-dependen.patch Manually refreshed: 310-linksys-use-eth0-as-cpu-port.patch 350-drivers-thermal-step_wise-add-support-for-hysteresis.patch 901-dt-bindings-Add-IEI-vendor-prefix-and-IEI-WT61P803-P.patch 902-drivers-mfd-Add-a-driver-for-IEI-WT61P803-PUZZLE-MCU.patch 910-drivers-leds-wt61p803-puzzle-improvements.patch All other patches automatically refreshed. Signed-off-by: Stefan Kalscheuer <stefan@stklcode.de> Link: https://github.com/openwrt/openwrt/pull/18975 Signed-off-by: Hauke Mehrtens <hauke@hauke-m.de>
This commit is contained in:
parent
6354ac76a4
commit
1738e87632
34 changed files with 67 additions and 3659 deletions
|
@ -1,44 +0,0 @@
|
|||
From 947c93eb29c2a581c0b0b6d5f21af3c2b7ff6d25 Mon Sep 17 00:00:00 2001
|
||||
From: Gabor Juhos <j4g8y7@gmail.com>
|
||||
Date: Wed, 14 May 2025 21:18:32 +0200
|
||||
Subject: [PATCH 1/7] pinctrl: armada-37xx: use correct OUTPUT_VAL register for
|
||||
GPIOs > 31
|
||||
|
||||
The controller has two consecutive OUTPUT_VAL registers and both
|
||||
holds output value for 32 GPIOs. Due to a missing adjustment, the
|
||||
current code always uses the first register while setting the
|
||||
output value whereas it should use the second one for GPIOs > 31.
|
||||
|
||||
Add the missing armada_37xx_update_reg() call to adjust the register
|
||||
according to the 'offset' parameter of the function to fix the issue.
|
||||
|
||||
Cc: stable@vger.kernel.org
|
||||
Fixes: 6702abb3bf23 ("pinctrl: armada-37xx: Fix direction_output() callback behavior")
|
||||
Signed-off-by: Imre Kaloz <kaloz@openwrt.org>
|
||||
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
|
||||
Signed-off-by: Gabor Juhos <j4g8y7@gmail.com>
|
||||
Link: https://lore.kernel.org/20250514-pinctrl-a37xx-fixes-v2-1-07e9ac1ab737@gmail.com
|
||||
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
|
||||
---
|
||||
drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 3 +++
|
||||
1 file changed, 3 insertions(+)
|
||||
|
||||
--- a/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c
|
||||
+++ b/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c
|
||||
@@ -417,6 +417,7 @@ static int armada_37xx_gpio_direction_ou
|
||||
unsigned int offset, int value)
|
||||
{
|
||||
struct armada_37xx_pinctrl *info = gpiochip_get_data(chip);
|
||||
+ unsigned int val_offset = offset;
|
||||
unsigned int reg = OUTPUT_EN;
|
||||
unsigned int mask, val, ret;
|
||||
|
||||
@@ -429,6 +430,8 @@ static int armada_37xx_gpio_direction_ou
|
||||
return ret;
|
||||
|
||||
reg = OUTPUT_VAL;
|
||||
+ armada_37xx_update_reg(®, &val_offset);
|
||||
+
|
||||
val = value ? mask : 0;
|
||||
regmap_update_bits(info->regmap, reg, mask, val);
|
||||
|
|
@ -1,58 +0,0 @@
|
|||
From e6ebd4942981f8ad37189bbb36a3c8495e21ef4c Mon Sep 17 00:00:00 2001
|
||||
From: Gabor Juhos <j4g8y7@gmail.com>
|
||||
Date: Wed, 14 May 2025 21:18:33 +0200
|
||||
Subject: [PATCH 2/7] pinctrl: armada-37xx: set GPIO output value before
|
||||
setting direction
|
||||
|
||||
Changing the direction before updating the output value in the
|
||||
OUTPUT_VAL register may result in a glitch on the output line
|
||||
if the previous value in the OUTPUT_VAL register is different
|
||||
from the one we want to set.
|
||||
|
||||
In order to avoid that, update the output value before changing
|
||||
the direction.
|
||||
|
||||
Cc: stable@vger.kernel.org
|
||||
Fixes: 6702abb3bf23 ("pinctrl: armada-37xx: Fix direction_output() callback behavior")
|
||||
Signed-off-by: Imre Kaloz <kaloz@openwrt.org>
|
||||
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
|
||||
Signed-off-by: Gabor Juhos <j4g8y7@gmail.com>
|
||||
Link: https://lore.kernel.org/20250514-pinctrl-a37xx-fixes-v2-2-07e9ac1ab737@gmail.com
|
||||
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
|
||||
---
|
||||
drivers/pinctrl/mvebu/pinctrl-armada-37xx.c | 15 +++++++--------
|
||||
1 file changed, 7 insertions(+), 8 deletions(-)
|
||||
|
||||
--- a/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c
|
||||
+++ b/drivers/pinctrl/mvebu/pinctrl-armada-37xx.c
|
||||
@@ -417,23 +417,22 @@ static int armada_37xx_gpio_direction_ou
|
||||
unsigned int offset, int value)
|
||||
{
|
||||
struct armada_37xx_pinctrl *info = gpiochip_get_data(chip);
|
||||
- unsigned int val_offset = offset;
|
||||
- unsigned int reg = OUTPUT_EN;
|
||||
+ unsigned int en_offset = offset;
|
||||
+ unsigned int reg = OUTPUT_VAL;
|
||||
unsigned int mask, val, ret;
|
||||
|
||||
armada_37xx_update_reg(®, &offset);
|
||||
mask = BIT(offset);
|
||||
+ val = value ? mask : 0;
|
||||
|
||||
- ret = regmap_update_bits(info->regmap, reg, mask, mask);
|
||||
-
|
||||
+ ret = regmap_update_bits(info->regmap, reg, mask, val);
|
||||
if (ret)
|
||||
return ret;
|
||||
|
||||
- reg = OUTPUT_VAL;
|
||||
- armada_37xx_update_reg(®, &val_offset);
|
||||
+ reg = OUTPUT_EN;
|
||||
+ armada_37xx_update_reg(®, &en_offset);
|
||||
|
||||
- val = value ? mask : 0;
|
||||
- regmap_update_bits(info->regmap, reg, mask, val);
|
||||
+ regmap_update_bits(info->regmap, reg, mask, mask);
|
||||
|
||||
return 0;
|
||||
}
|
|
@ -48,7 +48,7 @@ Cc: stable@vger.kernel.org
|
|||
|
||||
--- a/drivers/pci/controller/pci-aardvark.c
|
||||
+++ b/drivers/pci/controller/pci-aardvark.c
|
||||
@@ -212,6 +212,8 @@ enum {
|
||||
@@ -211,6 +211,8 @@ enum {
|
||||
};
|
||||
|
||||
#define VENDOR_ID_REG (LMI_BASE_ADDR + 0x44)
|
||||
|
@ -57,7 +57,7 @@ Cc: stable@vger.kernel.org
|
|||
|
||||
/* PCIe core controller registers */
|
||||
#define CTRL_CORE_BASE_ADDR 0x18000
|
||||
@@ -560,6 +562,11 @@ static void advk_pcie_setup_hw(struct ad
|
||||
@@ -559,6 +561,11 @@ static void advk_pcie_setup_hw(struct ad
|
||||
PCIE_CORE_CTRL2_TD_ENABLE;
|
||||
advk_writel(pcie, reg, PCIE_CORE_CTRL2_REG);
|
||||
|
||||
|
@ -69,7 +69,7 @@ Cc: stable@vger.kernel.org
|
|||
/* Set lane X1 */
|
||||
reg = advk_readl(pcie, PCIE_CORE_CTRL0_REG);
|
||||
reg &= ~LANE_CNT_MSK;
|
||||
@@ -1661,6 +1668,9 @@ static irqreturn_t advk_pcie_irq_handler
|
||||
@@ -1654,6 +1661,9 @@ static irqreturn_t advk_pcie_irq_handler
|
||||
struct advk_pcie *pcie = arg;
|
||||
u32 status;
|
||||
|
||||
|
|
|
@ -28,7 +28,7 @@ Signed-off-by: Michael Gray <michael.gray@lantisproject.com>
|
|||
|
||||
--- a/arch/arm/Kconfig
|
||||
+++ b/arch/arm/Kconfig
|
||||
@@ -1566,6 +1566,17 @@ config ARM_ATAG_DTB_COMPAT_CMDLINE_EXTEN
|
||||
@@ -1503,6 +1503,17 @@ config ARM_ATAG_DTB_COMPAT_CMDLINE_EXTEN
|
||||
The command-line arguments provided by the boot loader will be
|
||||
appended to the the device tree bootargs property.
|
||||
|
||||
|
@ -247,7 +247,7 @@ Signed-off-by: Michael Gray <michael.gray@lantisproject.com>
|
|||
}
|
||||
--- a/init/main.c
|
||||
+++ b/init/main.c
|
||||
@@ -112,6 +112,10 @@
|
||||
@@ -114,6 +114,10 @@
|
||||
|
||||
#include <kunit/test.h>
|
||||
|
||||
|
@ -258,7 +258,7 @@ Signed-off-by: Michael Gray <michael.gray@lantisproject.com>
|
|||
static int kernel_init(void *);
|
||||
|
||||
/*
|
||||
@@ -929,6 +933,18 @@ void start_kernel(void)
|
||||
@@ -960,6 +964,18 @@ void start_kernel(void)
|
||||
boot_cpu_hotplug_init();
|
||||
|
||||
pr_notice("Kernel command line: %s\n", saved_command_line);
|
||||
|
@ -275,5 +275,5 @@ Signed-off-by: Michael Gray <michael.gray@lantisproject.com>
|
|||
+#endif
|
||||
+
|
||||
/* parameters may set static keys */
|
||||
jump_label_init();
|
||||
parse_early_param();
|
||||
after_dashes = parse_args("Booting kernel",
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
--- a/arch/arm/boot/dts/marvell/armada-385-linksys.dtsi
|
||||
+++ b/arch/arm/boot/dts/marvell/armada-385-linksys.dtsi
|
||||
@@ -214,11 +214,19 @@
|
||||
@@ -212,11 +212,19 @@
|
||||
&pcie1 {
|
||||
/* Marvell 88W8864, 5GHz-only */
|
||||
status = "okay";
|
||||
|
|
|
@ -1,6 +1,6 @@
|
|||
--- a/arch/arm/boot/dts/marvell/armada-xp-linksys-mamba.dts
|
||||
+++ b/arch/arm/boot/dts/marvell/armada-xp-linksys-mamba.dts
|
||||
@@ -483,3 +483,7 @@
|
||||
@@ -481,3 +481,7 @@
|
||||
};
|
||||
};
|
||||
};
|
||||
|
|
|
@ -1,11 +1,11 @@
|
|||
--- a/arch/arm/boot/dts/marvell/armada-xp-linksys-mamba.dts
|
||||
+++ b/arch/arm/boot/dts/marvell/armada-xp-linksys-mamba.dts
|
||||
@@ -385,7 +385,7 @@
|
||||
@@ -383,7 +383,7 @@
|
||||
|
||||
port@4 {
|
||||
ethernet-port@4 {
|
||||
reg = <4>;
|
||||
- label = "internet";
|
||||
+ label = "wan";
|
||||
};
|
||||
|
||||
port@5 {
|
||||
ethernet-port@5 {
|
||||
|
|
|
@ -9,13 +9,13 @@
|
|||
phy-mode = "sgmii";
|
||||
buffer-manager = <&bm>;
|
||||
bm,pool-long = <2>;
|
||||
@@ -200,10 +200,10 @@
|
||||
@@ -198,10 +198,10 @@
|
||||
label = "wan";
|
||||
};
|
||||
|
||||
- port@5 {
|
||||
- ethernet-port@5 {
|
||||
- reg = <5>;
|
||||
+ port@6 {
|
||||
+ ethernet-port@6 {
|
||||
+ reg = <6>;
|
||||
phy-mode = "sgmii";
|
||||
- ethernet = <ð2>;
|
||||
|
|
|
@ -11,7 +11,7 @@ Signed-off-by: Tad Davanzo <tad@spotco.us>
|
|||
|
||||
--- a/arch/arm/boot/dts/marvell/armada-xp-linksys-mamba.dts
|
||||
+++ b/arch/arm/boot/dts/marvell/armada-xp-linksys-mamba.dts
|
||||
@@ -454,9 +454,9 @@
|
||||
@@ -452,9 +452,9 @@
|
||||
reg = <0xa00000 0x2800000>; /* 40MB */
|
||||
};
|
||||
|
||||
|
@ -23,7 +23,7 @@ Signed-off-by: Tad Davanzo <tad@spotco.us>
|
|||
};
|
||||
|
||||
/* kernel2 overlaps with rootfs2 by design */
|
||||
@@ -465,9 +465,9 @@
|
||||
@@ -463,9 +463,9 @@
|
||||
reg = <0x3200000 0x2800000>; /* 40MB */
|
||||
};
|
||||
|
||||
|
|
|
@ -24,28 +24,25 @@ Signed-off-by: Daniel Golle <daniel@makrotopia.org>
|
|||
|
||||
--- a/drivers/thermal/gov_step_wise.c
|
||||
+++ b/drivers/thermal/gov_step_wise.c
|
||||
@@ -86,22 +86,31 @@ static void thermal_zone_trip_update(str
|
||||
@@ -74,19 +74,28 @@ static void thermal_zone_trip_update(str
|
||||
int trip_id = thermal_zone_trip_id(tz, trip);
|
||||
struct thermal_instance *instance;
|
||||
bool throttle = false;
|
||||
int old_target;
|
||||
+ int hyst_temp;
|
||||
|
||||
trend = get_tz_trend(tz, trip_id);
|
||||
|
||||
- if (tz->temperature >= trip->temperature) {
|
||||
- if (tz->temperature >= trip_threshold) {
|
||||
- throttle = true;
|
||||
- trace_thermal_zone_trip(tz, trip_id, trip->type);
|
||||
- }
|
||||
-
|
||||
- dev_dbg(&tz->device, "Trip%d[type=%d,temp=%d]:trend=%d,throttle=%d\n",
|
||||
- trip_id, trip->type, trip->temperature, trend, throttle);
|
||||
- trip_id, trip->type, trip_threshold, trend, throttle);
|
||||
+ hyst_temp = trip->temperature - trip->hysteresis;
|
||||
+ dev_dbg(&tz->device, "Trip%d[type=%d,temp=%d,hyst=%d]:trend=%d,throttle=%d\n",
|
||||
+ trip_id, trip->type, trip->temperature, hyst_temp, trend, throttle);
|
||||
|
||||
list_for_each_entry(instance, &tz->thermal_instances, tz_node) {
|
||||
if (instance->trip != trip)
|
||||
continue;
|
||||
list_for_each_entry(instance, &td->thermal_instances, trip_node) {
|
||||
int old_target;
|
||||
|
||||
old_target = instance->target;
|
||||
+ throttle = false;
|
||||
|
@ -61,5 +58,5 @@ Signed-off-by: Daniel Golle <daniel@makrotopia.org>
|
|||
+ }
|
||||
+
|
||||
instance->target = get_target_state(instance, trend, throttle);
|
||||
dev_dbg(&instance->cdev->device, "old_target=%d, target=%d\n",
|
||||
old_target, (int)instance->target);
|
||||
|
||||
dev_dbg(&instance->cdev->device, "old_target=%d, target=%ld\n",
|
||||
|
|
|
@ -38,7 +38,7 @@ Signed-off-by: Imre Kaloz <kaloz@openwrt.org>
|
|||
+ }
|
||||
parts[i].name = partname;
|
||||
|
||||
if (of_get_property(pp, "read-only", &len))
|
||||
if (of_property_read_bool(pp, "read-only"))
|
||||
@@ -271,6 +278,18 @@ static int __init ofpart_parser_init(voi
|
||||
return 0;
|
||||
}
|
||||
|
|
|
@ -14,7 +14,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name>
|
|||
---
|
||||
--- a/drivers/net/ethernet/marvell/mvneta.c
|
||||
+++ b/drivers/net/ethernet/marvell/mvneta.c
|
||||
@@ -5294,6 +5294,16 @@ static int mvneta_setup_tc(struct net_de
|
||||
@@ -5296,6 +5296,16 @@ static int mvneta_setup_tc(struct net_de
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -31,7 +31,7 @@ Signed-off-by: Felix Fietkau <nbd@nbd.name>
|
|||
static const struct net_device_ops mvneta_netdev_ops = {
|
||||
.ndo_open = mvneta_open,
|
||||
.ndo_stop = mvneta_stop,
|
||||
@@ -5304,6 +5314,9 @@ static const struct net_device_ops mvnet
|
||||
@@ -5306,6 +5316,9 @@ static const struct net_device_ops mvnet
|
||||
.ndo_fix_features = mvneta_fix_features,
|
||||
.ndo_get_stats64 = mvneta_get_stats64,
|
||||
.ndo_eth_ioctl = mvneta_ioctl,
|
||||
|
|
|
@ -12,7 +12,7 @@ Signed-off-by: Tobias Schramm <tobias@t-sys.eu>
|
|||
---
|
||||
--- a/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c
|
||||
+++ b/drivers/net/ethernet/marvell/mvpp2/mvpp2_main.c
|
||||
@@ -6156,6 +6156,12 @@ static int mvpp2_port_copy_mac_addr(stru
|
||||
@@ -6179,6 +6179,12 @@ static int mvpp2_port_copy_mac_addr(stru
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -1,125 +0,0 @@
|
|||
From cdfed4d05780450817ef96567e2cd8d355ca9e70 Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Marek=20Beh=C3=BAn?= <kabel@kernel.org>
|
||||
Date: Mon, 1 Jul 2024 13:30:03 +0200
|
||||
Subject: [PATCH 01/11] dt-bindings: firmware: add cznic,turris-omnia-mcu
|
||||
binding
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
Add binding for cznic,turris-omnia-mcu, the device-tree node
|
||||
representing the system-controller features provided by the MCU on the
|
||||
Turris Omnia router.
|
||||
|
||||
Signed-off-by: Marek Behún <kabel@kernel.org>
|
||||
Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
|
||||
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
|
||||
Reviewed-by: Andy Shevchenko <andy@kernel.org>
|
||||
Acked-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
|
||||
Link: https://lore.kernel.org/r/20240701113010.16447-2-kabel@kernel.org
|
||||
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
|
||||
---
|
||||
.../firmware/cznic,turris-omnia-mcu.yaml | 86 +++++++++++++++++++
|
||||
MAINTAINERS | 1 +
|
||||
2 files changed, 87 insertions(+)
|
||||
create mode 100644 Documentation/devicetree/bindings/firmware/cznic,turris-omnia-mcu.yaml
|
||||
|
||||
--- /dev/null
|
||||
+++ b/Documentation/devicetree/bindings/firmware/cznic,turris-omnia-mcu.yaml
|
||||
@@ -0,0 +1,86 @@
|
||||
+# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause)
|
||||
+%YAML 1.2
|
||||
+---
|
||||
+$id: http://devicetree.org/schemas/firmware/cznic,turris-omnia-mcu.yaml#
|
||||
+$schema: http://devicetree.org/meta-schemas/core.yaml#
|
||||
+
|
||||
+title: CZ.NIC's Turris Omnia MCU
|
||||
+
|
||||
+maintainers:
|
||||
+ - Marek Behún <kabel@kernel.org>
|
||||
+
|
||||
+description:
|
||||
+ The MCU on Turris Omnia acts as a system controller providing additional
|
||||
+ GPIOs, interrupts, watchdog, system power off and wakeup configuration.
|
||||
+
|
||||
+properties:
|
||||
+ compatible:
|
||||
+ const: cznic,turris-omnia-mcu
|
||||
+
|
||||
+ reg:
|
||||
+ description: MCU I2C slave address
|
||||
+ maxItems: 1
|
||||
+
|
||||
+ interrupts:
|
||||
+ maxItems: 1
|
||||
+
|
||||
+ interrupt-controller: true
|
||||
+
|
||||
+ '#interrupt-cells':
|
||||
+ const: 2
|
||||
+ description: |
|
||||
+ The first cell specifies the interrupt number (0 to 63), the second cell
|
||||
+ specifies interrupt type (which can be one of IRQ_TYPE_EDGE_RISING,
|
||||
+ IRQ_TYPE_EDGE_FALLING or IRQ_TYPE_EDGE_BOTH).
|
||||
+ The interrupt numbers correspond sequentially to GPIO numbers, taking the
|
||||
+ GPIO banks into account:
|
||||
+ IRQ number GPIO bank GPIO pin within bank
|
||||
+ 0 - 15 0 0 - 15
|
||||
+ 16 - 47 1 0 - 31
|
||||
+ 48 - 63 2 0 - 15
|
||||
+ There are several exceptions:
|
||||
+ IRQ number meaning
|
||||
+ 11 LED panel brightness changed by button press
|
||||
+ 13 TRNG entropy ready
|
||||
+ 14 ECDSA message signature computation done
|
||||
+
|
||||
+ gpio-controller: true
|
||||
+
|
||||
+ '#gpio-cells':
|
||||
+ const: 3
|
||||
+ description:
|
||||
+ The first cell is bank number (0, 1 or 2), the second cell is pin number
|
||||
+ within the bank (0 to 15 for banks 0 and 2, 0 to 31 for bank 1), and the
|
||||
+ third cell specifies consumer flags.
|
||||
+
|
||||
+required:
|
||||
+ - compatible
|
||||
+ - reg
|
||||
+ - interrupts
|
||||
+ - interrupt-controller
|
||||
+ - gpio-controller
|
||||
+
|
||||
+additionalProperties: false
|
||||
+
|
||||
+examples:
|
||||
+ - |
|
||||
+ #include <dt-bindings/interrupt-controller/irq.h>
|
||||
+
|
||||
+ i2c {
|
||||
+ #address-cells = <1>;
|
||||
+ #size-cells = <0>;
|
||||
+
|
||||
+ system-controller@2a {
|
||||
+ compatible = "cznic,turris-omnia-mcu";
|
||||
+ reg = <0x2a>;
|
||||
+
|
||||
+ interrupt-parent = <&gpio1>;
|
||||
+ interrupts = <11 IRQ_TYPE_NONE>;
|
||||
+
|
||||
+ gpio-controller;
|
||||
+ #gpio-cells = <3>;
|
||||
+
|
||||
+ interrupt-controller;
|
||||
+ #interrupt-cells = <2>;
|
||||
+ };
|
||||
+ };
|
||||
--- a/MAINTAINERS
|
||||
+++ b/MAINTAINERS
|
||||
@@ -2104,6 +2104,7 @@ F: Documentation/ABI/testing/sysfs-bus-m
|
||||
F: Documentation/ABI/testing/sysfs-firmware-turris-mox-rwtm
|
||||
F: Documentation/devicetree/bindings/bus/moxtet.txt
|
||||
F: Documentation/devicetree/bindings/firmware/cznic,turris-mox-rwtm.txt
|
||||
+F: Documentation/devicetree/bindings/firmware/cznic,turris-omnia-mcu.yaml
|
||||
F: Documentation/devicetree/bindings/gpio/gpio-moxtet.txt
|
||||
F: Documentation/devicetree/bindings/leds/cznic,turris-omnia-leds.yaml
|
||||
F: Documentation/devicetree/bindings/watchdog/armada-37xx-wdt.txt
|
|
@ -1,922 +0,0 @@
|
|||
From 4a63f684c8badfc43f384df2291ed2566909a3bc Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Marek=20Beh=C3=BAn?= <kabel@kernel.org>
|
||||
Date: Mon, 1 Jul 2024 13:30:04 +0200
|
||||
Subject: [PATCH 02/11] platform: cznic: Add preliminary support for Turris
|
||||
Omnia MCU
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
Add the basic skeleton for a new platform driver for the microcontroller
|
||||
found on the Turris Omnia board.
|
||||
|
||||
Signed-off-by: Marek Behún <kabel@kernel.org>
|
||||
Reviewed-by: Andy Shevchenko <andy@kernel.org>
|
||||
Acked-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
|
||||
Link: https://lore.kernel.org/r/20240701113010.16447-3-kabel@kernel.org
|
||||
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
|
||||
---
|
||||
.../sysfs-bus-i2c-devices-turris-omnia-mcu | 81 ++++
|
||||
MAINTAINERS | 3 +
|
||||
drivers/platform/Kconfig | 2 +
|
||||
drivers/platform/Makefile | 1 +
|
||||
drivers/platform/cznic/Kconfig | 25 ++
|
||||
drivers/platform/cznic/Makefile | 4 +
|
||||
.../platform/cznic/turris-omnia-mcu-base.c | 393 ++++++++++++++++++
|
||||
drivers/platform/cznic/turris-omnia-mcu.h | 74 ++++
|
||||
include/linux/turris-omnia-mcu-interface.h | 249 +++++++++++
|
||||
9 files changed, 832 insertions(+)
|
||||
create mode 100644 Documentation/ABI/testing/sysfs-bus-i2c-devices-turris-omnia-mcu
|
||||
create mode 100644 drivers/platform/cznic/Kconfig
|
||||
create mode 100644 drivers/platform/cznic/Makefile
|
||||
create mode 100644 drivers/platform/cznic/turris-omnia-mcu-base.c
|
||||
create mode 100644 drivers/platform/cznic/turris-omnia-mcu.h
|
||||
create mode 100644 include/linux/turris-omnia-mcu-interface.h
|
||||
|
||||
--- /dev/null
|
||||
+++ b/Documentation/ABI/testing/sysfs-bus-i2c-devices-turris-omnia-mcu
|
||||
@@ -0,0 +1,81 @@
|
||||
+What: /sys/bus/i2c/devices/<mcu_device>/board_revision
|
||||
+Date: September 2024
|
||||
+KernelVersion: 6.11
|
||||
+Contact: Marek Behún <kabel@kernel.org>
|
||||
+Description: (RO) Contains board revision number.
|
||||
+
|
||||
+ Only available if board information is burned in the MCU (older
|
||||
+ revisions have board information burned in the ATSHA204-A chip).
|
||||
+
|
||||
+ Format: %u.
|
||||
+
|
||||
+What: /sys/bus/i2c/devices/<mcu_device>/first_mac_address
|
||||
+Date: September 2024
|
||||
+KernelVersion: 6.11
|
||||
+Contact: Marek Behún <kabel@kernel.org>
|
||||
+Description: (RO) Contains device first MAC address. Each Turris Omnia is
|
||||
+ allocated 3 MAC addresses. The two additional addresses are
|
||||
+ computed from the first one by incrementing it.
|
||||
+
|
||||
+ Only available if board information is burned in the MCU (older
|
||||
+ revisions have board information burned in the ATSHA204-A chip).
|
||||
+
|
||||
+ Format: %pM.
|
||||
+
|
||||
+What: /sys/bus/i2c/devices/<mcu_device>/fw_features
|
||||
+Date: September 2024
|
||||
+KernelVersion: 6.11
|
||||
+Contact: Marek Behún <kabel@kernel.org>
|
||||
+Description: (RO) Newer versions of the microcontroller firmware report the
|
||||
+ features they support. These can be read from this file. If the
|
||||
+ MCU firmware is too old, this file reads 0x0.
|
||||
+
|
||||
+ Format: 0x%x.
|
||||
+
|
||||
+What: /sys/bus/i2c/devices/<mcu_device>/fw_version_hash_application
|
||||
+Date: September 2024
|
||||
+KernelVersion: 6.11
|
||||
+Contact: Marek Behún <kabel@kernel.org>
|
||||
+Description: (RO) Contains the version hash (commit hash) of the application
|
||||
+ part of the microcontroller firmware.
|
||||
+
|
||||
+ Format: %s.
|
||||
+
|
||||
+What: /sys/bus/i2c/devices/<mcu_device>/fw_version_hash_bootloader
|
||||
+Date: September 2024
|
||||
+KernelVersion: 6.11
|
||||
+Contact: Marek Behún <kabel@kernel.org>
|
||||
+Description: (RO) Contains the version hash (commit hash) of the bootloader
|
||||
+ part of the microcontroller firmware.
|
||||
+
|
||||
+ Format: %s.
|
||||
+
|
||||
+What: /sys/bus/i2c/devices/<mcu_device>/mcu_type
|
||||
+Date: September 2024
|
||||
+KernelVersion: 6.11
|
||||
+Contact: Marek Behún <kabel@kernel.org>
|
||||
+Description: (RO) Contains the microcontroller type (STM32, GD32, MKL).
|
||||
+
|
||||
+ Format: %s.
|
||||
+
|
||||
+What: /sys/bus/i2c/devices/<mcu_device>/reset_selector
|
||||
+Date: September 2024
|
||||
+KernelVersion: 6.11
|
||||
+Contact: Marek Behún <kabel@kernel.org>
|
||||
+Description: (RO) Contains the selected factory reset level, determined by
|
||||
+ how long the rear reset button was held by the user during board
|
||||
+ reset.
|
||||
+
|
||||
+ Format: %i.
|
||||
+
|
||||
+What: /sys/bus/i2c/devices/<mcu_device>/serial_number
|
||||
+Date: September 2024
|
||||
+KernelVersion: 6.11
|
||||
+Contact: Marek Behún <kabel@kernel.org>
|
||||
+Description: (RO) Contains the 64-bit board serial number in hexadecimal
|
||||
+ format.
|
||||
+
|
||||
+ Only available if board information is burned in the MCU (older
|
||||
+ revisions have board information burned in the ATSHA204-A chip).
|
||||
+
|
||||
+ Format: %016X.
|
||||
--- a/MAINTAINERS
|
||||
+++ b/MAINTAINERS
|
||||
@@ -2100,6 +2100,7 @@ M: Marek Behún <kabel@kernel.org>
|
||||
S: Maintained
|
||||
W: https://www.turris.cz/
|
||||
F: Documentation/ABI/testing/debugfs-moxtet
|
||||
+F: Documentation/ABI/testing/sysfs-bus-i2c-devices-turris-omnia-mcu
|
||||
F: Documentation/ABI/testing/sysfs-bus-moxtet-devices
|
||||
F: Documentation/ABI/testing/sysfs-firmware-turris-mox-rwtm
|
||||
F: Documentation/devicetree/bindings/bus/moxtet.txt
|
||||
@@ -2113,10 +2114,12 @@ F: drivers/firmware/turris-mox-rwtm.c
|
||||
F: drivers/gpio/gpio-moxtet.c
|
||||
F: drivers/leds/leds-turris-omnia.c
|
||||
F: drivers/mailbox/armada-37xx-rwtm-mailbox.c
|
||||
+F: drivers/platform/cznic/
|
||||
F: drivers/watchdog/armada_37xx_wdt.c
|
||||
F: include/dt-bindings/bus/moxtet.h
|
||||
F: include/linux/armada-37xx-rwtm-mailbox.h
|
||||
F: include/linux/moxtet.h
|
||||
+F: include/linux/turris-omnia-mcu-interface.h
|
||||
|
||||
ARM/FARADAY FA526 PORT
|
||||
M: Hans Ulli Kroll <ulli.kroll@googlemail.com>
|
||||
--- a/drivers/platform/Kconfig
|
||||
+++ b/drivers/platform/Kconfig
|
||||
@@ -7,6 +7,8 @@ source "drivers/platform/goldfish/Kconfi
|
||||
|
||||
source "drivers/platform/chrome/Kconfig"
|
||||
|
||||
+source "drivers/platform/cznic/Kconfig"
|
||||
+
|
||||
source "drivers/platform/mellanox/Kconfig"
|
||||
|
||||
source "drivers/platform/olpc/Kconfig"
|
||||
--- a/drivers/platform/Makefile
|
||||
+++ b/drivers/platform/Makefile
|
||||
@@ -10,5 +10,6 @@ obj-$(CONFIG_MIPS) += mips/
|
||||
obj-$(CONFIG_OLPC_EC) += olpc/
|
||||
obj-$(CONFIG_GOLDFISH) += goldfish/
|
||||
obj-$(CONFIG_CHROME_PLATFORMS) += chrome/
|
||||
+obj-$(CONFIG_CZNIC_PLATFORMS) += cznic/
|
||||
obj-$(CONFIG_SURFACE_PLATFORMS) += surface/
|
||||
obj-$(CONFIG_MIKROTIK) += mikrotik/
|
||||
--- /dev/null
|
||||
+++ b/drivers/platform/cznic/Kconfig
|
||||
@@ -0,0 +1,25 @@
|
||||
+# SPDX-License-Identifier: GPL-2.0-only
|
||||
+#
|
||||
+# For a description of the syntax of this configuration file,
|
||||
+# see Documentation/kbuild/kconfig-language.rst.
|
||||
+#
|
||||
+
|
||||
+menuconfig CZNIC_PLATFORMS
|
||||
+ bool "Platform support for CZ.NIC's Turris hardware"
|
||||
+ help
|
||||
+ Say Y here to be able to choose driver support for CZ.NIC's Turris
|
||||
+ devices. This option alone does not add any kernel code.
|
||||
+
|
||||
+if CZNIC_PLATFORMS
|
||||
+
|
||||
+config TURRIS_OMNIA_MCU
|
||||
+ tristate "Turris Omnia MCU driver"
|
||||
+ depends on MACH_ARMADA_38X || COMPILE_TEST
|
||||
+ depends on I2C
|
||||
+ help
|
||||
+ Say Y here to add support for the features implemented by the
|
||||
+ microcontroller on the CZ.NIC's Turris Omnia SOHO router.
|
||||
+ To compile this driver as a module, choose M here; the module will be
|
||||
+ called turris-omnia-mcu.
|
||||
+
|
||||
+endif # CZNIC_PLATFORMS
|
||||
--- /dev/null
|
||||
+++ b/drivers/platform/cznic/Makefile
|
||||
@@ -0,0 +1,4 @@
|
||||
+# SPDX-License-Identifier: GPL-2.0-only
|
||||
+
|
||||
+obj-$(CONFIG_TURRIS_OMNIA_MCU) += turris-omnia-mcu.o
|
||||
+turris-omnia-mcu-y := turris-omnia-mcu-base.o
|
||||
--- /dev/null
|
||||
+++ b/drivers/platform/cznic/turris-omnia-mcu-base.c
|
||||
@@ -0,0 +1,393 @@
|
||||
+// SPDX-License-Identifier: GPL-2.0
|
||||
+/*
|
||||
+ * CZ.NIC's Turris Omnia MCU driver
|
||||
+ *
|
||||
+ * 2024 by Marek Behún <kabel@kernel.org>
|
||||
+ */
|
||||
+
|
||||
+#include <linux/bits.h>
|
||||
+#include <linux/device.h>
|
||||
+#include <linux/errno.h>
|
||||
+#include <linux/hex.h>
|
||||
+#include <linux/i2c.h>
|
||||
+#include <linux/module.h>
|
||||
+#include <linux/string.h>
|
||||
+#include <linux/sysfs.h>
|
||||
+#include <linux/types.h>
|
||||
+
|
||||
+#include <linux/turris-omnia-mcu-interface.h>
|
||||
+#include "turris-omnia-mcu.h"
|
||||
+
|
||||
+#define OMNIA_FW_VERSION_LEN 20
|
||||
+#define OMNIA_FW_VERSION_HEX_LEN (2 * OMNIA_FW_VERSION_LEN + 1)
|
||||
+#define OMNIA_BOARD_INFO_LEN 16
|
||||
+
|
||||
+int omnia_cmd_write_read(const struct i2c_client *client,
|
||||
+ void *cmd, unsigned int cmd_len,
|
||||
+ void *reply, unsigned int reply_len)
|
||||
+{
|
||||
+ struct i2c_msg msgs[2];
|
||||
+ int ret, num;
|
||||
+
|
||||
+ msgs[0].addr = client->addr;
|
||||
+ msgs[0].flags = 0;
|
||||
+ msgs[0].len = cmd_len;
|
||||
+ msgs[0].buf = cmd;
|
||||
+ num = 1;
|
||||
+
|
||||
+ if (reply_len) {
|
||||
+ msgs[1].addr = client->addr;
|
||||
+ msgs[1].flags = I2C_M_RD;
|
||||
+ msgs[1].len = reply_len;
|
||||
+ msgs[1].buf = reply;
|
||||
+ num++;
|
||||
+ }
|
||||
+
|
||||
+ ret = i2c_transfer(client->adapter, msgs, num);
|
||||
+ if (ret < 0)
|
||||
+ return ret;
|
||||
+ if (ret != num)
|
||||
+ return -EIO;
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
+
|
||||
+static int omnia_get_version_hash(struct omnia_mcu *mcu, bool bootloader,
|
||||
+ char version[static OMNIA_FW_VERSION_HEX_LEN])
|
||||
+{
|
||||
+ u8 reply[OMNIA_FW_VERSION_LEN];
|
||||
+ char *p;
|
||||
+ int err;
|
||||
+
|
||||
+ err = omnia_cmd_read(mcu->client,
|
||||
+ bootloader ? OMNIA_CMD_GET_FW_VERSION_BOOT
|
||||
+ : OMNIA_CMD_GET_FW_VERSION_APP,
|
||||
+ reply, sizeof(reply));
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ p = bin2hex(version, reply, OMNIA_FW_VERSION_LEN);
|
||||
+ *p = '\0';
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
+
|
||||
+static ssize_t fw_version_hash_show(struct device *dev, char *buf,
|
||||
+ bool bootloader)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = dev_get_drvdata(dev);
|
||||
+ char version[OMNIA_FW_VERSION_HEX_LEN];
|
||||
+ int err;
|
||||
+
|
||||
+ err = omnia_get_version_hash(mcu, bootloader, version);
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ return sysfs_emit(buf, "%s\n", version);
|
||||
+}
|
||||
+
|
||||
+static ssize_t fw_version_hash_application_show(struct device *dev,
|
||||
+ struct device_attribute *a,
|
||||
+ char *buf)
|
||||
+{
|
||||
+ return fw_version_hash_show(dev, buf, false);
|
||||
+}
|
||||
+static DEVICE_ATTR_RO(fw_version_hash_application);
|
||||
+
|
||||
+static ssize_t fw_version_hash_bootloader_show(struct device *dev,
|
||||
+ struct device_attribute *a,
|
||||
+ char *buf)
|
||||
+{
|
||||
+ return fw_version_hash_show(dev, buf, true);
|
||||
+}
|
||||
+static DEVICE_ATTR_RO(fw_version_hash_bootloader);
|
||||
+
|
||||
+static ssize_t fw_features_show(struct device *dev, struct device_attribute *a,
|
||||
+ char *buf)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = dev_get_drvdata(dev);
|
||||
+
|
||||
+ return sysfs_emit(buf, "0x%x\n", mcu->features);
|
||||
+}
|
||||
+static DEVICE_ATTR_RO(fw_features);
|
||||
+
|
||||
+static ssize_t mcu_type_show(struct device *dev, struct device_attribute *a,
|
||||
+ char *buf)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = dev_get_drvdata(dev);
|
||||
+
|
||||
+ return sysfs_emit(buf, "%s\n", mcu->type);
|
||||
+}
|
||||
+static DEVICE_ATTR_RO(mcu_type);
|
||||
+
|
||||
+static ssize_t reset_selector_show(struct device *dev,
|
||||
+ struct device_attribute *a, char *buf)
|
||||
+{
|
||||
+ u8 reply;
|
||||
+ int err;
|
||||
+
|
||||
+ err = omnia_cmd_read_u8(to_i2c_client(dev), OMNIA_CMD_GET_RESET,
|
||||
+ &reply);
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ return sysfs_emit(buf, "%d\n", reply);
|
||||
+}
|
||||
+static DEVICE_ATTR_RO(reset_selector);
|
||||
+
|
||||
+static ssize_t serial_number_show(struct device *dev,
|
||||
+ struct device_attribute *a, char *buf)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = dev_get_drvdata(dev);
|
||||
+
|
||||
+ return sysfs_emit(buf, "%016llX\n", mcu->board_serial_number);
|
||||
+}
|
||||
+static DEVICE_ATTR_RO(serial_number);
|
||||
+
|
||||
+static ssize_t first_mac_address_show(struct device *dev,
|
||||
+ struct device_attribute *a, char *buf)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = dev_get_drvdata(dev);
|
||||
+
|
||||
+ return sysfs_emit(buf, "%pM\n", mcu->board_first_mac);
|
||||
+}
|
||||
+static DEVICE_ATTR_RO(first_mac_address);
|
||||
+
|
||||
+static ssize_t board_revision_show(struct device *dev,
|
||||
+ struct device_attribute *a, char *buf)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = dev_get_drvdata(dev);
|
||||
+
|
||||
+ return sysfs_emit(buf, "%u\n", mcu->board_revision);
|
||||
+}
|
||||
+static DEVICE_ATTR_RO(board_revision);
|
||||
+
|
||||
+static struct attribute *omnia_mcu_base_attrs[] = {
|
||||
+ &dev_attr_fw_version_hash_application.attr,
|
||||
+ &dev_attr_fw_version_hash_bootloader.attr,
|
||||
+ &dev_attr_fw_features.attr,
|
||||
+ &dev_attr_mcu_type.attr,
|
||||
+ &dev_attr_reset_selector.attr,
|
||||
+ &dev_attr_serial_number.attr,
|
||||
+ &dev_attr_first_mac_address.attr,
|
||||
+ &dev_attr_board_revision.attr,
|
||||
+ NULL
|
||||
+};
|
||||
+
|
||||
+static umode_t omnia_mcu_base_attrs_visible(struct kobject *kobj,
|
||||
+ struct attribute *a, int n)
|
||||
+{
|
||||
+ struct device *dev = kobj_to_dev(kobj);
|
||||
+ struct omnia_mcu *mcu = dev_get_drvdata(dev);
|
||||
+
|
||||
+ if ((a == &dev_attr_serial_number.attr ||
|
||||
+ a == &dev_attr_first_mac_address.attr ||
|
||||
+ a == &dev_attr_board_revision.attr) &&
|
||||
+ !(mcu->features & OMNIA_FEAT_BOARD_INFO))
|
||||
+ return 0;
|
||||
+
|
||||
+ return a->mode;
|
||||
+}
|
||||
+
|
||||
+static const struct attribute_group omnia_mcu_base_group = {
|
||||
+ .attrs = omnia_mcu_base_attrs,
|
||||
+ .is_visible = omnia_mcu_base_attrs_visible,
|
||||
+};
|
||||
+
|
||||
+static const struct attribute_group *omnia_mcu_groups[] = {
|
||||
+ &omnia_mcu_base_group,
|
||||
+ NULL
|
||||
+};
|
||||
+
|
||||
+static void omnia_mcu_print_version_hash(struct omnia_mcu *mcu, bool bootloader)
|
||||
+{
|
||||
+ const char *type = bootloader ? "bootloader" : "application";
|
||||
+ struct device *dev = &mcu->client->dev;
|
||||
+ char version[OMNIA_FW_VERSION_HEX_LEN];
|
||||
+ int err;
|
||||
+
|
||||
+ err = omnia_get_version_hash(mcu, bootloader, version);
|
||||
+ if (err) {
|
||||
+ dev_err(dev, "Cannot read MCU %s firmware version: %d\n",
|
||||
+ type, err);
|
||||
+ return;
|
||||
+ }
|
||||
+
|
||||
+ dev_info(dev, "MCU %s firmware version hash: %s\n", type, version);
|
||||
+}
|
||||
+
|
||||
+static const char *omnia_status_to_mcu_type(u16 status)
|
||||
+{
|
||||
+ switch (status & OMNIA_STS_MCU_TYPE_MASK) {
|
||||
+ case OMNIA_STS_MCU_TYPE_STM32:
|
||||
+ return "STM32";
|
||||
+ case OMNIA_STS_MCU_TYPE_GD32:
|
||||
+ return "GD32";
|
||||
+ case OMNIA_STS_MCU_TYPE_MKL:
|
||||
+ return "MKL";
|
||||
+ default:
|
||||
+ return "unknown";
|
||||
+ }
|
||||
+}
|
||||
+
|
||||
+static void omnia_info_missing_feature(struct device *dev, const char *feature)
|
||||
+{
|
||||
+ dev_info(dev,
|
||||
+ "Your board's MCU firmware does not support the %s feature.\n",
|
||||
+ feature);
|
||||
+}
|
||||
+
|
||||
+static int omnia_mcu_read_features(struct omnia_mcu *mcu)
|
||||
+{
|
||||
+ static const struct {
|
||||
+ u16 mask;
|
||||
+ const char *name;
|
||||
+ } features[] = {
|
||||
+#define _DEF_FEAT(_n, _m) { OMNIA_FEAT_ ## _n, _m }
|
||||
+ _DEF_FEAT(EXT_CMDS, "extended control and status"),
|
||||
+ _DEF_FEAT(WDT_PING, "watchdog pinging"),
|
||||
+ _DEF_FEAT(LED_STATE_EXT_MASK, "peripheral LED pins reading"),
|
||||
+ _DEF_FEAT(NEW_INT_API, "new interrupt API"),
|
||||
+ _DEF_FEAT(POWEROFF_WAKEUP, "poweroff and wakeup"),
|
||||
+ _DEF_FEAT(TRNG, "true random number generator"),
|
||||
+#undef _DEF_FEAT
|
||||
+ };
|
||||
+ struct i2c_client *client = mcu->client;
|
||||
+ struct device *dev = &client->dev;
|
||||
+ bool suggest_fw_upgrade = false;
|
||||
+ u16 status;
|
||||
+ int err;
|
||||
+
|
||||
+ /* status word holds MCU type, which we need below */
|
||||
+ err = omnia_cmd_read_u16(client, OMNIA_CMD_GET_STATUS_WORD, &status);
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ /*
|
||||
+ * Check whether MCU firmware supports the OMNIA_CMD_GET_FEATURES
|
||||
+ * command.
|
||||
+ */
|
||||
+ if (status & OMNIA_STS_FEATURES_SUPPORTED) {
|
||||
+ /* try read 32-bit features */
|
||||
+ err = omnia_cmd_read_u32(client, OMNIA_CMD_GET_FEATURES,
|
||||
+ &mcu->features);
|
||||
+ if (err) {
|
||||
+ /* try read 16-bit features */
|
||||
+ u16 features16;
|
||||
+
|
||||
+ err = omnia_cmd_read_u16(client, OMNIA_CMD_GET_FEATURES,
|
||||
+ &features16);
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ mcu->features = features16;
|
||||
+ } else {
|
||||
+ if (mcu->features & OMNIA_FEAT_FROM_BIT_16_INVALID)
|
||||
+ mcu->features &= GENMASK(15, 0);
|
||||
+ }
|
||||
+ } else {
|
||||
+ dev_info(dev,
|
||||
+ "Your board's MCU firmware does not support feature reading.\n");
|
||||
+ suggest_fw_upgrade = true;
|
||||
+ }
|
||||
+
|
||||
+ mcu->type = omnia_status_to_mcu_type(status);
|
||||
+ dev_info(dev, "MCU type %s%s\n", mcu->type,
|
||||
+ (mcu->features & OMNIA_FEAT_PERIPH_MCU) ?
|
||||
+ ", with peripheral resets wired" : "");
|
||||
+
|
||||
+ omnia_mcu_print_version_hash(mcu, true);
|
||||
+
|
||||
+ if (mcu->features & OMNIA_FEAT_BOOTLOADER)
|
||||
+ dev_warn(dev,
|
||||
+ "MCU is running bootloader firmware. Was firmware upgrade interrupted?\n");
|
||||
+ else
|
||||
+ omnia_mcu_print_version_hash(mcu, false);
|
||||
+
|
||||
+ for (unsigned int i = 0; i < ARRAY_SIZE(features); i++) {
|
||||
+ if (mcu->features & features[i].mask)
|
||||
+ continue;
|
||||
+
|
||||
+ omnia_info_missing_feature(dev, features[i].name);
|
||||
+ suggest_fw_upgrade = true;
|
||||
+ }
|
||||
+
|
||||
+ if (suggest_fw_upgrade)
|
||||
+ dev_info(dev,
|
||||
+ "Consider upgrading MCU firmware with the omnia-mcutool utility.\n");
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
+
|
||||
+static int omnia_mcu_read_board_info(struct omnia_mcu *mcu)
|
||||
+{
|
||||
+ u8 reply[1 + OMNIA_BOARD_INFO_LEN];
|
||||
+ int err;
|
||||
+
|
||||
+ err = omnia_cmd_read(mcu->client, OMNIA_CMD_BOARD_INFO_GET, reply,
|
||||
+ sizeof(reply));
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ if (reply[0] != OMNIA_BOARD_INFO_LEN)
|
||||
+ return -EIO;
|
||||
+
|
||||
+ mcu->board_serial_number = get_unaligned_le64(&reply[1]);
|
||||
+
|
||||
+ /* we can't use ether_addr_copy() because reply is not u16-aligned */
|
||||
+ memcpy(mcu->board_first_mac, &reply[9], sizeof(mcu->board_first_mac));
|
||||
+
|
||||
+ mcu->board_revision = reply[15];
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
+
|
||||
+static int omnia_mcu_probe(struct i2c_client *client)
|
||||
+{
|
||||
+ struct device *dev = &client->dev;
|
||||
+ struct omnia_mcu *mcu;
|
||||
+ int err;
|
||||
+
|
||||
+ if (!client->irq)
|
||||
+ return dev_err_probe(dev, -EINVAL, "IRQ resource not found\n");
|
||||
+
|
||||
+ mcu = devm_kzalloc(dev, sizeof(*mcu), GFP_KERNEL);
|
||||
+ if (!mcu)
|
||||
+ return -ENOMEM;
|
||||
+
|
||||
+ mcu->client = client;
|
||||
+ i2c_set_clientdata(client, mcu);
|
||||
+
|
||||
+ err = omnia_mcu_read_features(mcu);
|
||||
+ if (err)
|
||||
+ return dev_err_probe(dev, err,
|
||||
+ "Cannot determine MCU supported features\n");
|
||||
+
|
||||
+ if (mcu->features & OMNIA_FEAT_BOARD_INFO) {
|
||||
+ err = omnia_mcu_read_board_info(mcu);
|
||||
+ if (err)
|
||||
+ return dev_err_probe(dev, err,
|
||||
+ "Cannot read board info\n");
|
||||
+ }
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
+
|
||||
+static const struct of_device_id of_omnia_mcu_match[] = {
|
||||
+ { .compatible = "cznic,turris-omnia-mcu" },
|
||||
+ {}
|
||||
+};
|
||||
+
|
||||
+static struct i2c_driver omnia_mcu_driver = {
|
||||
+ .probe = omnia_mcu_probe,
|
||||
+ .driver = {
|
||||
+ .name = "turris-omnia-mcu",
|
||||
+ .of_match_table = of_omnia_mcu_match,
|
||||
+ .dev_groups = omnia_mcu_groups,
|
||||
+ },
|
||||
+};
|
||||
+module_i2c_driver(omnia_mcu_driver);
|
||||
+
|
||||
+MODULE_AUTHOR("Marek Behun <kabel@kernel.org>");
|
||||
+MODULE_DESCRIPTION("CZ.NIC's Turris Omnia MCU");
|
||||
+MODULE_LICENSE("GPL");
|
||||
--- /dev/null
|
||||
+++ b/drivers/platform/cznic/turris-omnia-mcu.h
|
||||
@@ -0,0 +1,74 @@
|
||||
+/* SPDX-License-Identifier: GPL-2.0 */
|
||||
+/*
|
||||
+ * CZ.NIC's Turris Omnia MCU driver
|
||||
+ *
|
||||
+ * 2024 by Marek Behún <kabel@kernel.org>
|
||||
+ */
|
||||
+
|
||||
+#ifndef __TURRIS_OMNIA_MCU_H
|
||||
+#define __TURRIS_OMNIA_MCU_H
|
||||
+
|
||||
+#include <linux/if_ether.h>
|
||||
+#include <linux/types.h>
|
||||
+#include <asm/byteorder.h>
|
||||
+
|
||||
+struct i2c_client;
|
||||
+
|
||||
+struct omnia_mcu {
|
||||
+ struct i2c_client *client;
|
||||
+ const char *type;
|
||||
+ u32 features;
|
||||
+
|
||||
+ /* board information */
|
||||
+ u64 board_serial_number;
|
||||
+ u8 board_first_mac[ETH_ALEN];
|
||||
+ u8 board_revision;
|
||||
+};
|
||||
+
|
||||
+int omnia_cmd_write_read(const struct i2c_client *client,
|
||||
+ void *cmd, unsigned int cmd_len,
|
||||
+ void *reply, unsigned int reply_len);
|
||||
+
|
||||
+static inline int omnia_cmd_read(const struct i2c_client *client, u8 cmd,
|
||||
+ void *reply, unsigned int len)
|
||||
+{
|
||||
+ return omnia_cmd_write_read(client, &cmd, 1, reply, len);
|
||||
+}
|
||||
+
|
||||
+static inline int omnia_cmd_read_u32(const struct i2c_client *client, u8 cmd,
|
||||
+ u32 *dst)
|
||||
+{
|
||||
+ __le32 reply;
|
||||
+ int err;
|
||||
+
|
||||
+ err = omnia_cmd_read(client, cmd, &reply, sizeof(reply));
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ *dst = le32_to_cpu(reply);
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
+
|
||||
+static inline int omnia_cmd_read_u16(const struct i2c_client *client, u8 cmd,
|
||||
+ u16 *dst)
|
||||
+{
|
||||
+ __le16 reply;
|
||||
+ int err;
|
||||
+
|
||||
+ err = omnia_cmd_read(client, cmd, &reply, sizeof(reply));
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ *dst = le16_to_cpu(reply);
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
+
|
||||
+static inline int omnia_cmd_read_u8(const struct i2c_client *client, u8 cmd,
|
||||
+ u8 *reply)
|
||||
+{
|
||||
+ return omnia_cmd_read(client, cmd, reply, sizeof(*reply));
|
||||
+}
|
||||
+
|
||||
+#endif /* __TURRIS_OMNIA_MCU_H */
|
||||
--- /dev/null
|
||||
+++ b/include/linux/turris-omnia-mcu-interface.h
|
||||
@@ -0,0 +1,249 @@
|
||||
+/* SPDX-License-Identifier: GPL-2.0 */
|
||||
+/*
|
||||
+ * CZ.NIC's Turris Omnia MCU I2C interface commands definitions
|
||||
+ *
|
||||
+ * 2024 by Marek Behún <kabel@kernel.org>
|
||||
+ */
|
||||
+
|
||||
+#ifndef __TURRIS_OMNIA_MCU_INTERFACE_H
|
||||
+#define __TURRIS_OMNIA_MCU_INTERFACE_H
|
||||
+
|
||||
+#include <linux/bitfield.h>
|
||||
+#include <linux/bits.h>
|
||||
+
|
||||
+enum omnia_commands_e {
|
||||
+ OMNIA_CMD_GET_STATUS_WORD = 0x01, /* slave sends status word back */
|
||||
+ OMNIA_CMD_GENERAL_CONTROL = 0x02,
|
||||
+ OMNIA_CMD_LED_MODE = 0x03, /* default/user */
|
||||
+ OMNIA_CMD_LED_STATE = 0x04, /* LED on/off */
|
||||
+ OMNIA_CMD_LED_COLOR = 0x05, /* LED number + RED + GREEN + BLUE */
|
||||
+ OMNIA_CMD_USER_VOLTAGE = 0x06,
|
||||
+ OMNIA_CMD_SET_BRIGHTNESS = 0x07,
|
||||
+ OMNIA_CMD_GET_BRIGHTNESS = 0x08,
|
||||
+ OMNIA_CMD_GET_RESET = 0x09,
|
||||
+ OMNIA_CMD_GET_FW_VERSION_APP = 0x0A, /* 20B git hash number */
|
||||
+ OMNIA_CMD_SET_WATCHDOG_STATE = 0x0B, /* 0 - disable
|
||||
+ * 1 - enable / ping
|
||||
+ * after boot watchdog is started
|
||||
+ * with 2 minutes timeout
|
||||
+ */
|
||||
+
|
||||
+ /* OMNIA_CMD_WATCHDOG_STATUS = 0x0C, not implemented anymore */
|
||||
+
|
||||
+ OMNIA_CMD_GET_WATCHDOG_STATE = 0x0D,
|
||||
+ OMNIA_CMD_GET_FW_VERSION_BOOT = 0x0E, /* 20B Git hash number */
|
||||
+ OMNIA_CMD_GET_FW_CHECKSUM = 0x0F, /* 4B length, 4B checksum */
|
||||
+
|
||||
+ /* available if FEATURES_SUPPORTED bit set in status word */
|
||||
+ OMNIA_CMD_GET_FEATURES = 0x10,
|
||||
+
|
||||
+ /* available if EXT_CMD bit set in features */
|
||||
+ OMNIA_CMD_GET_EXT_STATUS_DWORD = 0x11,
|
||||
+ OMNIA_CMD_EXT_CONTROL = 0x12,
|
||||
+ OMNIA_CMD_GET_EXT_CONTROL_STATUS = 0x13,
|
||||
+
|
||||
+ /* available if NEW_INT_API bit set in features */
|
||||
+ OMNIA_CMD_GET_INT_AND_CLEAR = 0x14,
|
||||
+ OMNIA_CMD_GET_INT_MASK = 0x15,
|
||||
+ OMNIA_CMD_SET_INT_MASK = 0x16,
|
||||
+
|
||||
+ /* available if FLASHING bit set in features */
|
||||
+ OMNIA_CMD_FLASH = 0x19,
|
||||
+
|
||||
+ /* available if WDT_PING bit set in features */
|
||||
+ OMNIA_CMD_SET_WDT_TIMEOUT = 0x20,
|
||||
+ OMNIA_CMD_GET_WDT_TIMELEFT = 0x21,
|
||||
+
|
||||
+ /* available if POWEROFF_WAKEUP bit set in features */
|
||||
+ OMNIA_CMD_SET_WAKEUP = 0x22,
|
||||
+ OMNIA_CMD_GET_UPTIME_AND_WAKEUP = 0x23,
|
||||
+ OMNIA_CMD_POWER_OFF = 0x24,
|
||||
+
|
||||
+ /* available if USB_OVC_PROT_SETTING bit set in features */
|
||||
+ OMNIA_CMD_SET_USB_OVC_PROT = 0x25,
|
||||
+ OMNIA_CMD_GET_USB_OVC_PROT = 0x26,
|
||||
+
|
||||
+ /* available if TRNG bit set in features */
|
||||
+ OMNIA_CMD_TRNG_COLLECT_ENTROPY = 0x28,
|
||||
+
|
||||
+ /* available if CRYPTO bit set in features */
|
||||
+ OMNIA_CMD_CRYPTO_GET_PUBLIC_KEY = 0x29,
|
||||
+ OMNIA_CMD_CRYPTO_SIGN_MESSAGE = 0x2A,
|
||||
+ OMNIA_CMD_CRYPTO_COLLECT_SIGNATURE = 0x2B,
|
||||
+
|
||||
+ /* available if BOARD_INFO it set in features */
|
||||
+ OMNIA_CMD_BOARD_INFO_GET = 0x2C,
|
||||
+ OMNIA_CMD_BOARD_INFO_BURN = 0x2D,
|
||||
+
|
||||
+ /* available only at address 0x2b (LED-controller) */
|
||||
+ /* available only if LED_GAMMA_CORRECTION bit set in features */
|
||||
+ OMNIA_CMD_SET_GAMMA_CORRECTION = 0x30,
|
||||
+ OMNIA_CMD_GET_GAMMA_CORRECTION = 0x31,
|
||||
+
|
||||
+ /* available only at address 0x2b (LED-controller) */
|
||||
+ /* available only if PER_LED_CORRECTION bit set in features */
|
||||
+ /* available only if FROM_BIT_16_INVALID bit NOT set in features */
|
||||
+ OMNIA_CMD_SET_LED_CORRECTIONS = 0x32,
|
||||
+ OMNIA_CMD_GET_LED_CORRECTIONS = 0x33,
|
||||
+};
|
||||
+
|
||||
+enum omnia_flashing_commands_e {
|
||||
+ OMNIA_FLASH_CMD_UNLOCK = 0x01,
|
||||
+ OMNIA_FLASH_CMD_SIZE_AND_CSUM = 0x02,
|
||||
+ OMNIA_FLASH_CMD_PROGRAM = 0x03,
|
||||
+ OMNIA_FLASH_CMD_RESET = 0x04,
|
||||
+};
|
||||
+
|
||||
+enum omnia_sts_word_e {
|
||||
+ OMNIA_STS_MCU_TYPE_MASK = GENMASK(1, 0),
|
||||
+ OMNIA_STS_MCU_TYPE_STM32 = FIELD_PREP_CONST(OMNIA_STS_MCU_TYPE_MASK, 0),
|
||||
+ OMNIA_STS_MCU_TYPE_GD32 = FIELD_PREP_CONST(OMNIA_STS_MCU_TYPE_MASK, 1),
|
||||
+ OMNIA_STS_MCU_TYPE_MKL = FIELD_PREP_CONST(OMNIA_STS_MCU_TYPE_MASK, 2),
|
||||
+ OMNIA_STS_FEATURES_SUPPORTED = BIT(2),
|
||||
+ OMNIA_STS_USER_REGULATOR_NOT_SUPPORTED = BIT(3),
|
||||
+ OMNIA_STS_CARD_DET = BIT(4),
|
||||
+ OMNIA_STS_MSATA_IND = BIT(5),
|
||||
+ OMNIA_STS_USB30_OVC = BIT(6),
|
||||
+ OMNIA_STS_USB31_OVC = BIT(7),
|
||||
+ OMNIA_STS_USB30_PWRON = BIT(8),
|
||||
+ OMNIA_STS_USB31_PWRON = BIT(9),
|
||||
+ OMNIA_STS_ENABLE_4V5 = BIT(10),
|
||||
+ OMNIA_STS_BUTTON_MODE = BIT(11),
|
||||
+ OMNIA_STS_BUTTON_PRESSED = BIT(12),
|
||||
+ OMNIA_STS_BUTTON_COUNTER_MASK = GENMASK(15, 13),
|
||||
+};
|
||||
+
|
||||
+enum omnia_ctl_byte_e {
|
||||
+ OMNIA_CTL_LIGHT_RST = BIT(0),
|
||||
+ OMNIA_CTL_HARD_RST = BIT(1),
|
||||
+ /* BIT(2) is currently reserved */
|
||||
+ OMNIA_CTL_USB30_PWRON = BIT(3),
|
||||
+ OMNIA_CTL_USB31_PWRON = BIT(4),
|
||||
+ OMNIA_CTL_ENABLE_4V5 = BIT(5),
|
||||
+ OMNIA_CTL_BUTTON_MODE = BIT(6),
|
||||
+ OMNIA_CTL_BOOTLOADER = BIT(7),
|
||||
+};
|
||||
+
|
||||
+enum omnia_features_e {
|
||||
+ OMNIA_FEAT_PERIPH_MCU = BIT(0),
|
||||
+ OMNIA_FEAT_EXT_CMDS = BIT(1),
|
||||
+ OMNIA_FEAT_WDT_PING = BIT(2),
|
||||
+ OMNIA_FEAT_LED_STATE_EXT_MASK = GENMASK(4, 3),
|
||||
+ OMNIA_FEAT_LED_STATE_EXT = FIELD_PREP_CONST(OMNIA_FEAT_LED_STATE_EXT_MASK, 1),
|
||||
+ OMNIA_FEAT_LED_STATE_EXT_V32 = FIELD_PREP_CONST(OMNIA_FEAT_LED_STATE_EXT_MASK, 2),
|
||||
+ OMNIA_FEAT_LED_GAMMA_CORRECTION = BIT(5),
|
||||
+ OMNIA_FEAT_NEW_INT_API = BIT(6),
|
||||
+ OMNIA_FEAT_BOOTLOADER = BIT(7),
|
||||
+ OMNIA_FEAT_FLASHING = BIT(8),
|
||||
+ OMNIA_FEAT_NEW_MESSAGE_API = BIT(9),
|
||||
+ OMNIA_FEAT_BRIGHTNESS_INT = BIT(10),
|
||||
+ OMNIA_FEAT_POWEROFF_WAKEUP = BIT(11),
|
||||
+ OMNIA_FEAT_CAN_OLD_MESSAGE_API = BIT(12),
|
||||
+ OMNIA_FEAT_TRNG = BIT(13),
|
||||
+ OMNIA_FEAT_CRYPTO = BIT(14),
|
||||
+ OMNIA_FEAT_BOARD_INFO = BIT(15),
|
||||
+
|
||||
+ /*
|
||||
+ * Orginally the features command replied only 16 bits. If more were
|
||||
+ * read, either the I2C transaction failed or 0xff bytes were sent.
|
||||
+ * Therefore to consider bits 16 - 31 valid, one bit (20) was reserved
|
||||
+ * to be zero.
|
||||
+ */
|
||||
+
|
||||
+ /* Bits 16 - 19 correspond to bits 0 - 3 of status word */
|
||||
+ OMNIA_FEAT_MCU_TYPE_MASK = GENMASK(17, 16),
|
||||
+ OMNIA_FEAT_MCU_TYPE_STM32 = FIELD_PREP_CONST(OMNIA_FEAT_MCU_TYPE_MASK, 0),
|
||||
+ OMNIA_FEAT_MCU_TYPE_GD32 = FIELD_PREP_CONST(OMNIA_FEAT_MCU_TYPE_MASK, 1),
|
||||
+ OMNIA_FEAT_MCU_TYPE_MKL = FIELD_PREP_CONST(OMNIA_FEAT_MCU_TYPE_MASK, 2),
|
||||
+ OMNIA_FEAT_FEATURES_SUPPORTED = BIT(18),
|
||||
+ OMNIA_FEAT_USER_REGULATOR_NOT_SUPPORTED = BIT(19),
|
||||
+
|
||||
+ /* must not be set */
|
||||
+ OMNIA_FEAT_FROM_BIT_16_INVALID = BIT(20),
|
||||
+
|
||||
+ OMNIA_FEAT_PER_LED_CORRECTION = BIT(21),
|
||||
+ OMNIA_FEAT_USB_OVC_PROT_SETTING = BIT(22),
|
||||
+};
|
||||
+
|
||||
+enum omnia_ext_sts_dword_e {
|
||||
+ OMNIA_EXT_STS_SFP_nDET = BIT(0),
|
||||
+ OMNIA_EXT_STS_LED_STATES_MASK = GENMASK(31, 12),
|
||||
+ OMNIA_EXT_STS_WLAN0_MSATA_LED = BIT(12),
|
||||
+ OMNIA_EXT_STS_WLAN1_LED = BIT(13),
|
||||
+ OMNIA_EXT_STS_WLAN2_LED = BIT(14),
|
||||
+ OMNIA_EXT_STS_WPAN0_LED = BIT(15),
|
||||
+ OMNIA_EXT_STS_WPAN1_LED = BIT(16),
|
||||
+ OMNIA_EXT_STS_WPAN2_LED = BIT(17),
|
||||
+ OMNIA_EXT_STS_WAN_LED0 = BIT(18),
|
||||
+ OMNIA_EXT_STS_WAN_LED1 = BIT(19),
|
||||
+ OMNIA_EXT_STS_LAN0_LED0 = BIT(20),
|
||||
+ OMNIA_EXT_STS_LAN0_LED1 = BIT(21),
|
||||
+ OMNIA_EXT_STS_LAN1_LED0 = BIT(22),
|
||||
+ OMNIA_EXT_STS_LAN1_LED1 = BIT(23),
|
||||
+ OMNIA_EXT_STS_LAN2_LED0 = BIT(24),
|
||||
+ OMNIA_EXT_STS_LAN2_LED1 = BIT(25),
|
||||
+ OMNIA_EXT_STS_LAN3_LED0 = BIT(26),
|
||||
+ OMNIA_EXT_STS_LAN3_LED1 = BIT(27),
|
||||
+ OMNIA_EXT_STS_LAN4_LED0 = BIT(28),
|
||||
+ OMNIA_EXT_STS_LAN4_LED1 = BIT(29),
|
||||
+ OMNIA_EXT_STS_LAN5_LED0 = BIT(30),
|
||||
+ OMNIA_EXT_STS_LAN5_LED1 = BIT(31),
|
||||
+};
|
||||
+
|
||||
+enum omnia_ext_ctl_e {
|
||||
+ OMNIA_EXT_CTL_nRES_MMC = BIT(0),
|
||||
+ OMNIA_EXT_CTL_nRES_LAN = BIT(1),
|
||||
+ OMNIA_EXT_CTL_nRES_PHY = BIT(2),
|
||||
+ OMNIA_EXT_CTL_nPERST0 = BIT(3),
|
||||
+ OMNIA_EXT_CTL_nPERST1 = BIT(4),
|
||||
+ OMNIA_EXT_CTL_nPERST2 = BIT(5),
|
||||
+ OMNIA_EXT_CTL_PHY_SFP = BIT(6),
|
||||
+ OMNIA_EXT_CTL_PHY_SFP_AUTO = BIT(7),
|
||||
+ OMNIA_EXT_CTL_nVHV_CTRL = BIT(8),
|
||||
+};
|
||||
+
|
||||
+enum omnia_int_e {
|
||||
+ OMNIA_INT_CARD_DET = BIT(0),
|
||||
+ OMNIA_INT_MSATA_IND = BIT(1),
|
||||
+ OMNIA_INT_USB30_OVC = BIT(2),
|
||||
+ OMNIA_INT_USB31_OVC = BIT(3),
|
||||
+ OMNIA_INT_BUTTON_PRESSED = BIT(4),
|
||||
+ OMNIA_INT_SFP_nDET = BIT(5),
|
||||
+ OMNIA_INT_BRIGHTNESS_CHANGED = BIT(6),
|
||||
+ OMNIA_INT_TRNG = BIT(7),
|
||||
+ OMNIA_INT_MESSAGE_SIGNED = BIT(8),
|
||||
+
|
||||
+ OMNIA_INT_LED_STATES_MASK = GENMASK(31, 12),
|
||||
+ OMNIA_INT_WLAN0_MSATA_LED = BIT(12),
|
||||
+ OMNIA_INT_WLAN1_LED = BIT(13),
|
||||
+ OMNIA_INT_WLAN2_LED = BIT(14),
|
||||
+ OMNIA_INT_WPAN0_LED = BIT(15),
|
||||
+ OMNIA_INT_WPAN1_LED = BIT(16),
|
||||
+ OMNIA_INT_WPAN2_LED = BIT(17),
|
||||
+ OMNIA_INT_WAN_LED0 = BIT(18),
|
||||
+ OMNIA_INT_WAN_LED1 = BIT(19),
|
||||
+ OMNIA_INT_LAN0_LED0 = BIT(20),
|
||||
+ OMNIA_INT_LAN0_LED1 = BIT(21),
|
||||
+ OMNIA_INT_LAN1_LED0 = BIT(22),
|
||||
+ OMNIA_INT_LAN1_LED1 = BIT(23),
|
||||
+ OMNIA_INT_LAN2_LED0 = BIT(24),
|
||||
+ OMNIA_INT_LAN2_LED1 = BIT(25),
|
||||
+ OMNIA_INT_LAN3_LED0 = BIT(26),
|
||||
+ OMNIA_INT_LAN3_LED1 = BIT(27),
|
||||
+ OMNIA_INT_LAN4_LED0 = BIT(28),
|
||||
+ OMNIA_INT_LAN4_LED1 = BIT(29),
|
||||
+ OMNIA_INT_LAN5_LED0 = BIT(30),
|
||||
+ OMNIA_INT_LAN5_LED1 = BIT(31),
|
||||
+};
|
||||
+
|
||||
+enum omnia_cmd_poweroff_e {
|
||||
+ OMNIA_CMD_POWER_OFF_POWERON_BUTTON = BIT(0),
|
||||
+ OMNIA_CMD_POWER_OFF_MAGIC = 0xdead,
|
||||
+};
|
||||
+
|
||||
+enum omnia_cmd_usb_ovc_prot_e {
|
||||
+ OMNIA_CMD_xET_USB_OVC_PROT_PORT_MASK = GENMASK(3, 0),
|
||||
+ OMNIA_CMD_xET_USB_OVC_PROT_ENABLE = BIT(4),
|
||||
+};
|
||||
+
|
||||
+#endif /* __TURRIS_OMNIA_MCU_INTERFACE_H */
|
File diff suppressed because it is too large
Load diff
|
@ -1,415 +0,0 @@
|
|||
From f69e0a731ab471f3a57c48258ad2d9990820c173 Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Marek=20Beh=C3=BAn?= <kabel@kernel.org>
|
||||
Date: Mon, 1 Jul 2024 13:30:06 +0200
|
||||
Subject: [PATCH 04/11] platform: cznic: turris-omnia-mcu: Add support for
|
||||
poweroff and wakeup
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
Add support for true board poweroff (MCU can disable all unnecessary
|
||||
voltage regulators) and wakeup at a specified time, implemented via a
|
||||
RTC driver so that the rtcwake utility can be used to configure it.
|
||||
|
||||
Signed-off-by: Marek Behún <kabel@kernel.org>
|
||||
Reviewed-by: Andy Shevchenko <andy@kernel.org>
|
||||
Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
|
||||
Acked-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
|
||||
Link: https://lore.kernel.org/r/20240701113010.16447-5-kabel@kernel.org
|
||||
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
|
||||
---
|
||||
.../sysfs-bus-i2c-devices-turris-omnia-mcu | 16 ++
|
||||
drivers/platform/cznic/Kconfig | 4 +
|
||||
drivers/platform/cznic/Makefile | 1 +
|
||||
.../platform/cznic/turris-omnia-mcu-base.c | 5 +
|
||||
.../cznic/turris-omnia-mcu-sys-off-wakeup.c | 260 ++++++++++++++++++
|
||||
drivers/platform/cznic/turris-omnia-mcu.h | 20 ++
|
||||
6 files changed, 306 insertions(+)
|
||||
create mode 100644 drivers/platform/cznic/turris-omnia-mcu-sys-off-wakeup.c
|
||||
|
||||
--- a/Documentation/ABI/testing/sysfs-bus-i2c-devices-turris-omnia-mcu
|
||||
+++ b/Documentation/ABI/testing/sysfs-bus-i2c-devices-turris-omnia-mcu
|
||||
@@ -38,6 +38,22 @@ Description: (RW) The front button on th
|
||||
|
||||
Format: %s.
|
||||
|
||||
+What: /sys/bus/i2c/devices/<mcu_device>/front_button_poweron
|
||||
+Date: September 2024
|
||||
+KernelVersion: 6.11
|
||||
+Contact: Marek Behún <kabel@kernel.org>
|
||||
+Description: (RW) Newer versions of the microcontroller firmware of the
|
||||
+ Turris Omnia router support powering off the router into true
|
||||
+ low power mode. The router can be powered on by pressing the
|
||||
+ front button.
|
||||
+
|
||||
+ This file configures whether front button power on is enabled.
|
||||
+
|
||||
+ This file is present only if the power off feature is supported
|
||||
+ by the firmware.
|
||||
+
|
||||
+ Format: %i.
|
||||
+
|
||||
What: /sys/bus/i2c/devices/<mcu_device>/fw_features
|
||||
Date: September 2024
|
||||
KernelVersion: 6.11
|
||||
--- a/drivers/platform/cznic/Kconfig
|
||||
+++ b/drivers/platform/cznic/Kconfig
|
||||
@@ -18,10 +18,14 @@ config TURRIS_OMNIA_MCU
|
||||
depends on I2C
|
||||
select GPIOLIB
|
||||
select GPIOLIB_IRQCHIP
|
||||
+ select RTC_CLASS
|
||||
help
|
||||
Say Y here to add support for the features implemented by the
|
||||
microcontroller on the CZ.NIC's Turris Omnia SOHO router.
|
||||
The features include:
|
||||
+ - board poweroff into true low power mode (with voltage regulators
|
||||
+ disabled) and the ability to configure wake up from this mode (via
|
||||
+ rtcwake)
|
||||
- GPIO pins
|
||||
- to get front button press events (the front button can be
|
||||
configured either to generate press events to the CPU or to change
|
||||
--- a/drivers/platform/cznic/Makefile
|
||||
+++ b/drivers/platform/cznic/Makefile
|
||||
@@ -3,3 +3,4 @@
|
||||
obj-$(CONFIG_TURRIS_OMNIA_MCU) += turris-omnia-mcu.o
|
||||
turris-omnia-mcu-y := turris-omnia-mcu-base.o
|
||||
turris-omnia-mcu-y += turris-omnia-mcu-gpio.o
|
||||
+turris-omnia-mcu-y += turris-omnia-mcu-sys-off-wakeup.o
|
||||
--- a/drivers/platform/cznic/turris-omnia-mcu-base.c
|
||||
+++ b/drivers/platform/cznic/turris-omnia-mcu-base.c
|
||||
@@ -197,6 +197,7 @@ static const struct attribute_group omni
|
||||
static const struct attribute_group *omnia_mcu_groups[] = {
|
||||
&omnia_mcu_base_group,
|
||||
&omnia_mcu_gpio_group,
|
||||
+ &omnia_mcu_poweroff_group,
|
||||
NULL
|
||||
};
|
||||
|
||||
@@ -371,6 +372,10 @@ static int omnia_mcu_probe(struct i2c_cl
|
||||
"Cannot read board info\n");
|
||||
}
|
||||
|
||||
+ err = omnia_mcu_register_sys_off_and_wakeup(mcu);
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
return omnia_mcu_register_gpiochip(mcu);
|
||||
}
|
||||
|
||||
--- /dev/null
|
||||
+++ b/drivers/platform/cznic/turris-omnia-mcu-sys-off-wakeup.c
|
||||
@@ -0,0 +1,260 @@
|
||||
+// SPDX-License-Identifier: GPL-2.0
|
||||
+/*
|
||||
+ * CZ.NIC's Turris Omnia MCU system off and RTC wakeup driver
|
||||
+ *
|
||||
+ * This is not a true RTC driver (in the sense that it does not provide a
|
||||
+ * real-time clock), rather the MCU implements a wakeup from powered off state
|
||||
+ * at a specified time relative to MCU boot, and we expose this feature via RTC
|
||||
+ * alarm, so that it can be used via the rtcwake command, which is the standard
|
||||
+ * Linux command for this.
|
||||
+ *
|
||||
+ * 2024 by Marek Behún <kabel@kernel.org>
|
||||
+ */
|
||||
+
|
||||
+#include <linux/crc32.h>
|
||||
+#include <linux/delay.h>
|
||||
+#include <linux/device.h>
|
||||
+#include <linux/err.h>
|
||||
+#include <linux/i2c.h>
|
||||
+#include <linux/kstrtox.h>
|
||||
+#include <linux/reboot.h>
|
||||
+#include <linux/rtc.h>
|
||||
+#include <linux/sysfs.h>
|
||||
+#include <linux/types.h>
|
||||
+
|
||||
+#include <linux/turris-omnia-mcu-interface.h>
|
||||
+#include "turris-omnia-mcu.h"
|
||||
+
|
||||
+static int omnia_get_uptime_wakeup(const struct i2c_client *client, u32 *uptime,
|
||||
+ u32 *wakeup)
|
||||
+{
|
||||
+ __le32 reply[2];
|
||||
+ int err;
|
||||
+
|
||||
+ err = omnia_cmd_read(client, OMNIA_CMD_GET_UPTIME_AND_WAKEUP, reply,
|
||||
+ sizeof(reply));
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ if (uptime)
|
||||
+ *uptime = le32_to_cpu(reply[0]);
|
||||
+
|
||||
+ if (wakeup)
|
||||
+ *wakeup = le32_to_cpu(reply[1]);
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
+
|
||||
+static int omnia_read_time(struct device *dev, struct rtc_time *tm)
|
||||
+{
|
||||
+ u32 uptime;
|
||||
+ int err;
|
||||
+
|
||||
+ err = omnia_get_uptime_wakeup(to_i2c_client(dev), &uptime, NULL);
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ rtc_time64_to_tm(uptime, tm);
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
+
|
||||
+static int omnia_read_alarm(struct device *dev, struct rtc_wkalrm *alrm)
|
||||
+{
|
||||
+ struct i2c_client *client = to_i2c_client(dev);
|
||||
+ struct omnia_mcu *mcu = i2c_get_clientdata(client);
|
||||
+ u32 wakeup;
|
||||
+ int err;
|
||||
+
|
||||
+ err = omnia_get_uptime_wakeup(client, NULL, &wakeup);
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ alrm->enabled = !!wakeup;
|
||||
+ rtc_time64_to_tm(wakeup ?: mcu->rtc_alarm, &alrm->time);
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
+
|
||||
+static int omnia_set_alarm(struct device *dev, struct rtc_wkalrm *alrm)
|
||||
+{
|
||||
+ struct i2c_client *client = to_i2c_client(dev);
|
||||
+ struct omnia_mcu *mcu = i2c_get_clientdata(client);
|
||||
+
|
||||
+ mcu->rtc_alarm = rtc_tm_to_time64(&alrm->time);
|
||||
+
|
||||
+ if (alrm->enabled)
|
||||
+ return omnia_cmd_write_u32(client, OMNIA_CMD_SET_WAKEUP,
|
||||
+ mcu->rtc_alarm);
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
+
|
||||
+static int omnia_alarm_irq_enable(struct device *dev, unsigned int enabled)
|
||||
+{
|
||||
+ struct i2c_client *client = to_i2c_client(dev);
|
||||
+ struct omnia_mcu *mcu = i2c_get_clientdata(client);
|
||||
+
|
||||
+ return omnia_cmd_write_u32(client, OMNIA_CMD_SET_WAKEUP,
|
||||
+ enabled ? mcu->rtc_alarm : 0);
|
||||
+}
|
||||
+
|
||||
+static const struct rtc_class_ops omnia_rtc_ops = {
|
||||
+ .read_time = omnia_read_time,
|
||||
+ .read_alarm = omnia_read_alarm,
|
||||
+ .set_alarm = omnia_set_alarm,
|
||||
+ .alarm_irq_enable = omnia_alarm_irq_enable,
|
||||
+};
|
||||
+
|
||||
+static int omnia_power_off(struct sys_off_data *data)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = data->cb_data;
|
||||
+ __be32 tmp;
|
||||
+ u8 cmd[9];
|
||||
+ u16 arg;
|
||||
+ int err;
|
||||
+
|
||||
+ if (mcu->front_button_poweron)
|
||||
+ arg = OMNIA_CMD_POWER_OFF_POWERON_BUTTON;
|
||||
+ else
|
||||
+ arg = 0;
|
||||
+
|
||||
+ cmd[0] = OMNIA_CMD_POWER_OFF;
|
||||
+ put_unaligned_le16(OMNIA_CMD_POWER_OFF_MAGIC, &cmd[1]);
|
||||
+ put_unaligned_le16(arg, &cmd[3]);
|
||||
+
|
||||
+ /*
|
||||
+ * Although all values from and to MCU are passed in little-endian, the
|
||||
+ * MCU's CRC unit uses big-endian CRC32 polynomial (0x04c11db7), so we
|
||||
+ * need to use crc32_be() here.
|
||||
+ */
|
||||
+ tmp = cpu_to_be32(get_unaligned_le32(&cmd[1]));
|
||||
+ put_unaligned_le32(crc32_be(~0, (void *)&tmp, sizeof(tmp)), &cmd[5]);
|
||||
+
|
||||
+ err = omnia_cmd_write(mcu->client, cmd, sizeof(cmd));
|
||||
+ if (err)
|
||||
+ dev_err(&mcu->client->dev,
|
||||
+ "Unable to send the poweroff command: %d\n", err);
|
||||
+
|
||||
+ return NOTIFY_DONE;
|
||||
+}
|
||||
+
|
||||
+static int omnia_restart(struct sys_off_data *data)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = data->cb_data;
|
||||
+ u8 cmd[3];
|
||||
+ int err;
|
||||
+
|
||||
+ cmd[0] = OMNIA_CMD_GENERAL_CONTROL;
|
||||
+
|
||||
+ if (reboot_mode == REBOOT_HARD)
|
||||
+ cmd[1] = cmd[2] = OMNIA_CTL_HARD_RST;
|
||||
+ else
|
||||
+ cmd[1] = cmd[2] = OMNIA_CTL_LIGHT_RST;
|
||||
+
|
||||
+ err = omnia_cmd_write(mcu->client, cmd, sizeof(cmd));
|
||||
+ if (err)
|
||||
+ dev_err(&mcu->client->dev,
|
||||
+ "Unable to send the restart command: %d\n", err);
|
||||
+
|
||||
+ /*
|
||||
+ * MCU needs a little bit to process the I2C command, otherwise it will
|
||||
+ * do a light reset based on SOC SYSRES_OUT pin.
|
||||
+ */
|
||||
+ mdelay(1);
|
||||
+
|
||||
+ return NOTIFY_DONE;
|
||||
+}
|
||||
+
|
||||
+static ssize_t front_button_poweron_show(struct device *dev,
|
||||
+ struct device_attribute *a, char *buf)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = dev_get_drvdata(dev);
|
||||
+
|
||||
+ return sysfs_emit(buf, "%d\n", mcu->front_button_poweron);
|
||||
+}
|
||||
+
|
||||
+static ssize_t front_button_poweron_store(struct device *dev,
|
||||
+ struct device_attribute *a,
|
||||
+ const char *buf, size_t count)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = dev_get_drvdata(dev);
|
||||
+ bool val;
|
||||
+ int err;
|
||||
+
|
||||
+ err = kstrtobool(buf, &val);
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ mcu->front_button_poweron = val;
|
||||
+
|
||||
+ return count;
|
||||
+}
|
||||
+static DEVICE_ATTR_RW(front_button_poweron);
|
||||
+
|
||||
+static struct attribute *omnia_mcu_poweroff_attrs[] = {
|
||||
+ &dev_attr_front_button_poweron.attr,
|
||||
+ NULL
|
||||
+};
|
||||
+
|
||||
+static umode_t poweroff_attrs_visible(struct kobject *kobj, struct attribute *a,
|
||||
+ int n)
|
||||
+{
|
||||
+ struct device *dev = kobj_to_dev(kobj);
|
||||
+ struct omnia_mcu *mcu = dev_get_drvdata(dev);
|
||||
+
|
||||
+ if (mcu->features & OMNIA_FEAT_POWEROFF_WAKEUP)
|
||||
+ return a->mode;
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
+
|
||||
+const struct attribute_group omnia_mcu_poweroff_group = {
|
||||
+ .attrs = omnia_mcu_poweroff_attrs,
|
||||
+ .is_visible = poweroff_attrs_visible,
|
||||
+};
|
||||
+
|
||||
+int omnia_mcu_register_sys_off_and_wakeup(struct omnia_mcu *mcu)
|
||||
+{
|
||||
+ struct device *dev = &mcu->client->dev;
|
||||
+ int err;
|
||||
+
|
||||
+ /* MCU restart is always available */
|
||||
+ err = devm_register_sys_off_handler(dev, SYS_OFF_MODE_RESTART,
|
||||
+ SYS_OFF_PRIO_FIRMWARE,
|
||||
+ omnia_restart, mcu);
|
||||
+ if (err)
|
||||
+ return dev_err_probe(dev, err,
|
||||
+ "Cannot register system restart handler\n");
|
||||
+
|
||||
+ /*
|
||||
+ * Poweroff and wakeup are available only if POWEROFF_WAKEUP feature is
|
||||
+ * present.
|
||||
+ */
|
||||
+ if (!(mcu->features & OMNIA_FEAT_POWEROFF_WAKEUP))
|
||||
+ return 0;
|
||||
+
|
||||
+ err = devm_register_sys_off_handler(dev, SYS_OFF_MODE_POWER_OFF,
|
||||
+ SYS_OFF_PRIO_FIRMWARE,
|
||||
+ omnia_power_off, mcu);
|
||||
+ if (err)
|
||||
+ return dev_err_probe(dev, err,
|
||||
+ "Cannot register system power off handler\n");
|
||||
+
|
||||
+ mcu->rtcdev = devm_rtc_allocate_device(dev);
|
||||
+ if (IS_ERR(mcu->rtcdev))
|
||||
+ return dev_err_probe(dev, PTR_ERR(mcu->rtcdev),
|
||||
+ "Cannot allocate RTC device\n");
|
||||
+
|
||||
+ mcu->rtcdev->ops = &omnia_rtc_ops;
|
||||
+ mcu->rtcdev->range_max = U32_MAX;
|
||||
+ set_bit(RTC_FEATURE_ALARM_WAKEUP_ONLY, mcu->rtcdev->features);
|
||||
+
|
||||
+ err = devm_rtc_register_device(mcu->rtcdev);
|
||||
+ if (err)
|
||||
+ return dev_err_probe(dev, err, "Cannot register RTC device\n");
|
||||
+
|
||||
+ mcu->front_button_poweron = true;
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
--- a/drivers/platform/cznic/turris-omnia-mcu.h
|
||||
+++ b/drivers/platform/cznic/turris-omnia-mcu.h
|
||||
@@ -15,8 +15,10 @@
|
||||
#include <linux/types.h>
|
||||
#include <linux/workqueue.h>
|
||||
#include <asm/byteorder.h>
|
||||
+#include <asm/unaligned.h>
|
||||
|
||||
struct i2c_client;
|
||||
+struct rtc_device;
|
||||
|
||||
struct omnia_mcu {
|
||||
struct i2c_client *client;
|
||||
@@ -36,6 +38,11 @@ struct omnia_mcu {
|
||||
struct delayed_work button_release_emul_work;
|
||||
unsigned long last_status;
|
||||
bool button_pressed_emul;
|
||||
+
|
||||
+ /* RTC device for configuring wake-up */
|
||||
+ struct rtc_device *rtcdev;
|
||||
+ u32 rtc_alarm;
|
||||
+ bool front_button_poweron;
|
||||
};
|
||||
|
||||
int omnia_cmd_write_read(const struct i2c_client *client,
|
||||
@@ -48,6 +55,17 @@ static inline int omnia_cmd_write(const
|
||||
return omnia_cmd_write_read(client, cmd, len, NULL, 0);
|
||||
}
|
||||
|
||||
+static inline int omnia_cmd_write_u32(const struct i2c_client *client, u8 cmd,
|
||||
+ u32 val)
|
||||
+{
|
||||
+ u8 buf[5];
|
||||
+
|
||||
+ buf[0] = cmd;
|
||||
+ put_unaligned_le32(val, &buf[1]);
|
||||
+
|
||||
+ return omnia_cmd_write(client, buf, sizeof(buf));
|
||||
+}
|
||||
+
|
||||
static inline int omnia_cmd_read(const struct i2c_client *client, u8 cmd,
|
||||
void *reply, unsigned int len)
|
||||
{
|
||||
@@ -136,7 +154,9 @@ static inline int omnia_cmd_read_u8(cons
|
||||
}
|
||||
|
||||
extern const struct attribute_group omnia_mcu_gpio_group;
|
||||
+extern const struct attribute_group omnia_mcu_poweroff_group;
|
||||
|
||||
int omnia_mcu_register_gpiochip(struct omnia_mcu *mcu);
|
||||
+int omnia_mcu_register_sys_off_and_wakeup(struct omnia_mcu *mcu);
|
||||
|
||||
#endif /* __TURRIS_OMNIA_MCU_H */
|
|
@ -1,250 +0,0 @@
|
|||
From 33ae4e4c86bc6ff298489fb8b743e2743dd0af6d Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Marek=20Beh=C3=BAn?= <kabel@kernel.org>
|
||||
Date: Mon, 1 Jul 2024 13:30:07 +0200
|
||||
Subject: [PATCH 05/11] platform: cznic: turris-omnia-mcu: Add support for MCU
|
||||
watchdog
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
Add support for the watchdog mechanism provided by the MCU.
|
||||
|
||||
Signed-off-by: Marek Behún <kabel@kernel.org>
|
||||
Reviewed-by: Andy Shevchenko <andy@kernel.org>
|
||||
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
|
||||
Acked-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
|
||||
Link: https://lore.kernel.org/r/20240701113010.16447-6-kabel@kernel.org
|
||||
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
|
||||
---
|
||||
drivers/platform/cznic/Kconfig | 2 +
|
||||
drivers/platform/cznic/Makefile | 1 +
|
||||
.../platform/cznic/turris-omnia-mcu-base.c | 4 +
|
||||
.../cznic/turris-omnia-mcu-watchdog.c | 130 ++++++++++++++++++
|
||||
drivers/platform/cznic/turris-omnia-mcu.h | 24 ++++
|
||||
5 files changed, 161 insertions(+)
|
||||
create mode 100644 drivers/platform/cznic/turris-omnia-mcu-watchdog.c
|
||||
|
||||
--- a/drivers/platform/cznic/Kconfig
|
||||
+++ b/drivers/platform/cznic/Kconfig
|
||||
@@ -19,6 +19,7 @@ config TURRIS_OMNIA_MCU
|
||||
select GPIOLIB
|
||||
select GPIOLIB_IRQCHIP
|
||||
select RTC_CLASS
|
||||
+ select WATCHDOG_CORE
|
||||
help
|
||||
Say Y here to add support for the features implemented by the
|
||||
microcontroller on the CZ.NIC's Turris Omnia SOHO router.
|
||||
@@ -26,6 +27,7 @@ config TURRIS_OMNIA_MCU
|
||||
- board poweroff into true low power mode (with voltage regulators
|
||||
disabled) and the ability to configure wake up from this mode (via
|
||||
rtcwake)
|
||||
+ - MCU watchdog
|
||||
- GPIO pins
|
||||
- to get front button press events (the front button can be
|
||||
configured either to generate press events to the CPU or to change
|
||||
--- a/drivers/platform/cznic/Makefile
|
||||
+++ b/drivers/platform/cznic/Makefile
|
||||
@@ -4,3 +4,4 @@ obj-$(CONFIG_TURRIS_OMNIA_MCU) += turris
|
||||
turris-omnia-mcu-y := turris-omnia-mcu-base.o
|
||||
turris-omnia-mcu-y += turris-omnia-mcu-gpio.o
|
||||
turris-omnia-mcu-y += turris-omnia-mcu-sys-off-wakeup.o
|
||||
+turris-omnia-mcu-y += turris-omnia-mcu-watchdog.o
|
||||
--- a/drivers/platform/cznic/turris-omnia-mcu-base.c
|
||||
+++ b/drivers/platform/cznic/turris-omnia-mcu-base.c
|
||||
@@ -376,6 +376,10 @@ static int omnia_mcu_probe(struct i2c_cl
|
||||
if (err)
|
||||
return err;
|
||||
|
||||
+ err = omnia_mcu_register_watchdog(mcu);
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
return omnia_mcu_register_gpiochip(mcu);
|
||||
}
|
||||
|
||||
--- /dev/null
|
||||
+++ b/drivers/platform/cznic/turris-omnia-mcu-watchdog.c
|
||||
@@ -0,0 +1,130 @@
|
||||
+// SPDX-License-Identifier: GPL-2.0
|
||||
+/*
|
||||
+ * CZ.NIC's Turris Omnia MCU watchdog driver
|
||||
+ *
|
||||
+ * 2024 by Marek Behún <kabel@kernel.org>
|
||||
+ */
|
||||
+
|
||||
+#include <linux/bitops.h>
|
||||
+#include <linux/device.h>
|
||||
+#include <linux/i2c.h>
|
||||
+#include <linux/moduleparam.h>
|
||||
+#include <linux/types.h>
|
||||
+#include <linux/units.h>
|
||||
+#include <linux/watchdog.h>
|
||||
+
|
||||
+#include <linux/turris-omnia-mcu-interface.h>
|
||||
+#include "turris-omnia-mcu.h"
|
||||
+
|
||||
+#define WATCHDOG_TIMEOUT 120
|
||||
+
|
||||
+static unsigned int timeout;
|
||||
+module_param(timeout, int, 0);
|
||||
+MODULE_PARM_DESC(timeout, "Watchdog timeout in seconds");
|
||||
+
|
||||
+static bool nowayout = WATCHDOG_NOWAYOUT;
|
||||
+module_param(nowayout, bool, 0);
|
||||
+MODULE_PARM_DESC(nowayout, "Watchdog cannot be stopped once started (default="
|
||||
+ __MODULE_STRING(WATCHDOG_NOWAYOUT) ")");
|
||||
+
|
||||
+static int omnia_wdt_start(struct watchdog_device *wdt)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = watchdog_get_drvdata(wdt);
|
||||
+
|
||||
+ return omnia_cmd_write_u8(mcu->client, OMNIA_CMD_SET_WATCHDOG_STATE, 1);
|
||||
+}
|
||||
+
|
||||
+static int omnia_wdt_stop(struct watchdog_device *wdt)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = watchdog_get_drvdata(wdt);
|
||||
+
|
||||
+ return omnia_cmd_write_u8(mcu->client, OMNIA_CMD_SET_WATCHDOG_STATE, 0);
|
||||
+}
|
||||
+
|
||||
+static int omnia_wdt_ping(struct watchdog_device *wdt)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = watchdog_get_drvdata(wdt);
|
||||
+
|
||||
+ return omnia_cmd_write_u8(mcu->client, OMNIA_CMD_SET_WATCHDOG_STATE, 1);
|
||||
+}
|
||||
+
|
||||
+static int omnia_wdt_set_timeout(struct watchdog_device *wdt,
|
||||
+ unsigned int timeout)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = watchdog_get_drvdata(wdt);
|
||||
+
|
||||
+ return omnia_cmd_write_u16(mcu->client, OMNIA_CMD_SET_WDT_TIMEOUT,
|
||||
+ timeout * DECI);
|
||||
+}
|
||||
+
|
||||
+static unsigned int omnia_wdt_get_timeleft(struct watchdog_device *wdt)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = watchdog_get_drvdata(wdt);
|
||||
+ u16 timeleft;
|
||||
+ int err;
|
||||
+
|
||||
+ err = omnia_cmd_read_u16(mcu->client, OMNIA_CMD_GET_WDT_TIMELEFT,
|
||||
+ &timeleft);
|
||||
+ if (err) {
|
||||
+ dev_err(&mcu->client->dev, "Cannot get watchdog timeleft: %d\n",
|
||||
+ err);
|
||||
+ return 0;
|
||||
+ }
|
||||
+
|
||||
+ return timeleft / DECI;
|
||||
+}
|
||||
+
|
||||
+static const struct watchdog_info omnia_wdt_info = {
|
||||
+ .options = WDIOF_SETTIMEOUT | WDIOF_KEEPALIVEPING | WDIOF_MAGICCLOSE,
|
||||
+ .identity = "Turris Omnia MCU Watchdog",
|
||||
+};
|
||||
+
|
||||
+static const struct watchdog_ops omnia_wdt_ops = {
|
||||
+ .owner = THIS_MODULE,
|
||||
+ .start = omnia_wdt_start,
|
||||
+ .stop = omnia_wdt_stop,
|
||||
+ .ping = omnia_wdt_ping,
|
||||
+ .set_timeout = omnia_wdt_set_timeout,
|
||||
+ .get_timeleft = omnia_wdt_get_timeleft,
|
||||
+};
|
||||
+
|
||||
+int omnia_mcu_register_watchdog(struct omnia_mcu *mcu)
|
||||
+{
|
||||
+ struct device *dev = &mcu->client->dev;
|
||||
+ u8 state;
|
||||
+ int err;
|
||||
+
|
||||
+ if (!(mcu->features & OMNIA_FEAT_WDT_PING))
|
||||
+ return 0;
|
||||
+
|
||||
+ mcu->wdt.info = &omnia_wdt_info;
|
||||
+ mcu->wdt.ops = &omnia_wdt_ops;
|
||||
+ mcu->wdt.parent = dev;
|
||||
+ mcu->wdt.min_timeout = 1;
|
||||
+ mcu->wdt.max_timeout = 65535 / DECI;
|
||||
+
|
||||
+ mcu->wdt.timeout = WATCHDOG_TIMEOUT;
|
||||
+ watchdog_init_timeout(&mcu->wdt, timeout, dev);
|
||||
+
|
||||
+ watchdog_set_drvdata(&mcu->wdt, mcu);
|
||||
+
|
||||
+ omnia_wdt_set_timeout(&mcu->wdt, mcu->wdt.timeout);
|
||||
+
|
||||
+ err = omnia_cmd_read_u8(mcu->client, OMNIA_CMD_GET_WATCHDOG_STATE,
|
||||
+ &state);
|
||||
+ if (err)
|
||||
+ return dev_err_probe(dev, err,
|
||||
+ "Cannot get MCU watchdog state\n");
|
||||
+
|
||||
+ if (state)
|
||||
+ set_bit(WDOG_HW_RUNNING, &mcu->wdt.status);
|
||||
+
|
||||
+ watchdog_set_nowayout(&mcu->wdt, nowayout);
|
||||
+ watchdog_stop_on_reboot(&mcu->wdt);
|
||||
+ err = devm_watchdog_register_device(dev, &mcu->wdt);
|
||||
+ if (err)
|
||||
+ return dev_err_probe(dev, err,
|
||||
+ "Cannot register MCU watchdog\n");
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
--- a/drivers/platform/cznic/turris-omnia-mcu.h
|
||||
+++ b/drivers/platform/cznic/turris-omnia-mcu.h
|
||||
@@ -13,6 +13,7 @@
|
||||
#include <linux/if_ether.h>
|
||||
#include <linux/mutex.h>
|
||||
#include <linux/types.h>
|
||||
+#include <linux/watchdog.h>
|
||||
#include <linux/workqueue.h>
|
||||
#include <asm/byteorder.h>
|
||||
#include <asm/unaligned.h>
|
||||
@@ -43,6 +44,9 @@ struct omnia_mcu {
|
||||
struct rtc_device *rtcdev;
|
||||
u32 rtc_alarm;
|
||||
bool front_button_poweron;
|
||||
+
|
||||
+ /* MCU watchdog */
|
||||
+ struct watchdog_device wdt;
|
||||
};
|
||||
|
||||
int omnia_cmd_write_read(const struct i2c_client *client,
|
||||
@@ -55,6 +59,25 @@ static inline int omnia_cmd_write(const
|
||||
return omnia_cmd_write_read(client, cmd, len, NULL, 0);
|
||||
}
|
||||
|
||||
+static inline int omnia_cmd_write_u8(const struct i2c_client *client, u8 cmd,
|
||||
+ u8 val)
|
||||
+{
|
||||
+ u8 buf[2] = { cmd, val };
|
||||
+
|
||||
+ return omnia_cmd_write(client, buf, sizeof(buf));
|
||||
+}
|
||||
+
|
||||
+static inline int omnia_cmd_write_u16(const struct i2c_client *client, u8 cmd,
|
||||
+ u16 val)
|
||||
+{
|
||||
+ u8 buf[3];
|
||||
+
|
||||
+ buf[0] = cmd;
|
||||
+ put_unaligned_le16(val, &buf[1]);
|
||||
+
|
||||
+ return omnia_cmd_write(client, buf, sizeof(buf));
|
||||
+}
|
||||
+
|
||||
static inline int omnia_cmd_write_u32(const struct i2c_client *client, u8 cmd,
|
||||
u32 val)
|
||||
{
|
||||
@@ -158,5 +181,6 @@ extern const struct attribute_group omni
|
||||
|
||||
int omnia_mcu_register_gpiochip(struct omnia_mcu *mcu);
|
||||
int omnia_mcu_register_sys_off_and_wakeup(struct omnia_mcu *mcu);
|
||||
+int omnia_mcu_register_watchdog(struct omnia_mcu *mcu);
|
||||
|
||||
#endif /* __TURRIS_OMNIA_MCU_H */
|
|
@ -1,225 +0,0 @@
|
|||
From b3ed8645c45567b598bef0868dca166f8ed166a0 Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Marek=20Beh=C3=BAn?= <kabel@kernel.org>
|
||||
Date: Mon, 1 Jul 2024 13:30:08 +0200
|
||||
Subject: [PATCH 06/11] platform: cznic: turris-omnia-mcu: Add support for MCU
|
||||
provided TRNG
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
Add support for true random number generator provided by the MCU.
|
||||
New Omnia boards come without the Atmel SHA204-A chip. Instead the
|
||||
crypto functionality is provided by new microcontroller, which has
|
||||
a TRNG peripheral.
|
||||
|
||||
Signed-off-by: Marek Behún <kabel@kernel.org>
|
||||
Acked-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
|
||||
Link: https://lore.kernel.org/r/20240701113010.16447-7-kabel@kernel.org
|
||||
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
|
||||
---
|
||||
drivers/platform/cznic/Kconfig | 2 +
|
||||
drivers/platform/cznic/Makefile | 1 +
|
||||
.../platform/cznic/turris-omnia-mcu-base.c | 6 +-
|
||||
.../platform/cznic/turris-omnia-mcu-gpio.c | 2 +-
|
||||
.../platform/cznic/turris-omnia-mcu-trng.c | 105 ++++++++++++++++++
|
||||
drivers/platform/cznic/turris-omnia-mcu.h | 8 ++
|
||||
6 files changed, 122 insertions(+), 2 deletions(-)
|
||||
create mode 100644 drivers/platform/cznic/turris-omnia-mcu-trng.c
|
||||
|
||||
--- a/drivers/platform/cznic/Kconfig
|
||||
+++ b/drivers/platform/cznic/Kconfig
|
||||
@@ -18,6 +18,7 @@ config TURRIS_OMNIA_MCU
|
||||
depends on I2C
|
||||
select GPIOLIB
|
||||
select GPIOLIB_IRQCHIP
|
||||
+ select HW_RANDOM
|
||||
select RTC_CLASS
|
||||
select WATCHDOG_CORE
|
||||
help
|
||||
@@ -27,6 +28,7 @@ config TURRIS_OMNIA_MCU
|
||||
- board poweroff into true low power mode (with voltage regulators
|
||||
disabled) and the ability to configure wake up from this mode (via
|
||||
rtcwake)
|
||||
+ - true random number generator (if available on the MCU)
|
||||
- MCU watchdog
|
||||
- GPIO pins
|
||||
- to get front button press events (the front button can be
|
||||
--- a/drivers/platform/cznic/Makefile
|
||||
+++ b/drivers/platform/cznic/Makefile
|
||||
@@ -4,4 +4,5 @@ obj-$(CONFIG_TURRIS_OMNIA_MCU) += turris
|
||||
turris-omnia-mcu-y := turris-omnia-mcu-base.o
|
||||
turris-omnia-mcu-y += turris-omnia-mcu-gpio.o
|
||||
turris-omnia-mcu-y += turris-omnia-mcu-sys-off-wakeup.o
|
||||
+turris-omnia-mcu-y += turris-omnia-mcu-trng.o
|
||||
turris-omnia-mcu-y += turris-omnia-mcu-watchdog.o
|
||||
--- a/drivers/platform/cznic/turris-omnia-mcu-base.c
|
||||
+++ b/drivers/platform/cznic/turris-omnia-mcu-base.c
|
||||
@@ -380,7 +380,11 @@ static int omnia_mcu_probe(struct i2c_cl
|
||||
if (err)
|
||||
return err;
|
||||
|
||||
- return omnia_mcu_register_gpiochip(mcu);
|
||||
+ err = omnia_mcu_register_gpiochip(mcu);
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ return omnia_mcu_register_trng(mcu);
|
||||
}
|
||||
|
||||
static const struct of_device_id of_omnia_mcu_match[] = {
|
||||
--- a/drivers/platform/cznic/turris-omnia-mcu-gpio.c
|
||||
+++ b/drivers/platform/cznic/turris-omnia-mcu-gpio.c
|
||||
@@ -194,7 +194,7 @@ static const struct omnia_gpio omnia_gpi
|
||||
};
|
||||
|
||||
/* mapping from interrupts to indexes of GPIOs in the omnia_gpios array */
|
||||
-static const u8 omnia_int_to_gpio_idx[32] = {
|
||||
+const u8 omnia_int_to_gpio_idx[32] = {
|
||||
[__bf_shf(OMNIA_INT_CARD_DET)] = 4,
|
||||
[__bf_shf(OMNIA_INT_MSATA_IND)] = 5,
|
||||
[__bf_shf(OMNIA_INT_USB30_OVC)] = 6,
|
||||
--- /dev/null
|
||||
+++ b/drivers/platform/cznic/turris-omnia-mcu-trng.c
|
||||
@@ -0,0 +1,105 @@
|
||||
+// SPDX-License-Identifier: GPL-2.0
|
||||
+/*
|
||||
+ * CZ.NIC's Turris Omnia MCU TRNG driver
|
||||
+ *
|
||||
+ * 2024 by Marek Behún <kabel@kernel.org>
|
||||
+ */
|
||||
+
|
||||
+#include <linux/bitfield.h>
|
||||
+#include <linux/completion.h>
|
||||
+#include <linux/container_of.h>
|
||||
+#include <linux/errno.h>
|
||||
+#include <linux/gpio/consumer.h>
|
||||
+#include <linux/gpio/driver.h>
|
||||
+#include <linux/hw_random.h>
|
||||
+#include <linux/i2c.h>
|
||||
+#include <linux/interrupt.h>
|
||||
+#include <linux/minmax.h>
|
||||
+#include <linux/string.h>
|
||||
+#include <linux/types.h>
|
||||
+
|
||||
+#include "../../gpio/gpiolib.h"
|
||||
+
|
||||
+#include <linux/turris-omnia-mcu-interface.h>
|
||||
+#include "turris-omnia-mcu.h"
|
||||
+
|
||||
+#define OMNIA_CMD_TRNG_MAX_ENTROPY_LEN 64
|
||||
+
|
||||
+static irqreturn_t omnia_trng_irq_handler(int irq, void *dev_id)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = dev_id;
|
||||
+
|
||||
+ complete(&mcu->trng_entropy_ready);
|
||||
+
|
||||
+ return IRQ_HANDLED;
|
||||
+}
|
||||
+
|
||||
+static int omnia_trng_read(struct hwrng *rng, void *data, size_t max, bool wait)
|
||||
+{
|
||||
+ struct omnia_mcu *mcu = container_of(rng, struct omnia_mcu, trng);
|
||||
+ u8 reply[1 + OMNIA_CMD_TRNG_MAX_ENTROPY_LEN];
|
||||
+ int err, bytes;
|
||||
+
|
||||
+ if (!wait && !completion_done(&mcu->trng_entropy_ready))
|
||||
+ return 0;
|
||||
+
|
||||
+ do {
|
||||
+ if (wait_for_completion_interruptible(&mcu->trng_entropy_ready))
|
||||
+ return -ERESTARTSYS;
|
||||
+
|
||||
+ err = omnia_cmd_read(mcu->client,
|
||||
+ OMNIA_CMD_TRNG_COLLECT_ENTROPY,
|
||||
+ reply, sizeof(reply));
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ bytes = min3(reply[0], max, OMNIA_CMD_TRNG_MAX_ENTROPY_LEN);
|
||||
+ } while (wait && !bytes);
|
||||
+
|
||||
+ memcpy(data, &reply[1], bytes);
|
||||
+
|
||||
+ return bytes;
|
||||
+}
|
||||
+
|
||||
+int omnia_mcu_register_trng(struct omnia_mcu *mcu)
|
||||
+{
|
||||
+ struct device *dev = &mcu->client->dev;
|
||||
+ u8 irq_idx, dummy;
|
||||
+ int irq, err;
|
||||
+
|
||||
+ if (!(mcu->features & OMNIA_FEAT_TRNG))
|
||||
+ return 0;
|
||||
+
|
||||
+ irq_idx = omnia_int_to_gpio_idx[__bf_shf(OMNIA_INT_TRNG)];
|
||||
+ irq = gpiod_to_irq(gpiochip_get_desc(&mcu->gc, irq_idx));
|
||||
+ if (!irq)
|
||||
+ return dev_err_probe(dev, -ENXIO, "Cannot get TRNG IRQ\n");
|
||||
+
|
||||
+ /*
|
||||
+ * If someone else cleared the TRNG interrupt but did not read the
|
||||
+ * entropy, a new interrupt won't be generated, and entropy collection
|
||||
+ * will be stuck. Ensure an interrupt will be generated by executing
|
||||
+ * the collect entropy command (and discarding the result).
|
||||
+ */
|
||||
+ err = omnia_cmd_read(mcu->client, OMNIA_CMD_TRNG_COLLECT_ENTROPY,
|
||||
+ &dummy, 1);
|
||||
+ if (err)
|
||||
+ return err;
|
||||
+
|
||||
+ init_completion(&mcu->trng_entropy_ready);
|
||||
+
|
||||
+ err = devm_request_threaded_irq(dev, irq, NULL, omnia_trng_irq_handler,
|
||||
+ IRQF_ONESHOT, "turris-omnia-mcu-trng",
|
||||
+ mcu);
|
||||
+ if (err)
|
||||
+ return dev_err_probe(dev, err, "Cannot request TRNG IRQ\n");
|
||||
+
|
||||
+ mcu->trng.name = "turris-omnia-mcu-trng";
|
||||
+ mcu->trng.read = omnia_trng_read;
|
||||
+
|
||||
+ err = devm_hwrng_register(dev, &mcu->trng);
|
||||
+ if (err)
|
||||
+ return dev_err_probe(dev, err, "Cannot register TRNG\n");
|
||||
+
|
||||
+ return 0;
|
||||
+}
|
||||
--- a/drivers/platform/cznic/turris-omnia-mcu.h
|
||||
+++ b/drivers/platform/cznic/turris-omnia-mcu.h
|
||||
@@ -9,7 +9,9 @@
|
||||
#define __TURRIS_OMNIA_MCU_H
|
||||
|
||||
#include <linux/bitops.h>
|
||||
+#include <linux/completion.h>
|
||||
#include <linux/gpio/driver.h>
|
||||
+#include <linux/hw_random.h>
|
||||
#include <linux/if_ether.h>
|
||||
#include <linux/mutex.h>
|
||||
#include <linux/types.h>
|
||||
@@ -47,6 +49,10 @@ struct omnia_mcu {
|
||||
|
||||
/* MCU watchdog */
|
||||
struct watchdog_device wdt;
|
||||
+
|
||||
+ /* true random number generator */
|
||||
+ struct hwrng trng;
|
||||
+ struct completion trng_entropy_ready;
|
||||
};
|
||||
|
||||
int omnia_cmd_write_read(const struct i2c_client *client,
|
||||
@@ -176,11 +182,13 @@ static inline int omnia_cmd_read_u8(cons
|
||||
return omnia_cmd_read(client, cmd, reply, sizeof(*reply));
|
||||
}
|
||||
|
||||
+extern const u8 omnia_int_to_gpio_idx[32];
|
||||
extern const struct attribute_group omnia_mcu_gpio_group;
|
||||
extern const struct attribute_group omnia_mcu_poweroff_group;
|
||||
|
||||
int omnia_mcu_register_gpiochip(struct omnia_mcu *mcu);
|
||||
int omnia_mcu_register_sys_off_and_wakeup(struct omnia_mcu *mcu);
|
||||
+int omnia_mcu_register_trng(struct omnia_mcu *mcu);
|
||||
int omnia_mcu_register_watchdog(struct omnia_mcu *mcu);
|
||||
|
||||
#endif /* __TURRIS_OMNIA_MCU_H */
|
|
@ -1,69 +0,0 @@
|
|||
From 4f11095a4ae00b2fe4cebb21e36ee37cc62f5e1a Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Marek=20Beh=C3=BAn?= <kabel@kernel.org>
|
||||
Date: Mon, 1 Jul 2024 13:30:09 +0200
|
||||
Subject: [PATCH 07/11] ARM: dts: turris-omnia: Add MCU system-controller node
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
Turris Omnia's MCU provides various features that can be configured over
|
||||
I2C at address 0x2a. Add device-tree node.
|
||||
|
||||
This does not carry a Fixes tag - we do not want this to get backported
|
||||
to stable kernels for the following reason: U-Boot since v2022.10
|
||||
inserts a phy-reset-gpio property into the WAN ethernet node pointing to
|
||||
the MCU node if it finds the MCU node with a cznic,turris-omnia-mcu
|
||||
compatible. Thus if this change got backported to a stable kernel, the
|
||||
WAN interface driver would defer probe indefinitely (since it would wait
|
||||
for the turris-omnia-mcu driver which would not be present).
|
||||
|
||||
Signed-off-by: Marek Behún <kabel@kernel.org>
|
||||
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
|
||||
Reviewed-by: Andy Shevchenko <andy@kernel.org>
|
||||
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
|
||||
Acked-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
|
||||
Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
|
||||
Link: https://lore.kernel.org/r/20240701113010.16447-8-kabel@kernel.org
|
||||
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
|
||||
---
|
||||
.../dts/marvell/armada-385-turris-omnia.dts | 22 ++++++++++++++++++-
|
||||
1 file changed, 21 insertions(+), 1 deletion(-)
|
||||
|
||||
--- a/arch/arm/boot/dts/marvell/armada-385-turris-omnia.dts
|
||||
+++ b/arch/arm/boot/dts/marvell/armada-385-turris-omnia.dts
|
||||
@@ -218,7 +218,22 @@
|
||||
#size-cells = <0>;
|
||||
reg = <0>;
|
||||
|
||||
- /* STM32F0 command interface at address 0x2a */
|
||||
+ mcu: system-controller@2a {
|
||||
+ compatible = "cznic,turris-omnia-mcu";
|
||||
+ reg = <0x2a>;
|
||||
+
|
||||
+ pinctrl-names = "default";
|
||||
+ pinctrl-0 = <&mcu_pins>;
|
||||
+
|
||||
+ interrupt-parent = <&gpio1>;
|
||||
+ interrupts = <11 IRQ_TYPE_NONE>;
|
||||
+
|
||||
+ gpio-controller;
|
||||
+ #gpio-cells = <3>;
|
||||
+
|
||||
+ interrupt-controller;
|
||||
+ #interrupt-cells = <2>;
|
||||
+ };
|
||||
|
||||
led-controller@2b {
|
||||
compatible = "cznic,turris-omnia-leds";
|
||||
@@ -503,6 +518,11 @@
|
||||
};
|
||||
|
||||
&pinctrl {
|
||||
+ mcu_pins: mcu-pins {
|
||||
+ marvell,pins = "mpp43";
|
||||
+ marvell,function = "gpio";
|
||||
+ };
|
||||
+
|
||||
pcawan_pins: pcawan-pins {
|
||||
marvell,pins = "mpp46";
|
||||
marvell,function = "gpio";
|
|
@ -1,46 +0,0 @@
|
|||
From c3eeabe0b8d22d7c869278cc0cb35b83512fbed5 Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Marek=20Beh=C3=BAn?= <kabel@kernel.org>
|
||||
Date: Mon, 1 Jul 2024 13:30:10 +0200
|
||||
Subject: [PATCH 08/11] ARM: dts: turris-omnia: Add GPIO key node for front
|
||||
button
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
Now that we have the MCU device-tree node, which acts as a GPIO
|
||||
controller, add GPIO key node for the front button.
|
||||
|
||||
Signed-off-by: Marek Behún <kabel@kernel.org>
|
||||
Reviewed-by: Andrew Lunn <andrew@lunn.ch>
|
||||
Reviewed-by: Andy Shevchenko <andy@kernel.org>
|
||||
Reviewed-by: Conor Dooley <conor.dooley@microchip.com>
|
||||
Acked-by: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
|
||||
Acked-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
|
||||
Link: https://lore.kernel.org/r/20240701113010.16447-9-kabel@kernel.org
|
||||
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
|
||||
---
|
||||
.../boot/dts/marvell/armada-385-turris-omnia.dts | 13 +++++++++++++
|
||||
1 file changed, 13 insertions(+)
|
||||
|
||||
--- a/arch/arm/boot/dts/marvell/armada-385-turris-omnia.dts
|
||||
+++ b/arch/arm/boot/dts/marvell/armada-385-turris-omnia.dts
|
||||
@@ -112,6 +112,19 @@
|
||||
status = "disabled";
|
||||
};
|
||||
|
||||
+ gpio-keys {
|
||||
+ compatible = "gpio-keys";
|
||||
+
|
||||
+ front-button {
|
||||
+ label = "Front Button";
|
||||
+ linux,code = <KEY_VENDOR>;
|
||||
+ linux,can-disable;
|
||||
+ gpios = <&mcu 0 12 GPIO_ACTIVE_HIGH>;
|
||||
+ /* debouncing is done by the microcontroller */
|
||||
+ debounce-interval = <0>;
|
||||
+ };
|
||||
+ };
|
||||
+
|
||||
sound {
|
||||
compatible = "simple-audio-card";
|
||||
simple-audio-card,name = "SPDIF";
|
|
@ -1,32 +0,0 @@
|
|||
From 08838657bbc35494276c7ba4ef53f30a9816f8c9 Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Marek=20Beh=C3=BAn?= <kabel@kernel.org>
|
||||
Date: Mon, 8 Jul 2024 13:40:01 +0200
|
||||
Subject: [PATCH 09/11] platform: cznic: turris-omnia-mcu: Depend on OF
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
Add depend on OF, otherwise the compilation fails with
|
||||
error: no member named 'of_gpio_n_cells' in 'struct gpio_chip'
|
||||
error: no member named 'of_xlate' in 'struct gpio_chip'
|
||||
|
||||
Fixes: dfa556e45ae9 ("platform: cznic: turris-omnia-mcu: Add support for MCU connected GPIOs")
|
||||
Reported-by: kernel test robot <lkp@intel.com>
|
||||
Closes: https://lore.kernel.org/oe-kbuild-all/202407031646.trNSwajF-lkp@intel.com/
|
||||
Signed-off-by: Marek Behún <kabel@kernel.org>
|
||||
Link: https://lore.kernel.org/r/20240708114002.4285-2-kabel@kernel.org
|
||||
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
|
||||
---
|
||||
drivers/platform/cznic/Kconfig | 1 +
|
||||
1 file changed, 1 insertion(+)
|
||||
|
||||
--- a/drivers/platform/cznic/Kconfig
|
||||
+++ b/drivers/platform/cznic/Kconfig
|
||||
@@ -16,6 +16,7 @@ config TURRIS_OMNIA_MCU
|
||||
tristate "Turris Omnia MCU driver"
|
||||
depends on MACH_ARMADA_38X || COMPILE_TEST
|
||||
depends on I2C
|
||||
+ depends on OF
|
||||
select GPIOLIB
|
||||
select GPIOLIB_IRQCHIP
|
||||
select HW_RANDOM
|
|
@ -1,32 +0,0 @@
|
|||
From 5e425e6eca155c162da58d4e58e896ed4109c7fd Mon Sep 17 00:00:00 2001
|
||||
From: =?UTF-8?q?Marek=20Beh=C3=BAn?= <kabel@kernel.org>
|
||||
Date: Mon, 8 Jul 2024 13:40:02 +0200
|
||||
Subject: [PATCH 10/11] platform: cznic: turris-omnia-mcu: Depend on WATCHDOG
|
||||
MIME-Version: 1.0
|
||||
Content-Type: text/plain; charset=UTF-8
|
||||
Content-Transfer-Encoding: 8bit
|
||||
|
||||
Add depend on WATCHDOG, otherwise modpost fails with
|
||||
ERROR: modpost: "watchdog_init_timeout" [drivers/platform/cznic/turris-omnia-mcu.ko] undefined!
|
||||
ERROR: modpost: "devm_watchdog_register_device" [drivers/platform/cznic/turris-omnia-mcu.ko] undefined!
|
||||
|
||||
Fixes: ab89fb5fb92c ("platform: cznic: turris-omnia-mcu: Add support for MCU watchdog")
|
||||
Reported-by: kernel test robot <lkp@intel.com>
|
||||
Closes: https://lore.kernel.org/oe-kbuild-all/202407040711.g19y3cWq-lkp@intel.com/
|
||||
Signed-off-by: Marek Behún <kabel@kernel.org>
|
||||
Link: https://lore.kernel.org/r/20240708114002.4285-3-kabel@kernel.org
|
||||
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
|
||||
---
|
||||
drivers/platform/cznic/Kconfig | 1 +
|
||||
1 file changed, 1 insertion(+)
|
||||
|
||||
--- a/drivers/platform/cznic/Kconfig
|
||||
+++ b/drivers/platform/cznic/Kconfig
|
||||
@@ -17,6 +17,7 @@ config TURRIS_OMNIA_MCU
|
||||
depends on MACH_ARMADA_38X || COMPILE_TEST
|
||||
depends on I2C
|
||||
depends on OF
|
||||
+ depends on WATCHDOG
|
||||
select GPIOLIB
|
||||
select GPIOLIB_IRQCHIP
|
||||
select HW_RANDOM
|
|
@ -1,45 +0,0 @@
|
|||
From 24c68c2525de5fcd0f3b16b2ad1028fb13b53393 Mon Sep 17 00:00:00 2001
|
||||
From: Arnd Bergmann <arnd@arndb.de>
|
||||
Date: Mon, 15 Jul 2024 08:02:30 +0200
|
||||
Subject: [PATCH 11/11] platform: cznic: turris-omnia-mcu: fix Kconfig
|
||||
dependencies
|
||||
|
||||
The newly added driver causes a Kconfig warning:
|
||||
|
||||
WARNING: unmet direct dependencies detected for RTC_CLASS
|
||||
Depends on [n]: !S390 [=y]
|
||||
Selected by [m]:
|
||||
- TURRIS_OMNIA_MCU [=m] && CZNIC_PLATFORMS [=y] && (MACH_ARMADA_38X || COMPILE_TEST [=y]) && I2C [=m] && OF [=y] && WATCHDOG [=y]
|
||||
|
||||
The problem here is that it selects entire subsystems, which normal
|
||||
device drivers should not do. Changes all of these to 'depends on'
|
||||
instead.
|
||||
|
||||
Fixes: dfa556e45ae9e ("platform: cznic: turris-omnia-mcu: Add support for MCU connected GPIOs")
|
||||
Fixes: 90e700fd12b61 ("platform: cznic: turris-omnia-mcu: Add support for poweroff and wakeup")
|
||||
Fixes: ab89fb5fb92c7 ("platform: cznic: turris-omnia-mcu: Add support for MCU watchdog")
|
||||
Fixes: 41bb142a40289 ("platform: cznic: turris-omnia-mcu: Add support for MCU provided TRNG")
|
||||
Reported-by: Nathan Chancellor <nathan@kernel.org>
|
||||
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
|
||||
---
|
||||
drivers/platform/cznic/Kconfig | 8 ++++----
|
||||
1 file changed, 4 insertions(+), 4 deletions(-)
|
||||
|
||||
--- a/drivers/platform/cznic/Kconfig
|
||||
+++ b/drivers/platform/cznic/Kconfig
|
||||
@@ -18,11 +18,11 @@ config TURRIS_OMNIA_MCU
|
||||
depends on I2C
|
||||
depends on OF
|
||||
depends on WATCHDOG
|
||||
- select GPIOLIB
|
||||
+ depends on GPIOLIB
|
||||
+ depends on HW_RANDOM
|
||||
+ depends on RTC_CLASS
|
||||
+ depends on WATCHDOG_CORE
|
||||
select GPIOLIB_IRQCHIP
|
||||
- select HW_RANDOM
|
||||
- select RTC_CLASS
|
||||
- select WATCHDOG_CORE
|
||||
help
|
||||
Say Y here to add support for the features implemented by the
|
||||
microcontroller on the CZ.NIC's Turris Omnia SOHO router.
|
|
@ -169,7 +169,7 @@ Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
|
|||
err_str = "no suitable method provided";
|
||||
--- a/include/linux/i2c.h
|
||||
+++ b/include/linux/i2c.h
|
||||
@@ -622,6 +622,9 @@ struct i2c_timings {
|
||||
@@ -635,6 +635,9 @@ struct i2c_timings {
|
||||
* for generic GPIO recovery.
|
||||
* @get_bus_free: Returns the bus free state as seen from the IP core in case it
|
||||
* has a more complex internal logic than just reading SDA. Optional.
|
||||
|
@ -179,7 +179,7 @@ Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
|
|||
* @prepare_recovery: This will be called before starting recovery. Platform may
|
||||
* configure padmux here for SDA/SCL line or something else they want.
|
||||
* @unprepare_recovery: This will be called after completing recovery. Platform
|
||||
@@ -646,6 +649,7 @@ struct i2c_bus_recovery_info {
|
||||
@@ -659,6 +662,7 @@ struct i2c_bus_recovery_info {
|
||||
void (*set_sda)(struct i2c_adapter *adap, int val);
|
||||
int (*get_bus_free)(struct i2c_adapter *adap);
|
||||
|
||||
|
|
|
@ -201,7 +201,7 @@ Signed-off-by: Imre Kaloz <kaloz@openwrt.org>
|
|||
|
||||
--- a/drivers/i2c/busses/i2c-pxa.c
|
||||
+++ b/drivers/i2c/busses/i2c-pxa.c
|
||||
@@ -1331,6 +1331,12 @@ static void i2c_pxa_unprepare_recovery(s
|
||||
@@ -1330,6 +1330,12 @@ static void i2c_pxa_unprepare_recovery(s
|
||||
i2c_pxa_enable(i2c);
|
||||
}
|
||||
|
||||
|
@ -214,7 +214,7 @@ Signed-off-by: Imre Kaloz <kaloz@openwrt.org>
|
|||
static int i2c_pxa_init_recovery(struct pxa_i2c *i2c)
|
||||
{
|
||||
struct i2c_bus_recovery_info *bri = &i2c->recovery;
|
||||
@@ -1399,6 +1405,7 @@ static int i2c_pxa_init_recovery(struct
|
||||
@@ -1398,6 +1404,7 @@ static int i2c_pxa_init_recovery(struct
|
||||
return 0;
|
||||
}
|
||||
|
||||
|
|
|
@ -226,7 +226,7 @@ the controller from the bad state described in the previous patch.
|
|||
|
||||
--- a/drivers/i2c/busses/i2c-pxa.c
|
||||
+++ b/drivers/i2c/busses/i2c-pxa.c
|
||||
@@ -70,6 +70,7 @@
|
||||
@@ -71,6 +71,7 @@
|
||||
#define ISR_GCAD (1 << 8) /* general call address detected */
|
||||
#define ISR_SAD (1 << 9) /* slave address detected */
|
||||
#define ISR_BED (1 << 10) /* bus error no ACK/NAK */
|
||||
|
@ -234,7 +234,7 @@ the controller from the bad state described in the previous patch.
|
|||
|
||||
#define ILCR_SLV_SHIFT 0
|
||||
#define ILCR_SLV_MASK (0x1FF << ILCR_SLV_SHIFT)
|
||||
@@ -262,6 +263,7 @@ struct pxa_i2c {
|
||||
@@ -263,6 +264,7 @@ struct pxa_i2c {
|
||||
bool highmode_enter;
|
||||
u32 fm_mask;
|
||||
u32 hs_mask;
|
||||
|
@ -242,7 +242,7 @@ the controller from the bad state described in the previous patch.
|
|||
|
||||
struct i2c_bus_recovery_info recovery;
|
||||
struct pinctrl *pinctrl;
|
||||
@@ -428,7 +430,7 @@ static int i2c_pxa_wait_bus_not_busy(str
|
||||
@@ -429,7 +431,7 @@ static int i2c_pxa_wait_bus_not_busy(str
|
||||
|
||||
while (1) {
|
||||
isr = readl(_ISR(i2c));
|
||||
|
@ -251,7 +251,7 @@ the controller from the bad state described in the previous patch.
|
|||
return 0;
|
||||
|
||||
if (isr & ISR_SAD)
|
||||
@@ -465,7 +467,7 @@ static int i2c_pxa_wait_master(struct px
|
||||
@@ -466,7 +468,7 @@ static int i2c_pxa_wait_master(struct px
|
||||
* quick check of the i2c lines themselves to ensure they've
|
||||
* gone high...
|
||||
*/
|
||||
|
@ -260,7 +260,7 @@ the controller from the bad state described in the previous patch.
|
|||
readl(_IBMR(i2c)) == (IBMR_SCLS | IBMR_SDAS)) {
|
||||
if (i2c_debug > 0)
|
||||
dev_dbg(&i2c->adap.dev, "%s: done\n", __func__);
|
||||
@@ -486,7 +488,7 @@ static int i2c_pxa_set_master(struct pxa
|
||||
@@ -487,7 +489,7 @@ static int i2c_pxa_set_master(struct pxa
|
||||
if (i2c_debug)
|
||||
dev_dbg(&i2c->adap.dev, "setting to bus master\n");
|
||||
|
||||
|
@ -269,7 +269,7 @@ the controller from the bad state described in the previous patch.
|
|||
dev_dbg(&i2c->adap.dev, "%s: unit is busy\n", __func__);
|
||||
if (!i2c_pxa_wait_master(i2c)) {
|
||||
dev_dbg(&i2c->adap.dev, "%s: error: unit busy\n", __func__);
|
||||
@@ -512,7 +514,7 @@ static int i2c_pxa_wait_slave(struct pxa
|
||||
@@ -513,7 +515,7 @@ static int i2c_pxa_wait_slave(struct pxa
|
||||
dev_dbg(&i2c->adap.dev, "%s: %ld: ISR=%08x, ICR=%08x, IBMR=%02x\n",
|
||||
__func__, (long)jiffies, readl(_ISR(i2c)), readl(_ICR(i2c)), readl(_IBMR(i2c)));
|
||||
|
||||
|
@ -278,7 +278,7 @@ the controller from the bad state described in the previous patch.
|
|||
(readl(_ISR(i2c)) & ISR_SAD) != 0 ||
|
||||
(readl(_ICR(i2c)) & ICR_SCLE) == 0) {
|
||||
if (i2c_debug > 1)
|
||||
@@ -1170,7 +1172,7 @@ static int i2c_pxa_pio_set_master(struct
|
||||
@@ -1171,7 +1173,7 @@ static int i2c_pxa_pio_set_master(struct
|
||||
/*
|
||||
* Wait for the bus to become free.
|
||||
*/
|
||||
|
@ -287,7 +287,7 @@ the controller from the bad state described in the previous patch.
|
|||
udelay(1000);
|
||||
|
||||
if (timeout < 0) {
|
||||
@@ -1317,7 +1319,7 @@ static void i2c_pxa_unprepare_recovery(s
|
||||
@@ -1316,7 +1318,7 @@ static void i2c_pxa_unprepare_recovery(s
|
||||
* handing control of the bus back to avoid the bus changing state.
|
||||
*/
|
||||
isr = readl(_ISR(i2c));
|
||||
|
@ -296,7 +296,7 @@ the controller from the bad state described in the previous patch.
|
|||
dev_dbg(&i2c->adap.dev,
|
||||
"recovery: resetting controller, ISR=0x%08x\n", isr);
|
||||
i2c_pxa_do_reset(i2c);
|
||||
@@ -1481,6 +1483,10 @@ static int i2c_pxa_probe(struct platform
|
||||
@@ -1480,6 +1482,10 @@ static int i2c_pxa_probe(struct platform
|
||||
i2c->fm_mask = pxa_reg_layout[i2c_type].fm;
|
||||
i2c->hs_mask = pxa_reg_layout[i2c_type].hs;
|
||||
|
||||
|
|
|
@ -16,8 +16,7 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
.../hwmon/iei,wt61p803-puzzle-hwmon.yaml | 53 ++++++++++++
|
||||
.../leds/iei,wt61p803-puzzle-leds.yaml | 39 +++++++++
|
||||
.../bindings/mfd/iei,wt61p803-puzzle.yaml | 82 +++++++++++++++++++
|
||||
.../devicetree/bindings/vendor-prefixes.yaml | 2 +
|
||||
4 files changed, 176 insertions(+)
|
||||
3 files changed, 174 insertions(+)
|
||||
create mode 100644 Documentation/devicetree/bindings/hwmon/iei,wt61p803-puzzle-hwmon.yaml
|
||||
create mode 100644 Documentation/devicetree/bindings/leds/iei,wt61p803-puzzle-leds.yaml
|
||||
create mode 100644 Documentation/devicetree/bindings/mfd/iei,wt61p803-puzzle.yaml
|
||||
|
@ -205,14 +204,3 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
+ };
|
||||
+ };
|
||||
+ };
|
||||
--- a/Documentation/devicetree/bindings/vendor-prefixes.yaml
|
||||
+++ b/Documentation/devicetree/bindings/vendor-prefixes.yaml
|
||||
@@ -611,6 +611,8 @@ patternProperties:
|
||||
description: IC Plus Corp.
|
||||
"^idt,.*":
|
||||
description: Integrated Device Technologies, Inc.
|
||||
+ "^iei,.*":
|
||||
+ description: IEI Integration Corp.
|
||||
"^ifi,.*":
|
||||
description: Ingenieurburo Fur Ic-Technologie (I/F/I)
|
||||
"^ilitek,.*":
|
||||
|
|
|
@ -20,13 +20,13 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
drivers/mfd/Makefile | 1 +
|
||||
drivers/mfd/iei-wt61p803-puzzle.c | 908 ++++++++++++++++++++++++
|
||||
include/linux/mfd/iei-wt61p803-puzzle.h | 66 ++
|
||||
4 files changed, 984 insertions(+)
|
||||
4 files changed, 981 insertions(+)
|
||||
create mode 100644 drivers/mfd/iei-wt61p803-puzzle.c
|
||||
create mode 100644 include/linux/mfd/iei-wt61p803-puzzle.h
|
||||
|
||||
--- a/drivers/mfd/Kconfig
|
||||
+++ b/drivers/mfd/Kconfig
|
||||
@@ -2274,6 +2274,15 @@ config SGI_MFD_IOC3
|
||||
@@ -2343,6 +2343,15 @@ config SGI_MFD_IOC3
|
||||
If you have an SGI Origin, Octane, or a PCI IOC3 card,
|
||||
then say Y. Otherwise say N.
|
||||
|
||||
|
@ -44,7 +44,7 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
select MFD_CORE
|
||||
--- a/drivers/mfd/Makefile
|
||||
+++ b/drivers/mfd/Makefile
|
||||
@@ -244,6 +244,7 @@ obj-$(CONFIG_MFD_RT4831) += rt4831.o
|
||||
@@ -250,6 +250,7 @@ obj-$(CONFIG_MFD_RT4831) += rt4831.o
|
||||
obj-$(CONFIG_MFD_RT5033) += rt5033.o
|
||||
obj-$(CONFIG_MFD_RT5120) += rt5120.o
|
||||
obj-$(CONFIG_MFD_SKY81452) += sky81452.o
|
||||
|
@ -54,7 +54,7 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
obj-$(CONFIG_INTEL_SOC_PMIC_BXTWC) += intel_soc_pmic_bxtwc.o
|
||||
--- /dev/null
|
||||
+++ b/drivers/mfd/iei-wt61p803-puzzle.c
|
||||
@@ -0,0 +1,912 @@
|
||||
@@ -0,0 +1,909 @@
|
||||
+// SPDX-License-Identifier: GPL-2.0-only
|
||||
+/* IEI WT61P803 PUZZLE MCU Driver
|
||||
+ * System management microcontroller for fan control, temperature sensor reading,
|
||||
|
@ -79,7 +79,7 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
+#include <linux/serdev.h>
|
||||
+#include <linux/slab.h>
|
||||
+#include <linux/sysfs.h>
|
||||
+#include <asm/unaligned.h>
|
||||
+#include <linux/unaligned.h>
|
||||
+
|
||||
+/* start, payload and XOR checksum at end */
|
||||
+#define IEI_WT61P803_PUZZLE_MAX_COMMAND_LENGTH (1 + 20 + 1)
|
||||
|
@ -190,8 +190,8 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
+ return checksum;
|
||||
+}
|
||||
+
|
||||
+static int iei_wt61p803_puzzle_process_resp(struct iei_wt61p803_puzzle *mcu,
|
||||
+ const unsigned char *raw_resp_data, size_t size)
|
||||
+static size_t iei_wt61p803_puzzle_process_resp(struct iei_wt61p803_puzzle *mcu,
|
||||
+ const u8 *raw_resp_data, size_t size)
|
||||
+{
|
||||
+ unsigned char checksum;
|
||||
+
|
||||
|
@ -227,11 +227,11 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
+ return size;
|
||||
+}
|
||||
+
|
||||
+static int iei_wt61p803_puzzle_recv_buf(struct serdev_device *serdev,
|
||||
+ const unsigned char *data, size_t size)
|
||||
+static size_t iei_wt61p803_puzzle_recv_buf(struct serdev_device *serdev,
|
||||
+ const u8 *data, size_t size)
|
||||
+{
|
||||
+ struct iei_wt61p803_puzzle *mcu = serdev_device_get_drvdata(serdev);
|
||||
+ int ret;
|
||||
+ size_t ret;
|
||||
+
|
||||
+ ret = iei_wt61p803_puzzle_process_resp(mcu, data, size);
|
||||
+ /* Return the number of processed bytes if function returns error,
|
||||
|
@ -514,8 +514,7 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
+ goto err;
|
||||
+ }
|
||||
+
|
||||
+ sprintf(mcu->version.serial_number, "%.*s",
|
||||
+ IEI_WT61P803_PUZZLE_SN_LENGTH, resp_buf + 4);
|
||||
+ memcpy(mcu->version.serial_number, resp_buf + 4, IEI_WT61P803_PUZZLE_SN_LENGTH);
|
||||
+err:
|
||||
+ mutex_unlock(&mcu->lock);
|
||||
+ return ret;
|
||||
|
@ -561,8 +560,7 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
+ }
|
||||
+ }
|
||||
+
|
||||
+ sprintf(mcu->version.serial_number, "%.*s",
|
||||
+ IEI_WT61P803_PUZZLE_SN_LENGTH, serial_number);
|
||||
+ memcpy(mcu->version.serial_number, serial_number, IEI_WT61P803_PUZZLE_SN_LENGTH);
|
||||
+err:
|
||||
+ mutex_unlock(&mcu->lock);
|
||||
+ return ret;
|
||||
|
@ -594,8 +592,7 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
+ goto err;
|
||||
+ }
|
||||
+
|
||||
+ sprintf(mcu->version.mac_address[index], "%.*s",
|
||||
+ IEI_WT61P803_PUZZLE_MAC_LENGTH, resp_buf + 4);
|
||||
+ memcpy(mcu->version.mac_address[index], resp_buf + 4, IEI_WT61P803_PUZZLE_MAC_LENGTH);
|
||||
+err:
|
||||
+ mutex_unlock(&mcu->lock);
|
||||
+ return ret;
|
||||
|
@ -644,8 +641,8 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
+ goto err;
|
||||
+ }
|
||||
+
|
||||
+ sprintf(mcu->version.mac_address[mac_address_idx], "%.*s",
|
||||
+ IEI_WT61P803_PUZZLE_MAC_LENGTH, mac_address);
|
||||
+ memcpy(mcu->version.mac_address[mac_address_idx], mac_address,
|
||||
+ IEI_WT61P803_PUZZLE_MAC_LENGTH);
|
||||
+err:
|
||||
+ mutex_unlock(&mcu->lock);
|
||||
+ return ret;
|
||||
|
|
|
@ -26,7 +26,7 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
|
||||
--- a/drivers/hwmon/Kconfig
|
||||
+++ b/drivers/hwmon/Kconfig
|
||||
@@ -785,6 +785,14 @@ config SENSORS_IBMPOWERNV
|
||||
@@ -828,6 +828,14 @@ config SENSORS_IBMPOWERNV
|
||||
This driver can also be built as a module. If so, the module
|
||||
will be called ibmpowernv.
|
||||
|
||||
|
@ -43,7 +43,7 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
depends on IIO
|
||||
--- a/drivers/hwmon/Makefile
|
||||
+++ b/drivers/hwmon/Makefile
|
||||
@@ -90,6 +90,7 @@ obj-$(CONFIG_SENSORS_HS3001) += hs3001.o
|
||||
@@ -94,6 +94,7 @@ obj-$(CONFIG_SENSORS_HS3001) += hs3001.o
|
||||
obj-$(CONFIG_SENSORS_ULTRA45) += ultra45_env.o
|
||||
obj-$(CONFIG_SENSORS_I5500) += i5500_temp.o
|
||||
obj-$(CONFIG_SENSORS_I5K_AMB) += i5k_amb.o
|
||||
|
|
|
@ -30,7 +30,7 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
|
||||
--- a/drivers/leds/Kconfig
|
||||
+++ b/drivers/leds/Kconfig
|
||||
@@ -316,6 +316,14 @@ config LEDS_IPAQ_MICRO
|
||||
@@ -354,6 +354,14 @@ config LEDS_IPAQ_MICRO
|
||||
Choose this option if you want to use the notification LED on
|
||||
Compaq/HP iPAQ h3100 and h3600.
|
||||
|
||||
|
@ -47,7 +47,7 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
depends on LEDS_CLASS
|
||||
--- a/drivers/leds/Makefile
|
||||
+++ b/drivers/leds/Makefile
|
||||
@@ -34,6 +34,7 @@ obj-$(CONFIG_LEDS_HP6XX) += leds-hp6xx.
|
||||
@@ -35,6 +35,7 @@ obj-$(CONFIG_LEDS_HP6XX) += leds-hp6xx.
|
||||
obj-$(CONFIG_LEDS_INTEL_SS4200) += leds-ss4200.o
|
||||
obj-$(CONFIG_LEDS_IP30) += leds-ip30.o
|
||||
obj-$(CONFIG_LEDS_IPAQ_MICRO) += leds-ipaq-micro.o
|
||||
|
|
|
@ -64,7 +64,7 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
+================= == =====================================================
|
||||
--- a/Documentation/hwmon/index.rst
|
||||
+++ b/Documentation/hwmon/index.rst
|
||||
@@ -82,6 +82,7 @@ Hardware Monitoring Kernel Drivers
|
||||
@@ -87,6 +87,7 @@ Hardware Monitoring Kernel Drivers
|
||||
ibmaem
|
||||
ibm-cffps
|
||||
ibmpowernv
|
||||
|
|
|
@ -16,8 +16,8 @@ Cc: Robert Marko <robert.marko@sartura.hr>
|
|||
|
||||
--- a/MAINTAINERS
|
||||
+++ b/MAINTAINERS
|
||||
@@ -10143,6 +10143,22 @@ IFCVF VIRTIO DATA PATH ACCELERATOR
|
||||
R: Zhu Lingshan <lingshan.zhu@intel.com>
|
||||
@@ -10965,6 +10965,22 @@ L: virtualization@lists.linux.dev
|
||||
S: Supported
|
||||
F: drivers/vdpa/ifcvf/
|
||||
|
||||
+IEI WT61P803 M801 MFD DRIVER
|
||||
|
|
|
@ -251,9 +251,9 @@
|
|||
#define IEI_WT61P803_PUZZLE_CMD_TEMP_ALL 0x41 /* A */
|
||||
--- a/drivers/mfd/iei-wt61p803-puzzle.c
|
||||
+++ b/drivers/mfd/iei-wt61p803-puzzle.c
|
||||
@@ -176,6 +176,9 @@ static int iei_wt61p803_puzzle_recv_buf(
|
||||
@@ -176,6 +176,9 @@ static size_t iei_wt61p803_puzzle_recv_b
|
||||
struct iei_wt61p803_puzzle *mcu = serdev_device_get_drvdata(serdev);
|
||||
int ret;
|
||||
size_t ret;
|
||||
|
||||
+ print_hex_dump_debug("puzzle-mcu rx: ", DUMP_PREFIX_NONE,
|
||||
+ 16, 1, data, size, false);
|
||||
|
|
Loading…
Reference in a new issue