Re: [PATCH] media: venus: Synchronize probe() between venus_core and enc/dec

To: Tadeusz Struk <tadeusz.struk@xxxxxxxxxx>, Stanimir Varbanov <stanimir.varbanov@xxxxxxxxxx>, Andy Gross <agross@xxxxxxxxxx>
Subject: Re: [PATCH] media: venus: Synchronize probe() between venus_core and enc/dec
From: "Bryan O'Donoghue" <bryan.odonoghue@xxxxxxxxxx>
Date: Mon, 25 Oct 2021 18:17:40 +0100
Cc: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>, Mauro Carvalho Chehab <mchehab@xxxxxxxxxx>, Lee Jones <lee.jones@xxxxxxxxxx>, Amit Pundir <amit.pundir@xxxxxxxxxx>, John Stultz <john.stultz@xxxxxxxxxx>, linux-media@xxxxxxxxxxxxxxx, linux-arm-msm@xxxxxxxxxxxxxxx, linux-kernel@xxxxxxxxxxxxxxx
In-reply-to: <25da5210-8e1f-7183-a8e7-8584f8dd2cef@linaro.org>
References: <20211025144345.267107-1-tadeusz.struk@linaro.org> <72f8dd7a-66c7-fb50-db23-f98ba753af1d@nexus-software.ie> <bba3acc1-cfa1-0c53-75de-f4ffa0a2bc9e@linaro.org> <00b817a4-f1ac-6a94-5f1e-836d8d313406@linaro.org> <25da5210-8e1f-7183-a8e7-8584f8dd2cef@linaro.org>
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.1.1

On 25/10/2021 17:57, Tadeusz Struk wrote:

agree, but I don't depend on of_platform_populate(). The orderingbetween the
three probe functions is enforced by the new sync mutex.


That explanation works for me.

Reviewed-by: Bryan O'Donoghue <bryan.odonoghue@xxxxxxxxxx>

Follow-Ups:
- Re: [PATCH] media: venus: Synchronize probe() between venus_core and enc/dec
  - From: Tadeusz Struk

References:
- [PATCH] media: venus: Synchronize probe() between venus_core and enc/dec
  - From: Tadeusz Struk
- Re: [PATCH] media: venus: Synchronize probe() between venus_core and enc/dec
  - From: Bryan O'Donoghue
- Re: [PATCH] media: venus: Synchronize probe() between venus_core and enc/dec
  - From: Tadeusz Struk
- Re: [PATCH] media: venus: Synchronize probe() between venus_core and enc/dec
  - From: Bryan O'Donoghue
- Re: [PATCH] media: venus: Synchronize probe() between venus_core and enc/dec
  - From: Tadeusz Struk