Messages in this thread | | | Date | Sun, 20 Mar 2022 00:55:06 +0100 | From | Wolfram Sang <> | Subject | Re: [PATCH] i2c: mux: demux-pinctrl: do not deactivate a master that is not active |
| |
On Wed, Mar 02, 2022 at 12:22:35PM +0100, Peter Rosin wrote: > Attempting to rollback the activation of the current master when > the current master has not been activated is bad. priv->cur_chan > and priv->cur_adap are both still zeroed out and the rollback > may result in attempts to revert an of changeset that has not been > applied and do result in calls to both del and put the zeroed out > i2c_adapter. Maybe it crashes, or whatever, but it's bad in any > case. > > Fixes: e9d1a0a41d44 ("i2c: mux: demux-pinctrl: Fix an error handling path in 'i2c_demux_pinctrl_probe()'") > Cc: Christophe JAILLET <christophe.jaillet@wanadoo.fr> > Signed-off-by: Peter Rosin <peda@axentia.se>
Applied to for-next, thanks!
[unhandled content-type:application/pgp-signature] | |