Hi Alain, Em Mon, 14 Mar 2022 09:39:42 +0100 Alain Volmat <alain.volmat@xxxxxxxxxxx> escreveu: > 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 ? Yeah, a folder like "st", "stm" (New York Stock Exchange uses this name), "stmicro", or even "stmicroelectronics" with either two folders on it or two separate submenus work from my side. I would prefer to have a short name there, like "st" or "stm", but it is up to you to choose whatever works best for ST Microelectronics. Just let me know what works best and I'll change it at the patch. > > 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@xxxxxxxxxx> > > --- > > > > 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@xxxxxxxxxx/ > > > > 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@xxxxxxxxxxxxxxx > > 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@xxxxxxxxxx> > > 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 > > Thanks, Mauro
