Re: [PATCH v5 0/5] media: atmel-isc: Rework the format list and clock provider

["Yang, Wenyou" <Wenyou.Yang@microchip.com>]

Hi Hans,


On 2017/10/27 20:41, Hans Verkuil wrote:
> Hi Wenyou,
>
> Unfortunately the v4 patch series was just merged instead of v5. Can you make a new
> patch applying just the v4 -> v5 changes?
Thank you for your merging.

Of course, I will send a patch to fix it.

>
> Thanks!
>
> 	Hans
>
> On 10/27/2017 05:21 AM, Wenyou Yang wrote:
>> To improve the readability of code, rework the format list table,
>> split the format array into two. Meanwhile, fix the issue of the
>> clock provider operation and the pm runtime support.
>>
>> Changes in v5:
>>   - Fix the clock ID which enters the runtime suspend should be
>>     ISC_ISPCK, instead of ISC_MCK for clk_prepare/clk_unprepare().
>>   - Fix the clock ID to ISC_ISPCK, instead of ISC_MCK for
>>     isc_clk_is_enabled().
>>
>> Changes in v4:
>>   - Call pm_runtime_get_sync() and pm_runtime_put_sync() in ->prepare
>>     and ->unprepare callback.
>>   - Move pm_runtime_enable() call from the complete callback to the
>>     end of probe.
>>   - Call pm_runtime_get_sync() and pm_runtime_put_sync() in
>>     ->is_enabled() callbacks.
>>   - Call clk_disable_unprepare() in ->remove callback.
>>
>> Changes in v3:
>>   - Fix the wrong used spinlock.
>>   - s/_/- on the subject.
>>   - Add a new flag for Raw Bayer format to remove MAX_RAW_FMT_INDEX define.
>>   - Add the comments for define of the format flag.
>>   - Rebase media_tree/master.
>>
>> Changes in v2:
>>   - Add the new patch to remove the unnecessary member from
>>     isc_subdev_entity struct.
>>   - Rebase on the patch set,
>>          [PATCH 0/6] [media] Atmel: Adjustments for seven function implementations
>>          https://www.mail-archive.com/linux-media@vger.kernel.org/msg118342.html
>>
>> Wenyou Yang (5):
>>    media: atmel-isc: Add spin lock for clock enable ops
>>    media: atmel-isc: Add prepare and unprepare ops
>>    media: atmel-isc: Enable the clocks during probe
>>    media: atmel-isc: Remove unnecessary member
>>    media: atmel-isc: Rework the format list
>>
>>   drivers/media/platform/atmel/atmel-isc-regs.h |   1 +
>>   drivers/media/platform/atmel/atmel-isc.c      | 629 ++++++++++++++++++++------
>>   2 files changed, 498 insertions(+), 132 deletions(-)
>>
Best Regards,
Wenyou Yang