Re: [PATCH v4] media: add a subsystem profile documentation
From: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Date: 2021-03-01 13:48:57
Also in:
linux-media, lkml
From: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Date: 2021-03-01 13:48:57
Also in:
linux-media, lkml
Em Mon, 1 Mar 2021 13:27:39 +0100 Hans Verkuil [off-list ref] escreveu:
Hi Mauro,
Thanks for your review. I'm addressing the points on a v5. Yet, there's one that, IMHO, we should elaborate more, if we add it to the doc:
quoted
+Provided that your patch is at https://patchwork.linuxtv.org, it should +be sooner or later handled, so you don't need to re-submit a patch. + +Except for bug fixes, we don't usually add new patches to the development +tree between -rc6 and the next -rc1. + +Please notice that the media subsystem is a high traffic one, so it +could take a while for us to be able to review your patches. Feel free +to ping if you don't get a feedback in a couple of weeks or to ask +other developers to publicly add Reviewed-by and, more importantly, +Tested-by tags.What I have noticed is that sometimes important bug fixes are missed due to the high traffic volume. I would like to see something along the lines of: "If important bug fixes are not reviewed or picked up within a week of posting, then do not hesitate to ping."
I see your point, but "important" is relative ;-) I mean, a bug is almost always important for the one reporting it. It doesn't necessarily means that such bug is visible by others or if the subsystem's core has a serious bug. IMO, if we add a paragraph like that, we should better explain what "important" means. Thanks, Mauro