Re: [PATCH 5/6] rtc: max77620: add support for max77620/max20024 RTC driver

From: Lee Jones
Date: Mon Jan 11 2016 - 00:46:59 EST


From: Linux Kernel <linuxkernelmails@xxxxxxxxx>

Really? Why the shroud of secrecy?

Any reason why you're not using your real name?

On Fri, 08 Jan 2016, Linux Kernel wrote:
> On Thursday 07 January 2016 08:08 PM, Laxman Dewangan wrote:
>
> >Maxim Semiconductor's PMIC MAX77620/MAX20024 has on chip
> >RTC module. This support for setting alarm and time.
> >
> >Add RTC driver to access this chip's RTC module via RTC
> >APIs.
> >
> >Signed-off-by: Laxman Dewangan <ldewangan@xxxxxxxxxx>
> >Signed-off-by: Chaitanya Bandi <bandik@xxxxxxxxxx>
> >Tested-by: Venkat Reddy Talla <vreddytalla@xxxxxxxxxx>
> >---
> > drivers/rtc/Kconfig | 9 +
> > drivers/rtc/Makefile | 1 +
> > drivers/rtc/rtc-max77620.c | 574 +++++++++++++++++++++++++++++++++++++++++++++
> > 3 files changed, 584 insertions(+)
> > create mode 100644 drivers/rtc/rtc-max77620.c
> >
> (...)
>
> >+static struct platform_driver max77620_rtc_driver = {
> >+ .probe = max77620_rtc_probe,
> >+ .remove = max77620_rtc_remove,
> >+ .id_table = max77620_rtc_devtype,
> >+ .driver = {
> >+ .name = "max77620-rtc",
> >+ .owner = THIS_MODULE,
>
> Drop this line, .owner will be populated by driver core.
>
> >+ .pm = &max77620_rtc_pm_ops,
> >+ },
> >+};
> >+
> >+module_platform_driver(max77620_rtc_driver);
> >+
> >+MODULE_DESCRIPTION("max77620 RTC driver");
> >+MODULE_AUTHOR("Chaitanya Bandi <bandik@xxxxxxxxxx>");
> >+MODULE_AUTHOR("Laxman Dewangan <ldewangan@xxxxxxxxxx>");
> >+MODULE_ALIAS("platform:max77620-rtc");
> >+MODULE_LICENSE("GPL v2");

--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog