lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Date:   Mon, 14 Mar 2022 12:14:54 +0100
From:   Mauro Carvalho Chehab <mchehab@...nel.org>
To:     Alain Volmat <alain.volmat@...s.st.com>
Cc:     Niklas Söderlund 
        <niklas.soderlund+renesas@...natech.se>,
        Alexandre Torgue <alexandre.torgue@...s.st.com>,
        Alexey Khoroshilov <khoroshilov@...ras.ru>,
        Dillon Min <dillon.minfei@...il.com>,
        Dmitriy Ulitin <ulitin@...ras.ru>,
        Dmitry Osipenko <digetx@...il.com>,
        Eugen Hristev <eugen.hristev@...rochip.com>,
        Hans Verkuil <hverkuil-cisco@...all.nl>,
        Hugues Fruchet <hugues.fruchet@...s.st.com>,
        Jacopo Mondi <jacopo@...ndi.org>,
        Jernej Skrabec <jernej.skrabec@...il.com>,
        Lad Prabhakar <prabhakar.mahadev-lad.rj@...renesas.com>,
        Laurent Pinchart <laurent.pinchart@...asonboard.com>,
        Leon Romanovsky <leon@...nel.org>,
        Maxime Coquelin <mcoquelin.stm32@...il.com>,
        Ming Qian <ming.qian@....com>,
        Sakari Ailus <sakari.ailus@...ux.intel.com>,
        Tomi Valkeinen <tomi.valkeinen@...asonboard.com>,
        <linux-arm-kernel@...ts.infradead.org>,
        <linux-kernel@...r.kernel.org>, <linux-media@...r.kernel.org>,
        <linux-stm32@...md-mailman.stormreply.com>
Subject: Re: [PATCH 17/24] media: platform: rename stm32/ to sti/stm32/

Hi Alain,

Em Mon, 14 Mar 2022 09:39:42 +0100
Alain Volmat <alain.volmat@...s.st.com> 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@...nel.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@...r.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@...nel.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
> >   



Thanks,
Mauro

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ