[lvc-project] [PATCH v2 0/2] media: dvb-usb-v2: add init_ready flag to prevent race during device initialization

[Fedor Pchelkin]

On Mon, 10. Nov 07:38, Sergey Nalivayko wrote:
> 
> >On Fri, 07. Nov 21:16, Nalivayko Sergey wrote:
> >> This patch series introduces an atomic init_ready flag in struct 
> >> dvb_usb_device and adds checks in relevant code paths.
> >> Drivers now verify that device initialization has completed before 
> >> performing operations that rely on initialized device state.
> >> If the device is not yet ready, the operation is deferred until 
> >> initialization finishes, preventing race conditions and improving 
> >> driver robustness.
> 
> >По какой причине направлена v2? Что поменялось и в связи с чем?
> 
> Пришла отбивка от меинтейнеров DVB, что патч не проходит по описанию и нет doc для поля структуры. 

По doc согласен, но насчёт описания  не кажется ли, что новый трейс менее
полезный и содержательный, чем то что было в первой версии?  Трейсы стоит
вставлять as-is, из них только можно убирать ненужную информацию в виде
содержимого регистров и прочего.  Обрезать строки и прочее *не надо*.
Это ругнулись не мэйнтейнеры, а checkpatch.pl с их CI-системы.  Замечания
checkpatch.pl не всегда корректны, особенно в части длинных строк описания
патча.

В любом случае, изменения между версиями стоит описывать в истории
изменений версий патча в зоне после ---.  Таким образом и мы, и
мэйнтенеры не будем гадать, что же произошло, и почему была выслана
v2...

> 
> >>
> >> Nalivayko Sergey (2):
> >>   media: dvb-usb-v2: add device init_ready flag
> >>   media: mxl111sf: fix i2c race condition during device  probe
> >>
> >>  drivers/media/usb/dvb-usb-v2/dvb_usb.h      | 4 ++++
> >>  drivers/media/usb/dvb-usb-v2/dvb_usb_core.c | 4 ++++  
> >> drivers/media/usb/dvb-usb-v2/mxl111sf-i2c.c | 2 +-
> >>  3 files changed, 9 insertions(+), 1 deletion(-)
> >>
> >> --
> >> 2.39.5