回复: [PATCH v9 2/3] reset: add Sky1 soc reset support
Gary Yang
gary.yang at cixtech.com
Tue Feb 24 02:34:01 PST 2026
Hi Philipp:
Thanks for your comments
> EXTERNAL EMAIL
>
> CAUTION: Suspicious Email from unusual domain.
>
> On Di, 2026-02-24 at 17:21 +0800, Gary Yang wrote:
> > Add support for the resets on Cix's Sky1 SoC.
> > There are two reset controllers on Cix Sky1 Soc. One is located in S0
> > domain, and the other is located in S5 domain.
>
> The second controller is not in S0 domain anymore?
> This change was not explained in the change log.
>
> This seems to contradict a comment in
> "cix,sky1-s5-system-control" -> variant_sky1 -> sky1_src_signals:
>
> /* reset group1 for s0 domain modules */
>
I'm sorry. It makes you confuse. It doesn't have any changes.
The reset controller in S0 domain is only used in S0 domain.
The other one in S5 domain is both used in S0 and S5 domain.
This description maybe be different from the different views.
> > Signed-off-by: Gary Yang <gary.yang at cixtech.com>
> > ---
> > drivers/reset/Kconfig | 7 +
> > drivers/reset/Makefile | 1 +
> > drivers/reset/reset-sky1.c | 367
> > +++++++++++++++++++++++++++++++++++++
> > 3 files changed, 375 insertions(+)
> > create mode 100644 drivers/reset/reset-sky1.c
> >
> > diff --git a/drivers/reset/Kconfig b/drivers/reset/Kconfig index
> > 6e5d6deffa7d..0aaac1b88d02 100644
> > --- a/drivers/reset/Kconfig
> > +++ b/drivers/reset/Kconfig
> > @@ -291,6 +291,13 @@ config RESET_SIMPLE
> > - SiFive FU740 SoCs
> > - Sophgo SoCs
> >
> > +config RESET_SKY1
> > + bool "Cix Sky1 reset controller"
> > + depends on OF && REGMAP_MMIO
>
> Select REGMAP_MMIO instead.
>
OK, we will do it next version.
> > + depends on ARCH_CIX || COMPILE_TEST
> > + help
> > + This enables the reset controller for Cix Sky1.
> > +
> > config RESET_SOCFPGA
> > bool "SoCFPGA Reset Driver" if COMPILE_TEST && (!ARM
> || !ARCH_INTEL_SOCFPGA)
> > default ARM && ARCH_INTEL_SOCFPGA diff --git
> > a/drivers/reset/Makefile b/drivers/reset/Makefile index
> > 9c3e484dfd81..0d2e1329561d 100644
> > --- a/drivers/reset/Makefile
> > +++ b/drivers/reset/Makefile
> > @@ -37,6 +37,7 @@ obj-$(CONFIG_RESET_RZG2L_USBPHY_CTRL) +=
> > reset-rzg2l-usbphy-ctrl.o
> > obj-$(CONFIG_RESET_RZV2H_USB2PHY) += reset-rzv2h-usb2phy.o
> > obj-$(CONFIG_RESET_SCMI) += reset-scmi.o
> > obj-$(CONFIG_RESET_SIMPLE) += reset-simple.o
> > +obj-$(CONFIG_RESET_SKY1) += reset-sky1.o
> > obj-$(CONFIG_RESET_SOCFPGA) += reset-socfpga.o
> > obj-$(CONFIG_RESET_SPACEMIT) += reset-spacemit.o
> > obj-$(CONFIG_RESET_SUNPLUS) += reset-sunplus.o diff --git
> > a/drivers/reset/reset-sky1.c b/drivers/reset/reset-sky1.c new file
> > mode 100644 index 000000000000..fb8a919fd3f7
> > --- /dev/null
> > +++ b/drivers/reset/reset-sky1.c
> > @@ -0,0 +1,367 @@
> > +// SPDX-License-Identifier: GPL-2.0-only
> > +/*
> > + *
> > + * CIX System Reset Controller (SRC) driver
> > + *
> > + * Author: Jerry Zhu <jerry.zhu at cixtech.com> */
> > +
> > +#include <linux/delay.h>
> > +#include <linux/mfd/syscon.h>
> > +#include <linux/module.h>
> > +#include <linux/of.h>
> > +#include <linux/platform_device.h>
> > +#include <linux/regmap.h>
> > +#include <linux/reset-controller.h>
> > +
> > +#include <dt-bindings/reset/cix,sky1-rst.h>
> > +#include <dt-bindings/reset/cix,sky1-rst-fch.h>
> > +
> > +#define SKY1_RESET_SLEEP_MIN_US 50
> > +#define SKY1_RESET_SLEEP_MAX_US 100
> > +
> > +struct sky1_src_signal {
> > + unsigned int offset;
> > + unsigned int bit;
> > +};
> > +
> > +struct sky1_src_variant {
> > + const struct sky1_src_signal *signals;
> > + unsigned int signals_num;
> > +};
> > +
> > +struct sky1_src {
> > + struct reset_controller_dev rcdev;
> > + const struct sky1_src_signal *signals;
> > + struct regmap *regmap;
> > +};
> > +
> > +enum {
> > + CSU_PM_RESET = 0x304,
> > + SENSORFUSION_RESET = 0x308,
> > + SENSORFUSION_NOC_RESET = 0x30c,
> > + RESET_GROUP0_S0_DOMAIN_0 = 0x400,
> > + RESET_GROUP0_S0_DOMAIN_1 = 0x404,
> > + RESET_GROUP1_USB_PHYS = 0x408,
> > + RESET_GROUP1_USB_CONTROLLERS = 0x40c,
> > + RESET_GROUP0_RCSU = 0x800,
> > + RESET_GROUP1_RCSU = 0x804,
> > +};
> > +
> > +static const struct sky1_src_signal sky1_src_signals[] = {
> > + /* reset group1 for s0 domain modules */
> > + [SKY1_CSU_PM_RESET_N] = { CSU_PM_RESET, BIT(0) },
> > + [SKY1_SENSORFUSION_RESET_N] = { SENSORFUSION_RESET,
> BIT(0) },
> > + [SKY1_SENSORFUSION_NOC_RESET_N] =
> { SENSORFUSION_NOC_RESET, BIT(0) },
> > + [SKY1_DDRC_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(0) },
> > + [SKY1_GIC_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(1) },
> > + [SKY1_CI700_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(2) },
> > + [SKY1_SYS_NI700_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(3) },
> > + [SKY1_MM_NI700_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(4) },
> > + [SKY1_PCIE_NI700_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(5) },
> > + [SKY1_GPU_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(6) },
> > + [SKY1_NPUTOP_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(7) },
> > + [SKY1_NPUCORE0_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(8) },
> > + [SKY1_NPUCORE1_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(9) },
> > + [SKY1_NPUCORE2_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(10) },
> > + [SKY1_VPU_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(11) },
> > + [SKY1_ISP_SRESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(12) },
> > + [SKY1_ISP_ARESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(13) },
> > + [SKY1_ISP_HRESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(14) },
> > + [SKY1_ISP_GDCRESET_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(15) },
> > + [SKY1_DPU_RESET0_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(16) },
> > + [SKY1_DPU_RESET1_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(17) },
> > + [SKY1_DPU_RESET2_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(18) },
> > + [SKY1_DPU_RESET3_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(19) },
> > + [SKY1_DPU_RESET4_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(20) },
> > + [SKY1_DP_RESET0_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(21) },
> > + [SKY1_DP_RESET1_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(22) },
> > + [SKY1_DP_RESET2_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(23) },
> > + [SKY1_DP_RESET3_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(24) },
> > + [SKY1_DP_RESET4_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(25) },
> > + [SKY1_DP_PHY_RST_N] =
> { RESET_GROUP0_S0_DOMAIN_0, BIT(26) },
> > +
> > + /* reset group1 for s0 domain modules */
> > + [SKY1_AUDIO_HIFI5_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(0) },
> > + [SKY1_AUDIO_HIFI5_NOC_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(1) },
> > + [SKY1_CSIDPHY_PRST0_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(2) },
> > + [SKY1_CSIDPHY_CMNRST0_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(3) },
> > + [SKY1_CSI0_RST_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(4) },
> > + [SKY1_CSIDPHY_PRST1_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(5) },
> > + [SKY1_CSIDPHY_CMNRST1_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(6) },
> > + [SKY1_CSI1_RST_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(7) },
> > + [SKY1_CSI2_RST_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(8) },
> > + [SKY1_CSI3_RST_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(9) },
> > + [SKY1_CSIBRDGE0_RST_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(10) },
> > + [SKY1_CSIBRDGE1_RST_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(11) },
> > + [SKY1_CSIBRDGE2_RST_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(12) },
> > + [SKY1_CSIBRDGE3_RST_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(13) },
> > + [SKY1_GMAC0_RST_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(14) },
> > + [SKY1_GMAC1_RST_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(15) },
> > + [SKY1_PCIE0_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(16) },
> > + [SKY1_PCIE1_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(17) },
> > + [SKY1_PCIE2_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(18) },
> > + [SKY1_PCIE3_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(19) },
> > + [SKY1_PCIE4_RESET_N] =
> { RESET_GROUP0_S0_DOMAIN_1, BIT(20) },
> > +
> > + /* reset group1 for usb phys */
> > + [SKY1_USB_DP_PHY0_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(0) },
> > + [SKY1_USB_DP_PHY1_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(1) },
> > + [SKY1_USB_DP_PHY2_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(2) },
> > + [SKY1_USB_DP_PHY3_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(3) },
> > + [SKY1_USB_DP_PHY0_RST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(4) },
> > + [SKY1_USB_DP_PHY1_RST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(5) },
> > + [SKY1_USB_DP_PHY2_RST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(6) },
> > + [SKY1_USB_DP_PHY3_RST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(7) },
> > + [SKY1_USBPHY_SS_PST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(8) },
> > + [SKY1_USBPHY_SS_RST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(9) },
> > + [SKY1_USBPHY_HS0_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(10) },
> > + [SKY1_USBPHY_HS1_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(11) },
> > + [SKY1_USBPHY_HS2_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(12) },
> > + [SKY1_USBPHY_HS3_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(13) },
> > + [SKY1_USBPHY_HS4_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(14) },
> > + [SKY1_USBPHY_HS5_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(15) },
> > + [SKY1_USBPHY_HS6_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(16) },
> > + [SKY1_USBPHY_HS7_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(17) },
> > + [SKY1_USBPHY_HS8_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(18) },
> > + [SKY1_USBPHY_HS9_PRST_N] =
> { RESET_GROUP1_USB_PHYS, BIT(19) },
> > +
> > + /* reset group1 for usb controllers */
> > + [SKY1_USBC_SS0_PRST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(0) },
> > + [SKY1_USBC_SS1_PRST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(1) },
> > + [SKY1_USBC_SS2_PRST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(2) },
> > + [SKY1_USBC_SS3_PRST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(3) },
> > + [SKY1_USBC_SS4_PRST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(4) },
> > + [SKY1_USBC_SS5_PRST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(5) },
> > + [SKY1_USBC_SS0_RST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(6) },
> > + [SKY1_USBC_SS1_RST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(7) },
> > + [SKY1_USBC_SS2_RST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(8) },
> > + [SKY1_USBC_SS3_RST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(9) },
> > + [SKY1_USBC_SS4_RST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(10) },
> > + [SKY1_USBC_SS5_RST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(11) },
> > + [SKY1_USBC_HS0_PRST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(12) },
> > + [SKY1_USBC_HS1_PRST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(13) },
> > + [SKY1_USBC_HS2_PRST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(14) },
> > + [SKY1_USBC_HS3_PRST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(15) },
> > + [SKY1_USBC_HS0_RST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(16) },
> > + [SKY1_USBC_HS1_RST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(17) },
> > + [SKY1_USBC_HS2_RST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(18) },
> > + [SKY1_USBC_HS3_RST_N] =
> { RESET_GROUP1_USB_CONTROLLERS, BIT(19) },
> > +
> > + /* reset group0 for rcsu */
> > + [SKY1_AUDIO_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(0) },
> > + [SKY1_CI700_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(1) },
> > + [SKY1_CSI_RCSU0_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(2) },
> > + [SKY1_CSI_RCSU1_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(3) },
> > + [SKY1_CSU_PM_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(4) },
> > + [SKY1_DDR_BROADCAST_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(5) },
> > + [SKY1_DDR_CTRL_RCSU_0_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(6) },
> > + [SKY1_DDR_CTRL_RCSU_1_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(7) },
> > + [SKY1_DDR_CTRL_RCSU_2_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(8) },
> > + [SKY1_DDR_CTRL_RCSU_3_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(9) },
> > + [SKY1_DDR_TZC400_RCSU_0_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(10) },
> > + [SKY1_DDR_TZC400_RCSU_1_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(11) },
> > + [SKY1_DDR_TZC400_RCSU_2_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(12) },
> > + [SKY1_DDR_TZC400_RCSU_3_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(13) },
> > + [SKY1_DP0_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(14) },
> > + [SKY1_DP1_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(15) },
> > + [SKY1_DP2_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(16) },
> > + [SKY1_DP3_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(17) },
> > + [SKY1_DP4_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(18) },
> > + [SKY1_DPU0_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(19) },
> > + [SKY1_DPU1_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(20) },
> > + [SKY1_DPU2_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(21) },
> > + [SKY1_DPU3_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(22) },
> > + [SKY1_DPU4_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(23) },
> > + [SKY1_DSU_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(24) },
> > + [SKY1_FCH_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(25) },
> > + [SKY1_GICD_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(26) },
> > + [SKY1_GMAC_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(27) },
> > + [SKY1_GPU_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(28) },
> > + [SKY1_ISP_RCSU0_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(29) },
> > + [SKY1_ISP_RCSU1_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(30) },
> > + [SKY1_NI700_MMHUB_RCSU_RESET_N] =
> { RESET_GROUP0_RCSU, BIT(31) },
> > +
> > + /* reset group1 for rcsu */
> > + [SKY1_NPU_RCSU_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(0) },
> > + [SKY1_NI700_PCIE_RCSU_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(1) },
> > + [SKY1_PCIE_X421_RCSU_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(2) },
> > + [SKY1_PCIE_X8_RCSU_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(3) },
> > + [SKY1_SF_RCSU_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(4) },
> > + [SKY1_RCSU_SMMU_MMHUB_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(5) },
> > + [SKY1_RCSU_SMMU_PCIEHUB_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(6) },
> > + [SKY1_RCSU_SYSHUB_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(7) },
> > + [SKY1_NI700_SMN_RCSU_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(8) },
> > + [SKY1_NI700_SYSHUB_RCSU_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(9) },
> > + [SKY1_RCSU_USB2_HOST0_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(10) },
> > + [SKY1_RCSU_USB2_HOST1_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(11) },
> > + [SKY1_RCSU_USB2_HOST2_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(12) },
> > + [SKY1_RCSU_USB2_HOST3_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(13) },
> > + [SKY1_RCSU_USB3_TYPEA_DRD_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(14) },
> > + [SKY1_RCSU_USB3_TYPEC_DRD_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(15) },
> > + [SKY1_RCSU_USB3_TYPEC_HOST0_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(16) },
> > + [SKY1_RCSU_USB3_TYPEC_HOST1_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(17) },
> > + [SKY1_RCSU_USB3_TYPEC_HOST2_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(18) },
> > + [SKY1_VPU_RCSU_RESET_N] =
> { RESET_GROUP1_RCSU, BIT(19) },
> > +};
> > +
> > +static const struct sky1_src_variant variant_sky1 = {
> > + .signals = sky1_src_signals,
> > + .signals_num = ARRAY_SIZE(sky1_src_signals), };
> > +
> > +enum {
> > + FCH_SW_RST_FUNC = 0x8,
> > + FCH_SW_RST_BUS = 0xc,
> > + FCH_SW_XSPI = 0x10,
> > +};
> > +
> > +static const struct sky1_src_signal sky1_src_fch_signals[] = {
> > + /* resets for fch_sw_rst_func */
> > + [SW_I3C0_RST_FUNC_G_N] = { FCH_SW_RST_FUNC, BIT(0) },
> > + [SW_I3C0_RST_FUNC_I_N] = { FCH_SW_RST_FUNC, BIT(1) },
> > + [SW_I3C1_RST_FUNC_G_N] = { FCH_SW_RST_FUNC, BIT(2) },
> > + [SW_I3C1_RST_FUNC_I_N] = { FCH_SW_RST_FUNC, BIT(3) },
> > + [SW_UART0_RST_FUNC_N] = { FCH_SW_RST_FUNC, BIT(4) },
> > + [SW_UART1_RST_FUNC_N] = { FCH_SW_RST_FUNC, BIT(5) },
> > + [SW_UART2_RST_FUNC_N] = { FCH_SW_RST_FUNC, BIT(6) },
> > + [SW_UART3_RST_FUNC_N] = { FCH_SW_RST_FUNC, BIT(7) },
> > + [SW_TIMER_RST_FUNC_N] = { FCH_SW_RST_FUNC, BIT(20) },
> > +
> > + /* resets for fch_sw_rst_bus */
> > + [SW_I3C0_RST_APB_N] = { FCH_SW_RST_BUS, BIT(0) },
> > + [SW_I3C1_RST_APB_N] = { FCH_SW_RST_BUS, BIT(1) },
> > + [SW_DMA_RST_AXI_N] = { FCH_SW_RST_BUS, BIT(2) },
> > + [SW_UART0_RST_APB_N] = { FCH_SW_RST_BUS, BIT(4) },
> > + [SW_UART1_RST_APB_N] = { FCH_SW_RST_BUS, BIT(5) },
> > + [SW_UART2_RST_APB_N] = { FCH_SW_RST_BUS, BIT(6) },
> > + [SW_UART3_RST_APB_N] = { FCH_SW_RST_BUS, BIT(7) },
> > + [SW_SPI0_RST_APB_N] = { FCH_SW_RST_BUS, BIT(8) },
> > + [SW_SPI1_RST_APB_N] = { FCH_SW_RST_BUS, BIT(9) },
> > + [SW_I2C0_RST_APB_N] = { FCH_SW_RST_BUS, BIT(12) },
> > + [SW_I2C1_RST_APB_N] = { FCH_SW_RST_BUS, BIT(13) },
> > + [SW_I2C2_RST_APB_N] = { FCH_SW_RST_BUS, BIT(14) },
> > + [SW_I2C3_RST_APB_N] = { FCH_SW_RST_BUS, BIT(15) },
> > + [SW_I2C4_RST_APB_N] = { FCH_SW_RST_BUS, BIT(16) },
> > + [SW_I2C5_RST_APB_N] = { FCH_SW_RST_BUS, BIT(17) },
> > + [SW_I2C6_RST_APB_N] = { FCH_SW_RST_BUS, BIT(18) },
> > + [SW_I2C7_RST_APB_N] = { FCH_SW_RST_BUS, BIT(19) },
> > + [SW_GPIO_RST_APB_N] = { FCH_SW_RST_BUS, BIT(21) },
> > +
> > + /* resets for fch_sw_xspi */
> > + [SW_XSPI_REG_RST_N] = { FCH_SW_XSPI, BIT(0) },
> > + [SW_XSPI_SYS_RST_N] = { FCH_SW_XSPI, BIT(1) },
> > +};
> > +
> > +static const struct sky1_src_variant variant_sky1_fch = {
> > + .signals = sky1_src_fch_signals,
> > + .signals_num = ARRAY_SIZE(sky1_src_fch_signals), };
> > +
> > +static struct sky1_src *to_sky1_src(struct reset_controller_dev
> > +*rcdev) {
> > + return container_of(rcdev, struct sky1_src, rcdev); }
> > +
> > +static int sky1_reset_set(struct reset_controller_dev *rcdev,
> > + unsigned long id, bool assert) {
> > + struct sky1_src *sky1src = to_sky1_src(rcdev);
> > + const struct sky1_src_signal *signal = &sky1src->signals[id];
> > + unsigned int value = assert ? 0 : sky1src->signals[id].bit;
>
> This could be shortened to:
>
> unsigned int value = assert ? 0 : signal->bit;
>
Yes, you're right. We will do it next version
> > +
> > + return regmap_update_bits(sky1src->regmap,
> > + signal->offset, signal->bit, value); }
> > +
> > +static int sky1_reset_assert(struct reset_controller_dev *rcdev,
> > + unsigned long id) {
> > + sky1_reset_set(rcdev, id, true);
> > + usleep_range(SKY1_RESET_SLEEP_MIN_US,
> > + SKY1_RESET_SLEEP_MAX_US);
> > + return 0;
> > +}
> > +
> > +static int sky1_reset_deassert(struct reset_controller_dev *rcdev,
> > + unsigned long id) {
> > + sky1_reset_set(rcdev, id, false);
> > + usleep_range(SKY1_RESET_SLEEP_MIN_US,
> > + SKY1_RESET_SLEEP_MAX_US);
> > + return 0;
> > +}
> > +
> > +static int sky1_reset(struct reset_controller_dev *rcdev,
> > + unsigned long id)
> > +{
> > + sky1_reset_assert(rcdev, id);
> > + sky1_reset_deassert(rcdev, id);
> > + return 0;
> > +}
> > +
> > +static int sky1_reset_status(struct reset_controller_dev *rcdev,
> > + unsigned long id) {
> > + unsigned int value = 0;
> > + struct sky1_src *sky1src = to_sky1_src(rcdev);
> > + const struct sky1_src_signal *signal = &sky1src->signals[id];
> > +
> > + regmap_read(sky1src->regmap, signal->offset, &value);
> > + return !(value & signal->bit);
> > +}
> > +
> > +static const struct reset_control_ops sky1_src_ops = {
> > + .reset = sky1_reset,
> > + .assert = sky1_reset_assert,
> > + .deassert = sky1_reset_deassert,
> > + .status = sky1_reset_status
> > +};
> > +
> > +static int sky1_reset_probe(struct platform_device *pdev) {
> > + struct sky1_src *sky1src;
> > + struct device *dev = &pdev->dev;
> > + const struct sky1_src_variant *variant;
> > +
> > + sky1src = devm_kzalloc(dev, sizeof(*sky1src), GFP_KERNEL);
> > + if (!sky1src)
> > + return -ENOMEM;
> > +
> > + variant = of_device_get_match_data(dev);
> > +
> > + sky1src->regmap = device_node_to_regmap(dev->of_node);
> > + if (IS_ERR(sky1src->regmap)) {
> > + dev_err(dev, "Unable to get sky1-src regmap");
> > + return PTR_ERR(sky1src->regmap);
>
> This can be simplified with dev_err_probe().
>
The same
If miss any information, please remind me. Thanks
Best Regards
Gary
> regards
> Philipp
More information about the linux-arm-kernel
mailing list