On 10/25/2017 04:28 PM, Marian Mihailescu wrote:

Hi Shuah,

For MFC patch, you can delete the "dev" variable since it's not being
used anymore and results in a compile warning.

- struct s5p_mfc_dev *dev = ctx->dev;

Hi Marian,

This series doesn't have the unused warn problem. I fixed the problem
in media: s5p-mfc: fix lockdep warning patch that has the warning and
sent v2.

On Thu, Oct 26, 2017 at 7:54 AM, Shuah Khan [off-list ref] wrote:

quoted

On 10/06/2017 03:30 PM, Shuah Khan wrote:

quoted

This patch series fixes inefficiencies and lock contention in the request
firmware paths.

Shuah Khan (2):
  media: s5p-mfc: check for firmware allocation before requesting
    firmware
  media: s5p-mfc: fix lock confection - request_firmware() once and keep
    state

 drivers/media/platform/s5p-mfc/s5p_mfc.c        |  4 ++++
 drivers/media/platform/s5p-mfc/s5p_mfc_common.h |  3 +++
 drivers/media/platform/s5p-mfc/s5p_mfc_ctrl.c   | 15 ++++++++++-----
 3 files changed, 17 insertions(+), 5 deletions(-)

Any feedback on this series?

Still waiting for feedback on this series.

thanks,
-- Shuah