On 12/20/2023 12:02 AM, Jacek Lawrynowicz wrote:
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>
No problem. I thought maybe I had an issue on my end. I appreciate the
reviews. Hopefully your out of office was enjoyable.
Happy Holidays.
-Jeff
