Thread (36 messages) 36 messages, 3 authors, 2021-01-26

Re: [PATCH v2 01/14] media: v4l2-async: Clean v4l2_async_notifier_add_fwnode_remote_subdev

From: Sakari Ailus <sakari.ailus@linux.intel.com>
Date: 2021-01-25 11:46:42

Hi Helen,

Thanks for the comments.

On Tue, Jan 19, 2021 at 03:51:32PM -0300, Helen Koike wrote:
Hi Ezequiel,

Thanks for you patch, I just have minor comments.

I think there are more improvements that could be done to
reduce some identation levels, but I see the idea here is
to change drivers the least as possible, so I won't comment
on that.
I discussed with Ezequiel that I can take the set forwards from here.

I'd be inclined to avoiding making larger than necessary changes here in
drivers that won't end up being tested this time around. The set already
does a lot, such changes could be done later on of course.

-- 
Kind regards,

Sakari Ailus
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help