On 08/12/2015 01:15 AM, Uwe Kleine-König wrote:
This function is new in v4.2-rc1 and makes a forward declaration of the
match table superfluous which can so be removed.

Signed-off-by: Uwe Kleine-König <[email protected]>

Reviewed-by: Guenter Roeck <[email protected]>

---
  drivers/watchdog/mpc8xxx_wdt.c | 6 ++----
  1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/drivers/watchdog/mpc8xxx_wdt.c b/drivers/watchdog/mpc8xxx_wdt.c
index e0acda3282d9..fb1fe967cf57 100644
--- a/drivers/watchdog/mpc8xxx_wdt.c
+++ b/drivers/watchdog/mpc8xxx_wdt.c
@@ -139,7 +139,6 @@ static struct watchdog_device mpc8xxx_wdt_dev = {
        .ops = &mpc8xxx_wdt_ops,
  };

-static const struct of_device_id mpc8xxx_wdt_match[];
  static int mpc8xxx_wdt_probe(struct platform_device *ofdev)
  {
        int ret;
@@ -150,10 +149,9 @@ static int mpc8xxx_wdt_probe(struct platform_device *ofdev)
        bool enabled;
        unsigned int timeout_sec;

-       match = of_match_device(mpc8xxx_wdt_match, &ofdev->dev);
-       if (!match)
+       wdt_type = of_device_get_match_data(&ofdev->dev);
+       if (!wdt_type)
                return -EINVAL;
-       wdt_type = match->data;

        if (!freq || freq == -1)
                return -EINVAL;


--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to