| 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
| ||
|
Message-ID: <1122a956-0650-f948-512b-d3447e34df30@foss.st.com>
Date: Mon, 14 Mar 2022 12:34:47 +0100
From: Hugues FRUCHET - FOSS <hugues.fruchet@...s.st.com>
To: Mauro Carvalho Chehab <mchehab@...nel.org>,
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>,
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 Mauro,
Ideally we would like to stick to sti/ and stm32/ at same level, as it
is done in various other locations:
./Documentation/arm/sti/
./Documentation/arm/stm32/
./arch/arm/mach-sti
./arch/arm/mach-stm32
./drivers/media/cec/platform/sti/
./drivers/media/cec/platform/stm32/
./drivers/gpu/drm/sti/
./drivers/gpu/drm/stm/
./sound/soc/sti/
./sound/soc/stm/
Is this possible to keep those media folders as is ?
Best regards,
Hugues.
On 3/14/22 12:14 PM, Mauro Carvalho Chehab wrote:
> 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