Re: [PATCH 17/24] media: platform: rename stm32/ to sti/stm32/

From: Alain Volmat <alain.volmat@foss.st.com>
To: Mauro Carvalho Chehab <mchehab@kernel.org>
Cc: "Niklas Söderlund" <niklas.soderlund+renesas@ragnatech.se>,
	"Alexandre Torgue" <alexandre.torgue@foss.st.com>,
	"Alexey Khoroshilov" <khoroshilov@ispras.ru>,
	"Dillon Min" <dillon.minfei@gmail.com>,
	"Dmitriy Ulitin" <ulitin@ispras.ru>,
	"Dmitry Osipenko" <digetx@gmail.com>,
	"Eugen Hristev" <eugen.hristev@microchip.com>,
	"Hans Verkuil" <hverkuil-cisco@xs4all.nl>,
	"Hugues Fruchet" <hugues.fruchet@foss.st.com>,
	"Jacopo Mondi" <jacopo@jmondi.org>,
	"Jernej Skrabec" <jernej.skrabec@gmail.com>,
	"Lad Prabhakar" <prabhakar.mahadev-lad.rj@bp.renesas.com>,
	"Laurent Pinchart" <laurent.pinchart@ideasonboard.com>,
	"Leon Romanovsky" <leon@kernel.org>,
	"Maxime Coquelin" <mcoquelin.stm32@gmail.com>,
	"Ming Qian" <ming.qian@nxp.com>,
	"Sakari Ailus" <sakari.ailus@linux.intel.com>,
	"Tomi Valkeinen" <tomi.valkeinen@ideasonboard.com>,
	linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org, linux-media@vger.kernel.org,
	linux-stm32@st-md-mailman.stormreply.com
Subject: Re: [PATCH 17/24] media: platform: rename stm32/ to sti/stm32/
Date: Mon, 14 Mar 2022 09:39:42 +0100	[thread overview]
Message-ID: <20220314083942.GA526468@gnbcxd0016.gnb.st.com> (raw)
In-Reply-To: <dc5be62a56ac19c6f49f4c8432558fd7b0efe7e6.1647167750.git.mchehab@kernel.org>

Hi Mauro,

I don't think stm32 should be put within the sti platform folder.
sti and stm32 are 2 different platforms from ST Microelectronics.
STi refers to the platform with SoCs such as STiH407/STiH410 and STiH418
while stm32 are all STM32 ones. Those two platforms aren't related.
What about having a folder stmicro or stmicroelectronics (too long
probably :D) with the 2 folders sti and stm32 into it ?

Alain

On Sun, Mar 13, 2022 at 11:51:58AM +0100, Mauro Carvalho Chehab wrote:
> As the end goal is to have platform drivers split by vendor,
> rename stm32/ to sti/stm32/.
> 
> Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
> ---
> 
> To avoid mailbombing on a large number of people, only mailing lists were C/C on the cover.
> See [PATCH 00/24] at: https://lore.kernel.org/all/cover.1647167750.git.mchehab@kernel.org/
> 
>  MAINTAINERS                                               | 2 +-
>  drivers/media/platform/Kconfig                            | 2 +-
>  drivers/media/platform/Makefile                           | 2 +-
>  drivers/media/platform/{ => sti}/stm32/Kconfig            | 0
>  drivers/media/platform/{ => sti}/stm32/Makefile           | 0
>  drivers/media/platform/{ => sti}/stm32/dma2d/dma2d-hw.c   | 0
>  drivers/media/platform/{ => sti}/stm32/dma2d/dma2d-regs.h | 0
>  drivers/media/platform/{ => sti}/stm32/dma2d/dma2d.c      | 0
>  drivers/media/platform/{ => sti}/stm32/dma2d/dma2d.h      | 0
>  drivers/media/platform/{ => sti}/stm32/stm32-dcmi.c       | 0
>  10 files changed, 3 insertions(+), 3 deletions(-)
>  rename drivers/media/platform/{ => sti}/stm32/Kconfig (100%)
>  rename drivers/media/platform/{ => sti}/stm32/Makefile (100%)
>  rename drivers/media/platform/{ => sti}/stm32/dma2d/dma2d-hw.c (100%)
>  rename drivers/media/platform/{ => sti}/stm32/dma2d/dma2d-regs.h (100%)
>  rename drivers/media/platform/{ => sti}/stm32/dma2d/dma2d.c (100%)
>  rename drivers/media/platform/{ => sti}/stm32/dma2d/dma2d.h (100%)
>  rename drivers/media/platform/{ => sti}/stm32/stm32-dcmi.c (100%)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index 7711a5ea125e..620705e0f043 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -12046,7 +12046,7 @@ L:	linux-media@vger.kernel.org
>  S:	Supported
>  T:	git git://linuxtv.org/media_tree.git
>  F:	Documentation/devicetree/bindings/media/st,stm32-dcmi.yaml
> -F:	drivers/media/platform/stm32/stm32-dcmi.c
> +F:	drivers/media/platform/sti/stm32/stm32-dcmi.c
>  
>  MEDIA INPUT INFRASTRUCTURE (V4L/DVB)
>  M:	Mauro Carvalho Chehab <mchehab@kernel.org>
> diff --git a/drivers/media/platform/Kconfig b/drivers/media/platform/Kconfig
> index c3594807f8d7..cf373bfbca1b 100644
> --- a/drivers/media/platform/Kconfig
> +++ b/drivers/media/platform/Kconfig
> @@ -95,7 +95,7 @@ source "drivers/media/platform/samsung/s5p-g2d/Kconfig"
>  source "drivers/media/platform/samsung/s5p-jpeg/Kconfig"
>  source "drivers/media/platform/samsung/s5p-mfc/Kconfig"
>  source "drivers/media/platform/sti/Kconfig"
> -source "drivers/media/platform/stm32/Kconfig"
> +source "drivers/media/platform/sti/stm32/Kconfig"
>  source "drivers/media/platform/ti-vpe/Kconfig"
>  source "drivers/media/platform/via/Kconfig"
>  source "drivers/media/platform/xilinx/Kconfig"
> diff --git a/drivers/media/platform/Makefile b/drivers/media/platform/Makefile
> index 6a766acfbe37..e3dd2331003a 100644
> --- a/drivers/media/platform/Makefile
> +++ b/drivers/media/platform/Makefile
> @@ -40,7 +40,7 @@ obj-y += sti/bdisp/
>  obj-y += sti/c8sectpfe/
>  obj-y += sti/delta/
>  obj-y += sti/hva/
> -obj-y += stm32/
> +obj-y += sti/stm32/
>  obj-y += ti-vpe/
>  obj-y += via/
>  obj-y += xilinx/
> diff --git a/drivers/media/platform/stm32/Kconfig b/drivers/media/platform/sti/stm32/Kconfig
> similarity index 100%
> rename from drivers/media/platform/stm32/Kconfig
> rename to drivers/media/platform/sti/stm32/Kconfig
> diff --git a/drivers/media/platform/stm32/Makefile b/drivers/media/platform/sti/stm32/Makefile
> similarity index 100%
> rename from drivers/media/platform/stm32/Makefile
> rename to drivers/media/platform/sti/stm32/Makefile
> diff --git a/drivers/media/platform/stm32/dma2d/dma2d-hw.c b/drivers/media/platform/sti/stm32/dma2d/dma2d-hw.c
> similarity index 100%
> rename from drivers/media/platform/stm32/dma2d/dma2d-hw.c
> rename to drivers/media/platform/sti/stm32/dma2d/dma2d-hw.c
> diff --git a/drivers/media/platform/stm32/dma2d/dma2d-regs.h b/drivers/media/platform/sti/stm32/dma2d/dma2d-regs.h
> similarity index 100%
> rename from drivers/media/platform/stm32/dma2d/dma2d-regs.h
> rename to drivers/media/platform/sti/stm32/dma2d/dma2d-regs.h
> diff --git a/drivers/media/platform/stm32/dma2d/dma2d.c b/drivers/media/platform/sti/stm32/dma2d/dma2d.c
> similarity index 100%
> rename from drivers/media/platform/stm32/dma2d/dma2d.c
> rename to drivers/media/platform/sti/stm32/dma2d/dma2d.c
> diff --git a/drivers/media/platform/stm32/dma2d/dma2d.h b/drivers/media/platform/sti/stm32/dma2d/dma2d.h
> similarity index 100%
> rename from drivers/media/platform/stm32/dma2d/dma2d.h
> rename to drivers/media/platform/sti/stm32/dma2d/dma2d.h
> diff --git a/drivers/media/platform/stm32/stm32-dcmi.c b/drivers/media/platform/sti/stm32/stm32-dcmi.c
> similarity index 100%
> rename from drivers/media/platform/stm32/stm32-dcmi.c
> rename to drivers/media/platform/sti/stm32/stm32-dcmi.c
> -- 
> 2.35.1
> 