rtc: m48t86: put m48t86_nvmem_cfg on the stack
authorAlexandre Belloni <alexandre.belloni@bootlin.com>
Mon, 12 Feb 2018 22:47:27 +0000 (23:47 +0100)
committerAlexandre Belloni <alexandre.belloni@bootlin.com>
Thu, 1 Mar 2018 09:49:19 +0000 (10:49 +0100)
Move m48t86_nvmem_cfg to the stack of m48t86_rtc_probe. This results in a
very small code size reduction and make it safer on systems with two
similar RTCs:

   text    data     bss     dec     hex filename
   1733     164       0    1897     769 drivers/rtc/rtc-m48t86.o.before
   1793     100       0    1893     765 drivers/rtc/rtc-m48t86.o.after

Signed-off-by: Alexandre Belloni <alexandre.belloni@bootlin.com>
drivers/rtc/rtc-m48t86.c

index 3a48a7e..8d17be1 100644 (file)
@@ -218,21 +218,21 @@ static bool m48t86_verify_chip(struct platform_device *pdev)
        return false;
 }
 
-static struct nvmem_config m48t86_nvmem_cfg = {
-       .name = "m48t86_nvram",
-       .word_size = 1,
-       .stride = 1,
-       .size = M48T86_NVRAM_LEN,
-       .reg_read = m48t86_nvram_read,
-       .reg_write = m48t86_nvram_write,
-};
-
 static int m48t86_rtc_probe(struct platform_device *pdev)
 {
        struct m48t86_rtc_info *info;
        struct resource *res;
        unsigned char reg;
        int err;
+       struct nvmem_config m48t86_nvmem_cfg = {
+               .name = "m48t86_nvram",
+               .word_size = 1,
+               .stride = 1,
+               .size = M48T86_NVRAM_LEN,
+               .reg_read = m48t86_nvram_read,
+               .reg_write = m48t86_nvram_write,
+               .priv = &pdev->dev,
+       };
 
        info = devm_kzalloc(&pdev->dev, sizeof(*info), GFP_KERNEL);
        if (!info)
@@ -264,8 +264,6 @@ static int m48t86_rtc_probe(struct platform_device *pdev)
                return PTR_ERR(info->rtc);
 
        info->rtc->ops = &m48t86_rtc_ops;
-
-       m48t86_nvmem_cfg.priv = &pdev->dev;
        info->rtc->nvram_old_abi = true;
 
        err = rtc_register_device(info->rtc);