On 12/07/2022 15:03, Conor.Dooley@xxxxxxxxxxxxx wrote:OK.
On 12/07/2022 14:53, Yang Yingliang wrote:Ahh, I just saw your revert of 59ebbe40fb51 ("spi: simplify
[Some people who received this message don't often get email from yangyingliang@xxxxxxxxxx. Learn why this is important at https://aka.ms/LearnAboutSenderIdentification ]Hey Yang,
EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe
Switch to use devm_spi_alloc_master() to simpify error path.
Thanks for trying to fix my mistakes!
Forgive my innocence here, but why is it okay to remove the
spi_master_put() in remove() but not the one in the error path of
the probe function?
If the devm_add_action_or_reset() in devm_spi_register_controller()
fails won't the same thing apply to the probe error path?
IOW, I think this patch needs a fixes tag too b/c it also fixes a
refcount underflow. Please correct me if I am misunderstanding.
devm_spi register_controller"). With that, this makes a lot more
sense.
One other comment below.This comment still applies for this patch. dev_err_probe would be
nice.
Thanks,
Conor.