ARM: OMAP: I2C: Fix omap_register_i2c_bus() return value on success
authorLaurent Pinchart <laurent.pinchart@ideasonboard.com>
Wed, 2 Nov 2011 11:54:03 +0000 (12:54 +0100)
committerTony Lindgren <tony@atomide.com>
Sat, 5 Nov 2011 00:41:07 +0000 (17:41 -0700)
Commit 4d17aeb1c5b2375769446d13012a98e6d265ec13 ("OMAP: I2C: split
device registration and convert OMAP2+ to omap_device") makes
omap2_i2c_add_bus() return a pointer to an omap_device instead on
success instead of 0.

This breaks the omap_register_i2c_bus() ABI and results in the igep0020
board code detecting an I2C bus registration error when there is none.

Fix the problem by using PTR_RET() instead of PTR_ERR() in
omap2_i2c_add_bus().

Reported-by: Alexander Kinzer <a.kinzer@plusoptix.de>
Signed-off-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
[tony@atomide.com: updated to return pdev instead of od]
Signed-off-by: Tony Lindgren <tony@atomide.com>
arch/arm/plat-omap/i2c.c

index 679cbd49c01945633b81b7c68cf25b0c38dfdc56..db071bc71c4d61d8e5bd5938eb3e438a06f2b449 100644 (file)
@@ -184,7 +184,7 @@ static inline int omap2_i2c_add_bus(int bus_id)
                        NULL, 0, 0);
        WARN(IS_ERR(pdev), "Could not build omap_device for %s\n", name);
 
-       return PTR_ERR(pdev);
+       return PTR_RET(pdev);
 }
 #else
 static inline int omap2_i2c_add_bus(int bus_id)