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 <shuahkh@xxxxxxxxxxxxxxx> wrote: >> On 10/06/2017 03:30 PM, Shuah Khan wrote: >>> 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
