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

2017-12-20 Thread Lad, Prabhakar
Hi Mauro,

Thanks for the patch.

On Tue, Dec 19, 2017 at 11:18 AM, Mauro Carvalho Chehab
<mche...@s-opensource.com> 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.nawro...@samsung.com>
> Acked-by: Benoit Parrot <bpar...@ti.com>
> Acked-by: Alexandre Belloni <alexandre.bell...@free-electrons.com>
> Acked-by: Sakari Ailus <sakari.ai...@linux.intel.com>
> Acked-by: Philipp Zabel <p.za...@pengutronix.de>
> Signed-off-by: Mauro Carvalho Chehab <mche...@s-opensource.com>
> ---
>  drivers/media/platform/am437x/am437x-vpfe.c|  6 +++---

For above:

Acked-by: Lad, Prabhakar <prabhakar.cse...@gmail.com>

Cheers,
--Prabhakar Lad


Re: [PATCH] media: Convert to using %pOF instead of full_name

2017-07-20 Thread Lad, Prabhakar
On Tue, Jul 18, 2017 at 10:43 PM, Rob Herring <r...@kernel.org> wrote:
> Now that we have a custom printf format specifier, convert users of
> full_name to use %pOF instead. This is preparation to remove storing
> of the full path string for each node.
>
> Signed-off-by: Rob Herring <r...@kernel.org>
> Cc: Kyungmin Park <kyungmin.p...@samsung.com>
> Cc: Andrzej Hajda <a.ha...@samsung.com>
> Cc: Mauro Carvalho Chehab <mche...@kernel.org>
> Cc: "Lad, Prabhakar" <prabhakar.cse...@gmail.com>
> Cc: Songjun Wu <songjun...@microchip.com>
> Cc: Sylwester Nawrocki <s.nawro...@samsung.com>
> Cc: Kukjin Kim <kg...@kernel.org>
> Cc: Krzysztof Kozlowski <k...@kernel.org>
> Cc: Javier Martinez Canillas <jav...@osg.samsung.com>
> Cc: Minghsiu Tsai <minghsiu.t...@mediatek.com>
> Cc: Houlong Wei <houlong@mediatek.com>
> Cc: Andrew-CT Chen <andrew-ct.c...@mediatek.com>
> Cc: Matthias Brugger <matthias@gmail.com>
> Cc: Laurent Pinchart <laurent.pinch...@ideasonboard.com>
> Cc: "Niklas Söderlund" <niklas.soderl...@ragnatech.se>
> Cc: Guennadi Liakhovetski <g.liakhovet...@gmx.de>
> Cc: Hyun Kwon <hyun.k...@xilinx.com>
> Cc: Michal Simek <michal.si...@xilinx.com>
> Cc: "Sören Brinkmann" <soren.brinkm...@xilinx.com>
> Cc: linux-me...@vger.kernel.org
> Cc: linux-arm-ker...@lists.infradead.org
> Cc: linux-samsung-...@vger.kernel.org
> Cc: linux-media...@lists.infradead.org
> Cc: linux-renesas-soc@vger.kernel.org
> ---
>  drivers/media/platform/am437x/am437x-vpfe.c|  4 +-
>  drivers/media/platform/davinci/vpif_capture.c  | 16 

For above:

Acked-by: Lad, Prabhakar <prabhakar.cse...@gmail.com>

Cheers,
--Prabhakar Lad