serial: mxs-auart: Remove serial_mxs_probe_dt()
authorFabio Estevam <festevam@gmail.com>
Mon, 18 Jan 2021 15:21:53 +0000 (12:21 -0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 20 Jan 2021 17:48:19 +0000 (18:48 +0100)
The mxs platform is devicetree-only, so there is no need to check
whether it was instantiated via devicetree.

Simplify the code my removing serial_mxs_probe_dt() and add its
content into the main probe function.

Signed-off-by: Fabio Estevam <festevam@gmail.com>
Link: https://lore.kernel.org/r/20210118152154.1644569-1-festevam@gmail.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/mxs-auart.c

index 8ecf622..01fedb4 100644 (file)
@@ -1535,34 +1535,6 @@ disable_clk_ahb:
        return err;
 }
 
-/*
- * This function returns 1 if pdev isn't a device instatiated by dt, 0 if it
- * could successfully get all information from dt or a negative errno.
- */
-static int serial_mxs_probe_dt(struct mxs_auart_port *s,
-               struct platform_device *pdev)
-{
-       struct device_node *np = pdev->dev.of_node;
-       int ret;
-
-       if (!np)
-               /* no device tree device */
-               return 1;
-
-       ret = of_alias_get_id(np, "serial");
-       if (ret < 0) {
-               dev_err(&pdev->dev, "failed to get alias id: %d\n", ret);
-               return ret;
-       }
-       s->port.line = ret;
-
-       if (of_get_property(np, "uart-has-rtscts", NULL) ||
-           of_get_property(np, "fsl,uart-has-rtscts", NULL) /* deprecated */)
-               set_bit(MXS_AUART_RTSCTS, &s->flags);
-
-       return 0;
-}
-
 static int mxs_auart_init_gpios(struct mxs_auart_port *s, struct device *dev)
 {
        enum mctrl_gpio_idx i;
@@ -1631,6 +1603,7 @@ static int mxs_auart_request_gpio_irq(struct mxs_auart_port *s)
 
 static int mxs_auart_probe(struct platform_device *pdev)
 {
+       struct device_node *np = pdev->dev.of_node;
        struct mxs_auart_port *s;
        u32 version;
        int ret, irq;
@@ -1643,11 +1616,17 @@ static int mxs_auart_probe(struct platform_device *pdev)
        s->port.dev = &pdev->dev;
        s->dev = &pdev->dev;
 
-       ret = serial_mxs_probe_dt(s, pdev);
-       if (ret > 0)
-               s->port.line = pdev->id < 0 ? 0 : pdev->id;
-       else if (ret < 0)
+       ret = of_alias_get_id(np, "serial");
+       if (ret < 0) {
+               dev_err(&pdev->dev, "failed to get alias id: %d\n", ret);
                return ret;
+       }
+       s->port.line = ret;
+
+       if (of_get_property(np, "uart-has-rtscts", NULL) ||
+           of_get_property(np, "fsl,uart-has-rtscts", NULL) /* deprecated */)
+               set_bit(MXS_AUART_RTSCTS, &s->flags);
+
        if (s->port.line >= ARRAY_SIZE(auart_port)) {
                dev_err(&pdev->dev, "serial%d out of range\n", s->port.line);
                return -EINVAL;