lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 29 Oct 2021 14:37:51 +0200 From: Rafał Miłecki <rafal@...ecki.pl> To: Florian Fainelli <f.fainelli@...il.com> Cc: linux-kernel@...r.kernel.org, Wim Van Sebroeck <wim@...ux-watchdog.org>, Guenter Roeck <linux@...ck-us.net>, Rob Herring <robh+dt@...nel.org>, Thomas Bogendoerfer <tsbogend@...ha.franken.de>, bcm-kernel-feedback-list@...adcom.com, Justin Chen <justinpopo6@...il.com>, linux-watchdog@...r.kernel.org, devicetree@...r.kernel.org, linux-mips@...r.kernel.org, linux-arm-kernel@...ts.infradead.org Subject: Re: [PATCH 5/7] watchdog: bcm7038_wdt: Add platform device id for bcm63xx-wdt On 2021-10-28 19:23, Florian Fainelli wrote: > In order to phase out bcm63xx_wdt and use bcm7038_wdt instead, > introduce > a platform_device_id table that allows both names to be matched. > > Signed-off-by: Florian Fainelli <f.fainelli@...il.com> > --- > drivers/watchdog/bcm7038_wdt.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/watchdog/bcm7038_wdt.c > b/drivers/watchdog/bcm7038_wdt.c > index 506cd7ef9c77..2535f450e8a1 100644 > --- a/drivers/watchdog/bcm7038_wdt.c > +++ b/drivers/watchdog/bcm7038_wdt.c > @@ -223,6 +223,13 @@ static const struct of_device_id > bcm7038_wdt_match[] = { > }; > MODULE_DEVICE_TABLE(of, bcm7038_wdt_match); > > +static const struct platform_device_id bcm7038_wdt_devtype[] = { > + { .name = "bcm7038-wdt" }, > + { .name = "bcm63xx-wdt" }, > + { /* sentinel */ }, > +}; > +MODULE_DEVICE_TABLE(platform, bcm7038_wdt_devtype); Do we really want "bcm7038-wdt" here? I don't think it will ever be used as apparently BCM7038 uses DT. I'd also prefer to have Rob's comment on mapping blocks vs. mapping registers. If we were to map whole hardware blocks, we should have per-SoC bindings and handling registers layouts in a driver. Right now bcm63xx arch code maps selected part of hardware block that is meant to match driver's logic (offsets 0x00 and 0x04).
Powered by blists - more mailing lists