power: reset: at91-poweroff: introduce struct shdwc_reg_config
authorClaudiu Beznea <claudiu.beznea@microchip.com>
Fri, 20 Dec 2019 15:31:17 +0000 (17:31 +0200)
committerSebastian Reichel <sre@kernel.org>
Tue, 14 Jan 2020 22:38:28 +0000 (23:38 +0100)
This driver uses AT91_PMC_MCKR in poweroff() function. But the
SAM9X60's PMC versions maps AT91_PMC_MCKR functionality at different
offset compared to the SAMA5D2's one. This patch prepares the field
so that different AT91_PMC_MCKR's offsets to be introduced in
struct reg_config so that proper offset to be used for AT91_PMC_MCKR
based on compatible string.

Signed-off-by: Claudiu Beznea <claudiu.beznea@microchip.com>
Acked-by: Nicolas Ferre <nicolas.ferre@microchip.com>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
drivers/power/reset/at91-sama5d2_shdwc.c

index 1c18f46..84806d2 100644 (file)
@@ -66,7 +66,7 @@
 
 #define SHDW_CFG_NOT_USED      (32)
 
-struct shdwc_config {
+struct shdwc_reg_config {
        u8 wkup_pin_input;
        u8 mr_rtcwk_shift;
        u8 mr_rttwk_shift;
@@ -74,8 +74,12 @@ struct shdwc_config {
        u8 sr_rttwk_shift;
 };
 
+struct reg_config {
+       struct shdwc_reg_config shdwc;
+};
+
 struct shdwc {
-       const struct shdwc_config *cfg;
+       const struct reg_config *rcfg;
        struct clk *sclk;
        void __iomem *shdwc_base;
        void __iomem *mpddrc_base;
@@ -95,6 +99,7 @@ static const unsigned long long sdwc_dbc_period[] = {
 static void __init at91_wakeup_status(struct platform_device *pdev)
 {
        struct shdwc *shdw = platform_get_drvdata(pdev);
+       const struct reg_config *rcfg = shdw->rcfg;
        u32 reg;
        char *reason = "unknown";
 
@@ -106,11 +111,11 @@ static void __init at91_wakeup_status(struct platform_device *pdev)
        if (!reg)
                return;
 
-       if (SHDW_WK_PIN(reg, shdw->cfg))
+       if (SHDW_WK_PIN(reg, &rcfg->shdwc))
                reason = "WKUP pin";
-       else if (SHDW_RTCWK(reg, shdw->cfg))
+       else if (SHDW_RTCWK(reg, &rcfg->shdwc))
                reason = "RTC";
-       else if (SHDW_RTTWK(reg, shdw->cfg))
+       else if (SHDW_RTTWK(reg, &rcfg->shdwc))
                reason = "RTT";
 
        pr_info("AT91: Wake-Up source: %s\n", reason);
@@ -215,6 +220,7 @@ static u32 at91_shdwc_get_wakeup_input(struct platform_device *pdev,
 static void at91_shdwc_dt_configure(struct platform_device *pdev)
 {
        struct shdwc *shdw = platform_get_drvdata(pdev);
+       const struct reg_config *rcfg = shdw->rcfg;
        struct device_node *np = pdev->dev.of_node;
        u32 mode = 0, tmp, input;
 
@@ -227,10 +233,10 @@ static void at91_shdwc_dt_configure(struct platform_device *pdev)
                mode |= AT91_SHDW_WKUPDBC(at91_shdwc_debouncer_value(pdev, tmp));
 
        if (of_property_read_bool(np, "atmel,wakeup-rtc-timer"))
-               mode |= SHDW_RTCWKEN(shdw->cfg);
+               mode |= SHDW_RTCWKEN(&rcfg->shdwc);
 
        if (of_property_read_bool(np, "atmel,wakeup-rtt-timer"))
-               mode |= SHDW_RTTWKEN(shdw->cfg);
+               mode |= SHDW_RTTWKEN(&rcfg->shdwc);
 
        dev_dbg(&pdev->dev, "%s: mode = %#x\n", __func__, mode);
        writel(mode, shdw->shdwc_base + AT91_SHDW_MR);
@@ -239,30 +245,34 @@ static void at91_shdwc_dt_configure(struct platform_device *pdev)
        writel(input, shdw->shdwc_base + AT91_SHDW_WUIR);
 }
 
-static const struct shdwc_config sama5d2_shdwc_config = {
-       .wkup_pin_input = 0,
-       .mr_rtcwk_shift = 17,
-       .mr_rttwk_shift = SHDW_CFG_NOT_USED,
-       .sr_rtcwk_shift = 5,
-       .sr_rttwk_shift = SHDW_CFG_NOT_USED,
+static const struct reg_config sama5d2_reg_config = {
+       .shdwc = {
+               .wkup_pin_input = 0,
+               .mr_rtcwk_shift = 17,
+               .mr_rttwk_shift = SHDW_CFG_NOT_USED,
+               .sr_rtcwk_shift = 5,
+               .sr_rttwk_shift = SHDW_CFG_NOT_USED,
+       },
 };
 
-static const struct shdwc_config sam9x60_shdwc_config = {
-       .wkup_pin_input = 0,
-       .mr_rtcwk_shift = 17,
-       .mr_rttwk_shift = 16,
-       .sr_rtcwk_shift = 5,
-       .sr_rttwk_shift = 4,
+static const struct reg_config sam9x60_reg_config = {
+       .shdwc = {
+               .wkup_pin_input = 0,
+               .mr_rtcwk_shift = 17,
+               .mr_rttwk_shift = 16,
+               .sr_rtcwk_shift = 5,
+               .sr_rttwk_shift = 4,
+       },
 };
 
 static const struct of_device_id at91_shdwc_of_match[] = {
        {
                .compatible = "atmel,sama5d2-shdwc",
-               .data = &sama5d2_shdwc_config,
+               .data = &sama5d2_reg_config,
        },
        {
                .compatible = "microchip,sam9x60-shdwc",
-               .data = &sam9x60_shdwc_config,
+               .data = &sam9x60_reg_config,
        }, {
                /*sentinel*/
        }
@@ -303,7 +313,7 @@ static int __init at91_shdwc_probe(struct platform_device *pdev)
        }
 
        match = of_match_node(at91_shdwc_of_match, pdev->dev.of_node);
-       at91_shdwc->cfg = match->data;
+       at91_shdwc->rcfg = match->data;
 
        at91_shdwc->sclk = devm_clk_get(&pdev->dev, NULL);
        if (IS_ERR(at91_shdwc->sclk))