watchdog: i6300esb: stop printing kernel addresses
authorMatteo Croce <mcroce@redhat.com>
Mon, 18 Mar 2019 01:19:15 +0000 (02:19 +0100)
committerWim Van Sebroeck <wim@linux-watchdog.org>
Sun, 5 May 2019 19:02:09 +0000 (21:02 +0200)
Since commit ad67b74d2469d9b8 ("printk: hash addresses printed with %p"),
i6300esb prints "____ptrval____" instead of actual addresses:

    i6300ESB timer 0000:00:03.0: initialized (0x(____ptrval____)). heartbeat=30 sec (nowayout=1)

Instead of changing the print to "%px", and leaking kernel addresses,
just remove the print completely, cfr. e.g. commit 071929dbdd865f77
("arm64: Stop printing the virtual memory layout").

Signed-off-by: Matteo Croce <mcroce@redhat.com>
Reviewed-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Guenter Roeck <linux@roeck-us.net>
Signed-off-by: Wim Van Sebroeck <wim@linux-watchdog.org>
Documentation/devicetree/bindings/watchdog/mtk-wdt.txt
drivers/watchdog/i6300esb.c

index 8682d6a93e5b7ef74e358a8e118e4dcceab1f254..fd380eb28df59f725d34fa8d816c3f866c882edb 100644 (file)
@@ -9,6 +9,7 @@ Required properties:
        "mediatek,mt7622-wdt", "mediatek,mt6589-wdt": for MT7622
        "mediatek,mt7623-wdt", "mediatek,mt6589-wdt": for MT7623
        "mediatek,mt7629-wdt", "mediatek,mt6589-wdt": for MT7629
+       "mediatek,mt8516-wdt", "mediatek,mt6589-wdt": for MT8516
 
 - reg : Specifies base physical address and size of the registers.
 
index 950c71a8bb22fdfe5d6902df618ab5de6f8e8c1e..e312a2aeecad3bf20c957c4a40730798cd8749bc 100644 (file)
@@ -328,8 +328,8 @@ static int esb_probe(struct pci_dev *pdev,
                goto err_unmap;
        }
        dev_info(&pdev->dev,
-               "initialized (0x%p). heartbeat=%d sec (nowayout=%d)\n",
-               edev->base, edev->wdd.timeout, nowayout);
+               "initialized. heartbeat=%d sec (nowayout=%d)\n",
+               edev->wdd.timeout, nowayout);
        return 0;
 
 err_unmap: