Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Mon, 14 Mar 2022 12:05:05 +0200
From:   Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To:     Pratyush Yadav <p.yadav@ti.com>
Cc:     Mauro Carvalho Chehab <mchehab@kernel.org>,
        Niklas =?utf-8?Q?S=C3=B6derlund?= 
        <niklas.soderlund+renesas@ragnatech.se>,
        Benoit Parrot <bparrot@ti.com>,
        Dmitry Osipenko <digetx@gmail.com>,
        Eugen Hristev <eugen.hristev@microchip.com>,
        Hans Verkuil <hverkuil-cisco@xs4all.nl>,
        Jacopo Mondi <jacopo@jmondi.org>,
        Jernej Skrabec <jernej.skrabec@gmail.com>,
        Ming Qian <ming.qian@nxp.com>,
        Sakari Ailus <sakari.ailus@linux.intel.com>,
        Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>,
        Yang Yingliang <yangyingliang@huawei.com>,
        Zhou Qingyang <zhou1615@umn.edu>, linux-kernel@vger.kernel.org,
        linux-media@vger.kernel.org
Subject: Re: [PATCH 61/64] media: platform: rename ti-vpe/ to ti/vpe/
Message-ID: <Yi8TUYsHj2Iv4zPy@pendragon.ideasonboard.com>
References: <cover.1647242578.git.mchehab@kernel.org>
 <fecdb3d1e4fffbb5c3e0003a7d403ef9669ca026.1647242579.git.mchehab@kernel.org>
 <20220314095350.mbvjs7alqoz4vvml@ti.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
In-Reply-To: <20220314095350.mbvjs7alqoz4vvml@ti.com>
Precedence: bulk

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@ti.com>
> 
> [0] https://patchwork.linuxtv.org/project/linux-media/patch/20220121142904.4091481-11-p.yadav@ti.com/
> 
> > 
> > Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
> > Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>

-- 
Regards,

Laurent Pinchart