Re: [PATCH v2 0/8] media: use 'time_left' instead of 'timeout' with wait_*() functions

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 07/08/2024 15:08, Hans Verkuil wrote:
> Hi Wolfram,
> 
> On 05/08/2024 23:51, Wolfram Sang wrote:
>> Changes since v1:
>> * fixed another occasion in the allegro driver (Thanks, Michael)
>> * added tags (Thanks Ismael and Thierry)
>> * rebased to 6.11-rc1
> 
> Can you resend this series? This patch series wasn't picked up by our patchwork,
> probably due to a full filesystem.

Actually, it's better to wait a bit: I now see that patchwork hasn't accepted new
patches since August 5th, so until that is fixed, there is no point in resending...

I'll let you know when it is OK again.

> Apologies for the inconvenience.

Even more apologies,

	Hans

> 
> Regards,
> 
> 	Hans
> 
>>
>> There is a confusing pattern in the kernel to use a variable named 'timeout' to
>> store the result of wait_*() functions causing patterns like:
>>
>>         timeout = wait_for_completion_timeout(...)
>>         if (!timeout) return -ETIMEDOUT;
>>
>> with all kinds of permutations. Use 'time_left' as a variable to make the code
>> obvious and self explaining. Also correct the type of the variable if
>> the original code got it wrong.
>>
>> This is part of a tree-wide series. The rest of the patches can be found here:
>>
>> git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/time_left
>>
>> Because these patches are generated, I audit them before sending. This is why I
>> will send series step by step. Build bot is happy with these patches, though.
>> No functional changes intended.
>>
>>
>> Wolfram Sang (8):
>>   media: allegro: use 'time_left' variable with
>>     wait_for_completion_timeout()
>>   media: atmel-isi: use 'time_left' variable with
>>     wait_for_completion_timeout()
>>   media: bdisp: use 'time_left' variable with wait_event_timeout()
>>   media: fimc-is: use 'time_left' variable with wait_event_timeout()
>>   media: platform: exynos-gsc: use 'time_left' variable with
>>     wait_event_timeout()
>>   media: solo6x10: use 'time_left' variable with
>>     wait_for_completion_timeout()
>>   media: tegra-vde: use 'time_left' variable with
>>     wait_for_completion_interruptible_timeout()
>>   media: ti: cal: use 'time_left' variable with wait_event_timeout()
>>
>>  drivers/media/pci/solo6x10/solo6x10-p2m.c     |  8 +++----
>>  .../media/platform/allegro-dvt/allegro-core.c | 24 +++++++++----------
>>  drivers/media/platform/atmel/atmel-isi.c      |  8 +++----
>>  .../media/platform/nvidia/tegra-vde/h264.c    | 10 ++++----
>>  .../platform/samsung/exynos-gsc/gsc-core.c    | 10 ++++----
>>  .../platform/samsung/exynos4-is/fimc-core.c   | 10 ++++----
>>  .../media/platform/st/sti/bdisp/bdisp-v4l2.c  | 10 ++++----
>>  drivers/media/platform/ti/cal/cal.c           |  8 +++----
>>  8 files changed, 44 insertions(+), 44 deletions(-)
>>
> 
> 





[Index of Archives]     [ARM Kernel]     [Linux ARM]     [Linux ARM MSM]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux