On 15.12.2023 19:06, Jeffrey Hugo wrote: > On 12/8/2023 9:34 AM, Jeffrey Hugo wrote: >> From: Pranjal Ramajor Asha Kanojiya <quic_pkanojiy@xxxxxxxxxxx> >> >> Offload the balancing of init and destroy calls to DRM managed APIs. >> mutex destroy for ->cntl_mutex is not called during device release and >> destroy workqueue is not called in error path of create_qdev(). So, use >> DRM managed APIs to manage the release of resources and avoid such >> problems. >> >> Signed-off-by: Pranjal Ramajor Asha Kanojiya <quic_pkanojiy@xxxxxxxxxxx> >> Reviewed-by: Jeffrey Hugo <quic_jhugo@xxxxxxxxxxx> >> Signed-off-by: Jeffrey Hugo <quic_jhugo@xxxxxxxxxxx> > > Jacek, I saw a review from you for 1-5, and 7 but not this patch (6). Did I miss something? Sorry, I was out of office for a couple days and I wasn't able to finish the review. Reviewed-by: Jacek Lawrynowicz <jacek.lawrynowicz@xxxxxxxxxxxxxxx>