[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Yi8TUYsHj2Iv4zPy@pendragon.ideasonboard.com>
Date: Mon, 14 Mar 2022 12:05:05 +0200
From: Laurent Pinchart <laurent.pinchart@...asonboard.com>
To: Pratyush Yadav <p.yadav@...com>
Cc: Mauro Carvalho Chehab <mchehab@...nel.org>,
Niklas Söderlund
<niklas.soderlund+renesas@...natech.se>,
Benoit Parrot <bparrot@...com>,
Dmitry Osipenko <digetx@...il.com>,
Eugen Hristev <eugen.hristev@...rochip.com>,
Hans Verkuil <hverkuil-cisco@...all.nl>,
Jacopo Mondi <jacopo@...ndi.org>,
Jernej Skrabec <jernej.skrabec@...il.com>,
Ming Qian <ming.qian@....com>,
Sakari Ailus <sakari.ailus@...ux.intel.com>,
Tomi Valkeinen <tomi.valkeinen@...asonboard.com>,
Yang Yingliang <yangyingliang@...wei.com>,
Zhou Qingyang <zhou1615@....edu>, linux-kernel@...r.kernel.org,
linux-media@...r.kernel.org
Subject: Re: [PATCH 61/64] media: platform: rename ti-vpe/ to ti/vpe/
Hi Pratyush,
On Mon, Mar 14, 2022 at 03:23:50PM +0530, Pratyush Yadav wrote:
> On 14/03/22 08:55AM, Mauro Carvalho Chehab wrote:
> > As the end goal is to have platform drivers split by vendor,
> > rename ti-vpe/ to ti/vpe/.
>
> I submitted a patch to do something similar [0]. I think it is a better
> one because there are actually 2 separate IP drivers here: CAL and VPE.
> My patch puts them both in their own directories, and a later patch
> introduces a 3rd driver, j721e-csi2rx.
>
> So I would like to see my patch's directory structure being used.
I like your directory structure. Mauro, could you incorporate [0] in
your series ?
> Nacked-by: Pratyush Yadav <p.yadav@...com>
>
> [0] https://patchwork.linuxtv.org/project/linux-media/patch/20220121142904.4091481-11-p.yadav@ti.com/
>
> >
> > Reviewed-by: Laurent Pinchart <laurent.pinchart@...asonboard.com>
> > Signed-off-by: Mauro Carvalho Chehab <mchehab@...nel.org>
--
Regards,
Laurent Pinchart
Powered by blists - more mailing lists