Hi, On 3/9/23 14:00, Sakari Ailus wrote: > On Thu, Mar 09, 2023 at 02:57:52PM +0200, Tomi Valkeinen wrote: >> On 09/03/2023 14:56, Sakari Ailus wrote: >>> On Thu, Mar 09, 2023 at 01:43:50PM +0100, Hans de Goede wrote: >>>> Hi Sakari, >>>> >>>> On 3/9/23 13:27, Sakari Ailus wrote: >>>>> Link validation currently accesses invalid pointers if the link passed to it >>>>> is not between two sub-devices. This is of course a driver bug. >>>>> >>>>> Ignore the error but print a debug message, as this is how it used to work >>>>> previously. >>>>> >>>>> Fixes: a6b995ed03ff ("media: subdev: use streams in v4l2_subdev_link_validate()") >>>>> Reported-by: Hans de Goede <hdegoede@xxxxxxxxxx> >>>>> Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> >>>>> Reported-and-tested-by: Hans de Goede <hdegoede@xxxxxxxxxx> >>>> >>>> FWIW you have my Reported-by in there twice now ... >>> >>> Ah, thanks. I'll drop the first tag. >> >> $ git grep Reported-and-tested-by Documentation/ >> Documentation/process/maintainer-tip.rst:Please do not use combined tags, e.g. ``Reported-and-tested-by``, as > > Uh-oh. Well, I'll just split that. Thanks. This is new to me too, but good to know. I'll try to get rid of the habbit of using the combined tags then. Regards, Hans