From: Fabio Estevam festevam@denx.de

When the reset_gpio GPIO is used, it is better to put the codec back into reset state when the driver unbinds.

Add a remove() function to accomplish that.

Suggested-by: Charles Keepax ckeepax@opensource.cirrus.com Signed-off-by: Fabio Estevam festevam@denx.de --- Changes since v2: - Keep using the current polarity logic.

sound/soc/codecs/cs4265.c | 11 +++++++++++ 1 file changed, 11 insertions(+)

diff --git a/sound/soc/codecs/cs4265.c b/sound/soc/codecs/cs4265.c index 294fa7ac16cb..8fa166e4b2a9 100644 --- a/sound/soc/codecs/cs4265.c +++ b/sound/soc/codecs/cs4265.c @@ -626,6 +626,16 @@ static int cs4265_i2c_probe(struct i2c_client *i2c_client, ARRAY_SIZE(cs4265_dai)); }

+static int cs4265_i2c_remove(struct i2c_client *i2c) +{ + struct cs4265_private *cs4265 = i2c_get_clientdata(i2c); + + if (cs4265->reset_gpio) + gpiod_set_value_cansleep(cs4265->reset_gpio, 0); + + return 0; +} + static const struct of_device_id cs4265_of_match[] = { { .compatible = "cirrus,cs4265", }, { } @@ -645,6 +655,7 @@ static struct i2c_driver cs4265_i2c_driver = { }, .id_table = cs4265_id, .probe = cs4265_i2c_probe, + .remove = cs4265_i2c_remove, };

module_i2c_driver(cs4265_i2c_driver);