reset: starfive: Factor out common JH71X0 reset code
authorEmil Renner Berthing <kernel@esmil.dk>
Sat, 9 Jul 2022 21:32:56 +0000 (23:32 +0200)
committerŁukasz Stelmach <l.stelmach@samsung.com>
Tue, 31 Jan 2023 15:43:38 +0000 (16:43 +0100)
The StarFive JH7100 SoC has additional reset controllers for audio and
video, but the registers follow the same structure. On the JH7110 the
reset registers don't get their own memory range, but instead follow the
clock control registers. The registers still follow the same structure
though, so let's factor out the common code to handle all these cases.

Signed-off-by: Emil Renner Berthing <kernel@esmil.dk>
Co-developed-by: Hal Feng <hal.feng@starfivetech.com>
Signed-off-by: Hal Feng <hal.feng@starfivetech.com>
drivers/reset/starfive/Kconfig
drivers/reset/starfive/Makefile
drivers/reset/starfive/reset-starfive-jh7100.c
drivers/reset/starfive/reset-starfive-jh71x0.c [new file with mode: 0644]
drivers/reset/starfive/reset-starfive-jh71x0.h [new file with mode: 0644]

index cddebdba71773b23c7ba495f99d19d3cca61fe97..9d15c4110e401537ad6fdd51ac487a5b51847428 100644 (file)
@@ -1,8 +1,12 @@
 # SPDX-License-Identifier: GPL-2.0-only
 
+config RESET_STARFIVE_JH71X0
+       bool
+
 config RESET_STARFIVE_JH7100
        bool "StarFive JH7100 Reset Driver"
        depends on SOC_STARFIVE || COMPILE_TEST
+       select RESET_STARFIVE_JH71X0
        default SOC_STARFIVE
        help
          This enables the reset controller driver for the StarFive JH7100 SoC.
index 670d049423f540962de24bb33c422d8d65a4acc0..f6aa12466fad9d1a6582a7c711b0aa5b1e085bac 100644 (file)
@@ -1,2 +1,4 @@
 # SPDX-License-Identifier: GPL-2.0
+obj-$(CONFIG_RESET_STARFIVE_JH71X0)            += reset-starfive-jh71x0.o
+
 obj-$(CONFIG_RESET_STARFIVE_JH7100)            += reset-starfive-jh7100.o
index fc44b2fb3e0313af08c0d5406d412b2054caa614..43248e8135fd5d283770ea929704cdc33d434fc9 100644 (file)
@@ -5,14 +5,10 @@
  * Copyright (C) 2021 Emil Renner Berthing <kernel@esmil.dk>
  */
 
-#include <linux/bitmap.h>
-#include <linux/io.h>
-#include <linux/io-64-nonatomic-lo-hi.h>
-#include <linux/iopoll.h>
 #include <linux/mod_devicetable.h>
 #include <linux/platform_device.h>
-#include <linux/reset-controller.h>
-#include <linux/spinlock.h>
+
+#include "reset-starfive-jh71x0.h"
 
 #include <dt-bindings/reset/starfive-jh7100.h>
 
@@ -48,114 +44,19 @@ static const u64 jh7100_reset_asserted[2] = {
        0,
 };
 
-struct jh7100_reset {
-       struct reset_controller_dev rcdev;
-       /* protect registers against concurrent read-modify-write */
-       spinlock_t lock;
-       void __iomem *base;
-};
-
-static inline struct jh7100_reset *
-jh7100_reset_from(struct reset_controller_dev *rcdev)
-{
-       return container_of(rcdev, struct jh7100_reset, rcdev);
-}
-
-static int jh7100_reset_update(struct reset_controller_dev *rcdev,
-                              unsigned long id, bool assert)
-{
-       struct jh7100_reset *data = jh7100_reset_from(rcdev);
-       unsigned long offset = BIT_ULL_WORD(id);
-       u64 mask = BIT_ULL_MASK(id);
-       void __iomem *reg_assert = data->base + JH7100_RESET_ASSERT0 + offset * sizeof(u64);
-       void __iomem *reg_status = data->base + JH7100_RESET_STATUS0 + offset * sizeof(u64);
-       u64 done = jh7100_reset_asserted[offset] & mask;
-       u64 value;
-       unsigned long flags;
-       int ret;
-
-       if (!assert)
-               done ^= mask;
-
-       spin_lock_irqsave(&data->lock, flags);
-
-       value = readq(reg_assert);
-       if (assert)
-               value |= mask;
-       else
-               value &= ~mask;
-       writeq(value, reg_assert);
-
-       /* if the associated clock is gated, deasserting might otherwise hang forever */
-       ret = readq_poll_timeout_atomic(reg_status, value, (value & mask) == done, 0, 1000);
-
-       spin_unlock_irqrestore(&data->lock, flags);
-       return ret;
-}
-
-static int jh7100_reset_assert(struct reset_controller_dev *rcdev,
-                              unsigned long id)
-{
-       return jh7100_reset_update(rcdev, id, true);
-}
-
-static int jh7100_reset_deassert(struct reset_controller_dev *rcdev,
-                                unsigned long id)
-{
-       return jh7100_reset_update(rcdev, id, false);
-}
-
-static int jh7100_reset_reset(struct reset_controller_dev *rcdev,
-                             unsigned long id)
-{
-       int ret;
-
-       ret = jh7100_reset_assert(rcdev, id);
-       if (ret)
-               return ret;
-
-       return jh7100_reset_deassert(rcdev, id);
-}
-
-static int jh7100_reset_status(struct reset_controller_dev *rcdev,
-                              unsigned long id)
-{
-       struct jh7100_reset *data = jh7100_reset_from(rcdev);
-       unsigned long offset = BIT_ULL_WORD(id);
-       u64 mask = BIT_ULL_MASK(id);
-       void __iomem *reg_status = data->base + JH7100_RESET_STATUS0 + offset * sizeof(u64);
-       u64 value = readq(reg_status);
-
-       return !((value ^ jh7100_reset_asserted[offset]) & mask);
-}
-
-static const struct reset_control_ops jh7100_reset_ops = {
-       .assert         = jh7100_reset_assert,
-       .deassert       = jh7100_reset_deassert,
-       .reset          = jh7100_reset_reset,
-       .status         = jh7100_reset_status,
-};
-
 static int __init jh7100_reset_probe(struct platform_device *pdev)
 {
-       struct jh7100_reset *data;
-
-       data = devm_kzalloc(&pdev->dev, sizeof(*data), GFP_KERNEL);
-       if (!data)
-               return -ENOMEM;
-
-       data->base = devm_platform_ioremap_resource(pdev, 0);
-       if (IS_ERR(data->base))
-               return PTR_ERR(data->base);
+       void __iomem *base = devm_platform_ioremap_resource(pdev, 0);
 
-       data->rcdev.ops = &jh7100_reset_ops;
-       data->rcdev.owner = THIS_MODULE;
-       data->rcdev.nr_resets = JH7100_RSTN_END;
-       data->rcdev.dev = &pdev->dev;
-       data->rcdev.of_node = pdev->dev.of_node;
-       spin_lock_init(&data->lock);
+       if (IS_ERR(base))
+               return PTR_ERR(base);
 
-       return devm_reset_controller_register(&pdev->dev, &data->rcdev);
+       return reset_starfive_jh7100_register(&pdev->dev, pdev->dev.of_node,
+                                             base + JH7100_RESET_ASSERT0,
+                                             base + JH7100_RESET_STATUS0,
+                                             jh7100_reset_asserted,
+                                             JH7100_RSTN_END,
+                                             true);
 }
 
 static const struct of_device_id jh7100_reset_dt_ids[] = {
diff --git a/drivers/reset/starfive/reset-starfive-jh71x0.c b/drivers/reset/starfive/reset-starfive-jh71x0.c
new file mode 100644 (file)
index 0000000..1e230f3
--- /dev/null
@@ -0,0 +1,131 @@
+// SPDX-License-Identifier: GPL-2.0-or-later
+/*
+ * Reset driver for the StarFive JH7100 SoC
+ *
+ * Copyright (C) 2021 Emil Renner Berthing <kernel@esmil.dk>
+ */
+
+#include <linux/bitmap.h>
+#include <linux/device.h>
+#include <linux/io.h>
+#include <linux/io-64-nonatomic-lo-hi.h>
+#include <linux/iopoll.h>
+#include <linux/reset-controller.h>
+#include <linux/spinlock.h>
+
+struct jh7100_reset {
+       struct reset_controller_dev rcdev;
+       /* protect registers against concurrent read-modify-write */
+       spinlock_t lock;
+       void __iomem *assert;
+       void __iomem *status;
+       const u64 *asserted;
+};
+
+static inline struct jh7100_reset *
+jh7100_reset_from(struct reset_controller_dev *rcdev)
+{
+       return container_of(rcdev, struct jh7100_reset, rcdev);
+}
+
+static int jh7100_reset_update(struct reset_controller_dev *rcdev,
+                              unsigned long id, bool assert)
+{
+       struct jh7100_reset *data = jh7100_reset_from(rcdev);
+       unsigned long offset = BIT_ULL_WORD(id);
+       u64 mask = BIT_ULL_MASK(id);
+       void __iomem *reg_assert = data->assert + offset * sizeof(u64);
+       void __iomem *reg_status = data->status + offset * sizeof(u64);
+       u64 done = data->asserted ? data->asserted[offset] & mask : 0;
+       u64 value;
+       unsigned long flags;
+       int ret;
+
+       if (!assert)
+               done ^= mask;
+
+       spin_lock_irqsave(&data->lock, flags);
+
+       value = readq(reg_assert);
+       if (assert)
+               value |= mask;
+       else
+               value &= ~mask;
+       writeq(value, reg_assert);
+
+       /* if the associated clock is gated, deasserting might otherwise hang forever */
+       ret = readq_poll_timeout_atomic(reg_status, value, (value & mask) == done, 0, 1000);
+
+       spin_unlock_irqrestore(&data->lock, flags);
+       return ret;
+}
+
+static int jh7100_reset_assert(struct reset_controller_dev *rcdev,
+                              unsigned long id)
+{
+       return jh7100_reset_update(rcdev, id, true);
+}
+
+static int jh7100_reset_deassert(struct reset_controller_dev *rcdev,
+                                unsigned long id)
+{
+       return jh7100_reset_update(rcdev, id, false);
+}
+
+static int jh7100_reset_reset(struct reset_controller_dev *rcdev,
+                             unsigned long id)
+{
+       int ret;
+
+       ret = jh7100_reset_assert(rcdev, id);
+       if (ret)
+               return ret;
+
+       return jh7100_reset_deassert(rcdev, id);
+}
+
+static int jh7100_reset_status(struct reset_controller_dev *rcdev,
+                              unsigned long id)
+{
+       struct jh7100_reset *data = jh7100_reset_from(rcdev);
+       unsigned long offset = BIT_ULL_WORD(id);
+       u64 mask = BIT_ULL_MASK(id);
+       void __iomem *reg_status = data->status + offset * sizeof(u64);
+       u64 value = readq(reg_status);
+
+       return !((value ^ data->asserted[offset]) & mask);
+}
+
+static const struct reset_control_ops jh7100_reset_ops = {
+       .assert         = jh7100_reset_assert,
+       .deassert       = jh7100_reset_deassert,
+       .reset          = jh7100_reset_reset,
+       .status         = jh7100_reset_status,
+};
+
+int reset_starfive_jh7100_register(struct device *dev, struct device_node *of_node,
+                                  void __iomem *assert, void __iomem *status,
+                                  const u64 *asserted, unsigned int nr_resets,
+                                  bool is_module)
+{
+       struct jh7100_reset *data;
+
+       data = devm_kzalloc(dev, sizeof(*data), GFP_KERNEL);
+       if (!data)
+               return -ENOMEM;
+
+       data->rcdev.ops = &jh7100_reset_ops;
+       if (is_module)
+               data->rcdev.owner = THIS_MODULE;
+       data->rcdev.nr_resets = nr_resets;
+       data->rcdev.dev = dev;
+       data->rcdev.of_node = of_node;
+
+       spin_lock_init(&data->lock);
+       data->assert = assert;
+       data->status = status;
+       data->asserted = asserted;
+
+       return devm_reset_controller_register(dev, &data->rcdev);
+}
+EXPORT_SYMBOL_GPL(reset_starfive_jh7100_register);
diff --git a/drivers/reset/starfive/reset-starfive-jh71x0.h b/drivers/reset/starfive/reset-starfive-jh71x0.h
new file mode 100644 (file)
index 0000000..10770c5
--- /dev/null
@@ -0,0 +1,14 @@
+/* SPDX-License-Identifier: GPL-2.0-or-later */
+/*
+ * Copyright (C) 2021 Emil Renner Berthing <kernel@esmil.dk>
+ */
+
+#ifndef __RESET_STARFIVE_JH71X0_H
+#define __RESET_STARFIVE_JH71X0_H
+
+int reset_starfive_jh7100_register(struct device *dev, struct device_node *of_node,
+                                  void __iomem *assert, void __iomem *status,
+                                  const u64 *asserted, unsigned int nr_resets,
+                                  bool is_module);
+
+#endif /* __RESET_STARFIVE_JH71X0_H */