can: flexcan: use of_property_read_u32 to get DT entry value
authorHui Wang <jason77.wang@gmail.com>
Thu, 28 Jun 2012 08:21:34 +0000 (16:21 +0800)
committerMarc Kleine-Budde <mkl@pengutronix.de>
Mon, 2 Jul 2012 09:08:37 +0000 (11:08 +0200)
of_property_read_u32() can auto handle endian problems, use this
function can make code clean and simple.

No need to check return value here since the following got value
check will handle this.

Cc: linux-can@vger.kernel.org
Cc: Marc Kleine-Budde <mkl@pengutronix.de>
Cc: Wolfgang Grandegger <wg@grandegger.com>
Cc: Shawn Guo <shawn.guo@linaro.org>
Signed-off-by: Hui Wang <jason77.wang@gmail.com>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
drivers/net/can/flexcan.c

index 0f88cd3..b429b3f 100644 (file)
@@ -938,14 +938,9 @@ static int __devinit flexcan_probe(struct platform_device *pdev)
        if (IS_ERR(pinctrl))
                return PTR_ERR(pinctrl);
 
-       if (pdev->dev.of_node) {
-               const __be32 *clock_freq_p;
-
-               clock_freq_p = of_get_property(pdev->dev.of_node,
-                                               "clock-frequency", NULL);
-               if (clock_freq_p)
-                       clock_freq = be32_to_cpup(clock_freq_p);
-       }
+       if (pdev->dev.of_node)
+               of_property_read_u32(pdev->dev.of_node,
+                                               "clock-frequency", &clock_freq);
 
        if (!clock_freq) {
                clk = clk_get(&pdev->dev, NULL);