[PATCH 3/8] media: v4l2-async: simplify v4l2_async_subdev structure

Sakari Ailus sakari.ailus at iki.fi
Tue Dec 19 00:27:17 PST 2017


On Mon, Dec 18, 2017 at 05:53:57PM -0200, Mauro Carvalho Chehab wrote:
> The V4L2_ASYNC_MATCH_FWNODE match criteria requires just one
> struct to be filled (struct fwnode_handle). The V4L2_ASYNC_MATCH_DEVNAME
> match criteria requires just a device name.
> 
> So, it doesn't make sense to enclose those into structs,
> as the criteria can go directly into the union.
> 
> That makes easier to document it, as we don't need to document
> weird senseless structs.
> 
> At drivers, this makes even clearer about the match criteria.
> 
> Acked-by: Sylwester Nawrocki <s.nawrocki at samsung.com>
> Signed-off-by: Mauro Carvalho Chehab <mchehab at s-opensource.com>

I'm not sure this is needed but it doesn't break anything either.

Feel free to add:

Acked-by: Sakari Ailus <sakari.ailus at linux.intel.com>

-- 
Sakari Ailus
e-mail: sakari.ailus at iki.fi



More information about the linux-arm-kernel mailing list