> … In order to avoid leak spi_contriller_put must > be called in case of failure for devm_add_action_or_reset. How does this wording fit to the diff display that you would like to add the function call “spi_master_put(master)” in one if branch? > Fixes: 8b797490b4db ("spi: gpio: Make sure spi_master_put() is called in every error path") Is there a need to complete the corresponding exception handling at any more source code places? Regards, Markus