| Message ID | ecd0c36a9e1766e327b54afa6bf986e64890b881.1477501000.git.stwiss.opensource@diasemi.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Wed, Oct 26, 2016 at 05:56:39PM +0100, Steve Twiss wrote: > From: Steve Twiss <stwiss.opensource@diasemi.com> > > The of_device_id match array is added to support "dlg,da9062-watchdog" > as a valid .compatible string. > > The watchdog_info structure is linked to this device tree compatible > string in the .data section. Extra code is added into the probe function to > search-for and assign this match data. > > This patch now assumes the use of a DA9062 fallback compatible string for > the DTS to pick up the DA9062 device driver for use with the DA9061 > watchdog hardware. > > Copyright header is updated to add DA9061 in its description and the module > description macro is extended to include DA9061. > > Kconfig is updated to reflect support for DA9061/62. > > Signed-off-by: Steve Twiss <stwiss.opensource@diasemi.com> > > --- > This patch applies against linux-next and v4.8 > > v1 -> v2 > - Patch renamed from [PATCH V1 04/10] to [PATCH V2 08/10] -- these > changes were made to fix checkpatch warnings caused by the patch > set dependency order > - Removal of the DA9061 compatible entries for this device driver. > - Additional explanation in the patch description for the use of a > fall-back compatible DTS string > > Guenter, > > Alterations have been made in accordance with the previous e-mail thread > on the use of compatible strings: https://lkml.org/lkml/2016/10/7/641 > This patch now assumes the use of a fallback compatible string in the DTS. > Of the form: compatible = "dlg,da9061-watchdog", "dlg,da9062-watchdog"; > > Regards, > Steve Twiss, Dialog Semiconductor Ltd. > > > drivers/watchdog/Kconfig | 4 ++-- > drivers/watchdog/da9062_wdt.c | 18 +++++++++++++++--- > 2 files changed, 17 insertions(+), 5 deletions(-) > > diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig > index 1bffe00..d6b4088 100644 > --- a/drivers/watchdog/Kconfig > +++ b/drivers/watchdog/Kconfig > @@ -104,11 +104,11 @@ config DA9063_WATCHDOG > This driver can be built as a module. The module name is da9063_wdt. > > config DA9062_WATCHDOG > - tristate "Dialog DA9062 Watchdog" > + tristate "Dialog DA9062/61 Watchdog" > depends on MFD_DA9062 > select WATCHDOG_CORE > help > - Support for the watchdog in the DA9062 PMIC. > + Support for the watchdog in the DA9062 and DA9061 PMICs. > > This driver can be built as a module. The module name is da9062_wdt. > > diff --git a/drivers/watchdog/da9062_wdt.c b/drivers/watchdog/da9062_wdt.c > index 7386111..4b20b7a 100644 > --- a/drivers/watchdog/da9062_wdt.c > +++ b/drivers/watchdog/da9062_wdt.c > @@ -1,5 +1,5 @@ > /* > - * da9062_wdt.c - WDT device driver for DA9062 > + * Watchdog device driver for DA9062 and DA9061 PMICs > * Copyright (C) 2015 Dialog Semiconductor Ltd. > * > * This program is free software; you can redistribute it and/or > @@ -188,23 +188,34 @@ static const struct watchdog_ops da9062_watchdog_ops = { > .set_timeout = da9062_wdt_set_timeout, > }; > > +static const struct of_device_id da9062_compatible_id_table[] = { > + { .compatible = "dlg,da9062-watchdog", .data = &da9062_watchdog_info }, > + { }, > +}; > + > static int da9062_wdt_probe(struct platform_device *pdev) > { > int ret; > struct da9062 *chip; > struct da9062_watchdog *wdt; > + const struct of_device_id *match; > > chip = dev_get_drvdata(pdev->dev.parent); > if (!chip) > return -EINVAL; > > + match = of_match_node(da9062_compatible_id_table, > + pdev->dev.of_node); > + if (!match) > + return -ENXIO; > + > wdt = devm_kzalloc(&pdev->dev, sizeof(*wdt), GFP_KERNEL); > if (!wdt) > return -ENOMEM; > > wdt->hw = chip; > > - wdt->wdtdev.info = &da9062_watchdog_info; > + wdt->wdtdev.info = (const struct watchdog_info *)match->data; I don't see why you would need match->data or of_match_node above. Guenter > wdt->wdtdev.ops = &da9062_watchdog_ops; > wdt->wdtdev.min_timeout = DA9062_WDT_MIN_TIMEOUT; > wdt->wdtdev.max_timeout = DA9062_WDT_MAX_TIMEOUT; > @@ -244,11 +255,12 @@ static struct platform_driver da9062_wdt_driver = { > .remove = da9062_wdt_remove, > .driver = { > .name = "da9062-watchdog", > + .of_match_table = da9062_compatible_id_table, > }, > }; > module_platform_driver(da9062_wdt_driver); > > MODULE_AUTHOR("S Twiss <stwiss.opensource@diasemi.com>"); > -MODULE_DESCRIPTION("WDT device driver for Dialog DA9062"); > +MODULE_DESCRIPTION("WDT device driver for Dialog DA9062 and DA9061"); > MODULE_LICENSE("GPL"); > MODULE_ALIAS("platform:da9062-watchdog"); > -- > end-of-patch for PATCH V2 > -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 26 October 2016 19:59 Guenter Roeck wrote: > On Wed, Oct 26, 2016 at 05:56:39PM +0100, Steve Twiss wrote: > > From: Steve Twiss <stwiss.opensource@diasemi.com> > > > > +static const struct of_device_id da9062_compatible_id_table[] = { > > + { .compatible = "dlg,da9062-watchdog", .data = &da9062_watchdog_info }, > > + { }, > > +}; > > + > > static int da9062_wdt_probe(struct platform_device *pdev) > > { > > int ret; > > struct da9062 *chip; > > struct da9062_watchdog *wdt; > > + const struct of_device_id *match; > > > > chip = dev_get_drvdata(pdev->dev.parent); > > if (!chip) > > return -EINVAL; > > > > + match = of_match_node(da9062_compatible_id_table, > > + pdev->dev.of_node); > > + if (!match) > > + return -ENXIO; > > + > > wdt = devm_kzalloc(&pdev->dev, sizeof(*wdt), GFP_KERNEL); > > if (!wdt) > > return -ENOMEM; > > > > wdt->hw = chip; > > > > - wdt->wdtdev.info = &da9062_watchdog_info; > > + wdt->wdtdev.info = (const struct watchdog_info *)match->data; > > I don't see why you would need match->data or of_match_node above. Hi Guenter, True. I do not need to do any matching on the table any more if the assumption is to use a fall-back compatible string for DA9061 devices (to use the da9062 device driver). I can erase most of that content. Including the .data = &da9062_watchdog_info. Regards, Steve -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig index 1bffe00..d6b4088 100644 --- a/drivers/watchdog/Kconfig +++ b/drivers/watchdog/Kconfig @@ -104,11 +104,11 @@ config DA9063_WATCHDOG This driver can be built as a module. The module name is da9063_wdt. config DA9062_WATCHDOG - tristate "Dialog DA9062 Watchdog" + tristate "Dialog DA9062/61 Watchdog" depends on MFD_DA9062 select WATCHDOG_CORE help - Support for the watchdog in the DA9062 PMIC. + Support for the watchdog in the DA9062 and DA9061 PMICs. This driver can be built as a module. The module name is da9062_wdt. diff --git a/drivers/watchdog/da9062_wdt.c b/drivers/watchdog/da9062_wdt.c index 7386111..4b20b7a 100644 --- a/drivers/watchdog/da9062_wdt.c +++ b/drivers/watchdog/da9062_wdt.c @@ -1,5 +1,5 @@ /* - * da9062_wdt.c - WDT device driver for DA9062 + * Watchdog device driver for DA9062 and DA9061 PMICs * Copyright (C) 2015 Dialog Semiconductor Ltd. * * This program is free software; you can redistribute it and/or @@ -188,23 +188,34 @@ static const struct watchdog_ops da9062_watchdog_ops = { .set_timeout = da9062_wdt_set_timeout, }; +static const struct of_device_id da9062_compatible_id_table[] = { + { .compatible = "dlg,da9062-watchdog", .data = &da9062_watchdog_info }, + { }, +}; + static int da9062_wdt_probe(struct platform_device *pdev) { int ret; struct da9062 *chip; struct da9062_watchdog *wdt; + const struct of_device_id *match; chip = dev_get_drvdata(pdev->dev.parent); if (!chip) return -EINVAL; + match = of_match_node(da9062_compatible_id_table, + pdev->dev.of_node); + if (!match) + return -ENXIO; + wdt = devm_kzalloc(&pdev->dev, sizeof(*wdt), GFP_KERNEL); if (!wdt) return -ENOMEM; wdt->hw = chip; - wdt->wdtdev.info = &da9062_watchdog_info; + wdt->wdtdev.info = (const struct watchdog_info *)match->data; wdt->wdtdev.ops = &da9062_watchdog_ops; wdt->wdtdev.min_timeout = DA9062_WDT_MIN_TIMEOUT; wdt->wdtdev.max_timeout = DA9062_WDT_MAX_TIMEOUT; @@ -244,11 +255,12 @@ static struct platform_driver da9062_wdt_driver = { .remove = da9062_wdt_remove, .driver = { .name = "da9062-watchdog", + .of_match_table = da9062_compatible_id_table, }, }; module_platform_driver(da9062_wdt_driver); MODULE_AUTHOR("S Twiss <stwiss.opensource@diasemi.com>"); -MODULE_DESCRIPTION("WDT device driver for Dialog DA9062"); +MODULE_DESCRIPTION("WDT device driver for Dialog DA9062 and DA9061"); MODULE_LICENSE("GPL"); MODULE_ALIAS("platform:da9062-watchdog");