i2c: imx: init bus recovery info before adding i2c adapter
authorGao Pan <b54642@freescale.com>
Wed, 9 Dec 2015 03:08:22 +0000 (11:08 +0800)
committerWolfram Sang <wsa@the-dreams.de>
Sun, 3 Jan 2016 18:06:50 +0000 (19:06 +0100)
commita5f650182bcf956b2b0341575e8bff01e402224f
tree379b9a84ff590a2b7113b606bf2cbc359086143e
parent7ee24eb508d61e5b74dcc80f644adfb5916f7580
i2c: imx: init bus recovery info before adding i2c adapter

During driver probe, i2c_imx_init_recovery_info() must come before
i2c_add_numbered_adapter(), because the get/set_scl() functions
are assigned in i2c_register_adapter() under the conditon that bus
recover_info are initialized. Otherwise, get/set_scl() function
pointers never get assigned.

In such case, when i2c_generic_gpio_recovery() is used for bus recovery,
there will be kernel crash because bri->set_scl is NULL.

The solution to this bug is moving i2c_imx_init_recovery_info() before
i2c_register_adapter().

Signed-off-by: Gao Pan <b54642@freescale.com>
Signed-off-by: Fugang Duan <B38611@freescale.com>
Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Cc: stable@kernel.org
drivers/i2c/busses/i2c-imx.c