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:   Tue, 15 Mar 2022 06:40:05 +0100
From:   Mauro Carvalho Chehab <mchehab@kernel.org>
To:     Jernej =?UTF-8?B?xaBrcmFiZWM=?= <jernej.skrabec@gmail.com>
Cc:     Niklas =?UTF-8?B?U8O2ZGVybHVuZA==?= 
        <niklas.soderlund+renesas@ragnatech.se>,
        Cai Huoqing <caihuoqing@baidu.com>,
        Chen-Yu Tsai <wens@csie.org>,
        Dinghao Liu <dinghao.liu@zju.edu.cn>,
        Dmitry Osipenko <digetx@gmail.com>,
        Eugen Hristev <eugen.hristev@microchip.com>,
        Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>,
        Hans Verkuil <hverkuil-cisco@xs4all.nl>,
        Jacopo Mondi <jacopo@jmondi.org>,
        Jia-Ju Bai <baijiaju1990@gmail.com>,
        Jonathan Cameron <Jonathan.Cameron@huawei.com>,
        Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
        Maxime Ripard <mripard@kernel.org>,
        Mikhail Rudenko <mike.rudenko@gmail.com>,
        Ming Qian <ming.qian@nxp.com>,
        Ondrej Jirman <megous@megous.com>,
        Robert Foss <robert.foss@linaro.org>,
        Sakari Ailus <sakari.ailus@linux.intel.com>,
        Tomi Valkeinen <tomi.valkeinen@ideasonboard.com>,
        Yong Deng <yong.deng@magewell.com>,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
        linux-media@vger.kernel.org, linux-sunxi@lists.linux.dev
Subject: Re: [PATCH v2 47/67] media: platform: rename sunxi/ to allwinner/
Message-ID: <20220315064005.10ecdab2@coco.lan>
In-Reply-To: <2816975.e9J7NaK4W3@kista>
References: <cover.1647274406.git.mchehab@kernel.org>
        <85266b480902079391d4206b8aa276ff131a730f.1647274407.git.mchehab@kernel.org>
        <2816975.e9J7NaK4W3@kista>
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Em Mon, 14 Mar 2022 18:22:20 +0100
Jernej =C5=A0krabec <jernej.skrabec@gmail.com> escreveu:

> Dne ponedeljek, 14. marec 2022 ob 17:34:42 CET je Mauro Carvalho Chehab=20
> napisal(a):
> > As the end goal is to have platform drivers split by vendor,
> > rename sunxi/ to allwinner/.
> >=20
> > Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org> =20
>=20
> I would rather not do that. Everything related to Allwinner is called sun=
xi,=20
> albeit there are a few outliers. This is similar to Amlogic/meson situati=
on.

The rationale of having one directory per manufacturer is that, if drivers
for newer platforms with different names from the same manufacturers are=20
added, those will still fit under allwinner/ and amlogic/.

The Kconfig names for sunxi and meson didn't change, nor the driver's name.=
=20
Also, the directories under allwinner preserve sun<x>i name on them:

	drivers/media/platform/allwinner/
	=E2=94=9C=E2=94=80=E2=94=80 sun4i-csi
	=E2=94=9C=E2=94=80=E2=94=80 sun6i-csi
	=E2=94=9C=E2=94=80=E2=94=80 sun8i-di
	=E2=94=94=E2=94=80=E2=94=80 sun8i-rotate

and so the directory under amlogic/:

	drivers/media/platform/amlogic/
	=E2=94=94=E2=94=80=E2=94=80 meson-ge2d

Now, if Allinner decides to release a new platforms named after another sta=
r,
let's say, "Vega" and "Rigel", it would be just a matter of adding=20
"vega/" and "rigel/" directories under allwinner. No need to touch
media/platform/Kconfig and media/platform/Makefile. Everything will happen
on much more smaller vendor-specific Kconfig/Makefile.

See for instance, TI. We had initially a single driver, and everywhere it
was called "omap", which was had gained different versions (omap2, omap3,=20
...). Then, TI decided to use other names for newer IP (davinci, am437x),=20
and now the current development is at cal and vpe.

If we had placed a ti/ directory by the time omap were added (and the
same for the other vendors), we would have avoided the need of this
reorg.

Thanks,
Mauro