On Tue, Jul 18, 2017 at 04:27:13PM +0530, Keerthy wrote:

Use the devm version of gpiochip_add_data and pass on the
return value. Reset the static variables to 0 before returning.

You need to describe not just what you do, but also why you it. In this
case, your fixing memory leaks and the gpio chip being left registered
if this driver is unbound.

Johan