Re: [PATCH v2.2 5/5] omap3isp: Quit using struct v4l2_subdev.host_priv field

2017-08-16 Thread Sakari Ailus
On Wed, Aug 16, 2017 at 08:24:19PM +0300, Laurent Pinchart wrote: > Hi Sakari, > > Thank you for the patch. > > On Wednesday 16 Aug 2017 20:05:52 Sakari Ailus wrote: > > struct v4l2_subdev.host_priv is intended to be used by another driver. This > > is hardly good design but back in the days of p

Re: [PATCH v2.2 5/5] omap3isp: Quit using struct v4l2_subdev.host_priv field

2017-08-16 Thread Laurent Pinchart
Hi Sakari, Thank you for the patch. On Wednesday 16 Aug 2017 20:05:52 Sakari Ailus wrote: > struct v4l2_subdev.host_priv is intended to be used by another driver. This > is hardly good design but back in the days of platform data was a quick > hack to get things done. > > As the sub-device speci

[PATCH v2.2 5/5] omap3isp: Quit using struct v4l2_subdev.host_priv field

2017-08-16 Thread Sakari Ailus
struct v4l2_subdev.host_priv is intended to be used by another driver. This is hardly good design but back in the days of platform data was a quick hack to get things done. As the sub-device specific bus information can be stored to the ISP driver specific struct allocated along with v4l2_async_su