All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@kernel.org>
To: Hugues FRUCHET - FOSS <hugues.fruchet@foss.st.com>
Cc: "Alain Volmat" <alain.volmat@foss.st.com>,
	"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>,
	"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 14:10:08 +0100	[thread overview]
Message-ID: <20220314141008.13e32acd@coco.lan> (raw)
In-Reply-To: <1122a956-0650-f948-512b-d3447e34df30@foss.st.com>

Em Mon, 14 Mar 2022 12:34:47 +0100
Hugues FRUCHET - FOSS <hugues.fruchet@foss.st.com> escreveu:

> 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 ?

If we add them under:
	./drivers/media/platform/stm/sti/
	./drivers/media/platform/stm/stm32/

They'll both be at the same level. 

It has been increasingly hard to maintain media/platform, as the file
become too big and disorganized. Sorting it per-vendor seems to be the
best thing, as it will keep things better organized. 

As a side effect, with such kind of change, it is now possible to 
have a MAINTAINERS entry that would point to a mailing list and/or 
some SoC maintainers/reviewers that should be c/c to all patches 
affecting drivers/media/platform/<vendor>.

Thanks,
Mauro

WARNING: multiple messages have this Message-ID (diff)
From: Mauro Carvalho Chehab <mchehab@kernel.org>
To: Hugues FRUCHET - FOSS <hugues.fruchet@foss.st.com>
Cc: "Alain Volmat" <alain.volmat@foss.st.com>,
	"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>,
	"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 14:10:08 +0100	[thread overview]
Message-ID: <20220314141008.13e32acd@coco.lan> (raw)
In-Reply-To: <1122a956-0650-f948-512b-d3447e34df30@foss.st.com>

Em Mon, 14 Mar 2022 12:34:47 +0100
Hugues FRUCHET - FOSS <hugues.fruchet@foss.st.com> escreveu:

> 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 ?

If we add them under:
	./drivers/media/platform/stm/sti/
	./drivers/media/platform/stm/stm32/

They'll both be at the same level. 

It has been increasingly hard to maintain media/platform, as the file
become too big and disorganized. Sorting it per-vendor seems to be the
best thing, as it will keep things better organized. 

As a side effect, with such kind of change, it is now possible to 
have a MAINTAINERS entry that would point to a mailing list and/or 
some SoC maintainers/reviewers that should be c/c to all patches 
affecting drivers/media/platform/<vendor>.

Thanks,
Mauro

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2022-03-14 13:10 UTC|newest]

Thread overview: 96+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-03-13 10:51 [PATCH 00/24] Organize media platform drivers per manufacturer Mauro Carvalho Chehab
2022-03-13 10:51 ` Mauro Carvalho Chehab
2022-03-13 10:51 ` Mauro Carvalho Chehab
2022-03-13 10:51 ` Mauro Carvalho Chehab
2022-03-13 10:51 ` Mauro Carvalho Chehab
2022-03-13 10:51 ` Mauro Carvalho Chehab
2022-03-13 10:51 ` Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 01/24] media: platform: rename coda/ to chips-media/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 02/24] media: platform: rename marvell-ccic/ to marvell/ Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 03/24] media: platform: rename meson/ge2d/ to amlogic/meson-ge2d/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 04/24] media: platform: rename mtk-jpeg/ to mediatek/mtk-jpeg/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 05/24] media: platform: rename mtk-mdp/ to mediatek/mtk-mdp/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 06/24] media: platform: rename mtk-vcodec/ to mediatek/mtk-vcodec/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-14 13:09   ` Nicolas Dufresne
2022-03-14 13:09     ` Nicolas Dufresne
2022-03-14 13:09     ` Nicolas Dufresne
2022-03-14 14:37     ` Mauro Carvalho Chehab
2022-03-14 14:37       ` Mauro Carvalho Chehab
2022-03-14 14:37       ` Mauro Carvalho Chehab
2022-03-14 15:02       ` AngeloGioacchino Del Regno
2022-03-14 15:02         ` AngeloGioacchino Del Regno
2022-03-14 15:02         ` AngeloGioacchino Del Regno
2022-03-14 15:42         ` Mauro Carvalho Chehab
2022-03-14 15:42           ` Mauro Carvalho Chehab
2022-03-14 15:42           ` Mauro Carvalho Chehab
2022-03-14 15:43           ` AngeloGioacchino Del Regno
2022-03-14 15:43             ` AngeloGioacchino Del Regno
2022-03-14 15:43             ` AngeloGioacchino Del Regno
2022-03-13 10:51 ` [PATCH 07/24] media: platform: rename mtk-vpu/ to mediatek/mtk-vpu/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 08/24] media: platform: rename sunxi/ to allwinner/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 09/24] media: platform: rename tegra/vde/ to nvidia/tegra-vde/ Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 10/24] media: platform: rename amphion/ to nxp/amphion/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-15 18:26   ` Nicolas Dufresne
2022-03-15 18:26     ` Nicolas Dufresne
2022-03-13 10:51 ` [PATCH 11/24] media: platform: rename exynos4-is/ to samsung/exynos4-is/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 12/24] media: platform: rename exynos-gsc/ to samsung/exynos-gsc/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 13/24] media: platform: rename s3c-camif/ to samsung/s3c-camif/ Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 14/24] media: platform: rename s5p-g2d/ to samsung/s5p-g2d/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 15/24] media: platform: rename s5p-jpeg/ to samsung/s5p-jpeg/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-14  8:54   ` Andrzej Pietrasiewicz
2022-03-14  8:54     ` Andrzej Pietrasiewicz
2022-03-13 10:51 ` [PATCH 16/24] media: platform: rename s5p-mfc/ to samsung/s5p-mfc/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 17/24] media: platform: rename stm32/ to sti/stm32/ Mauro Carvalho Chehab
2022-03-13 10:51   ` Mauro Carvalho Chehab
2022-03-14  8:39   ` Alain Volmat
2022-03-14  8:39     ` Alain Volmat
2022-03-14 11:14     ` Mauro Carvalho Chehab
2022-03-14 11:14       ` Mauro Carvalho Chehab
2022-03-14 11:34       ` Hugues FRUCHET - FOSS
2022-03-14 11:34         ` Hugues FRUCHET - FOSS
2022-03-14 13:10         ` Mauro Carvalho Chehab [this message]
2022-03-14 13:10           ` Mauro Carvalho Chehab
2022-03-15  9:02           ` Hugues FRUCHET - FOSS
2022-03-15  9:02             ` Hugues FRUCHET - FOSS
2022-03-16 13:28             ` Mauro Carvalho Chehab
2022-03-16 13:28               ` Mauro Carvalho Chehab
2022-03-13 10:51 ` [PATCH 18/24] media: platform: rename am437x/ to ti/am437x/ Mauro Carvalho Chehab
2022-03-13 10:52 ` [PATCH 19/24] media: platform: rename davinci/ to ti/davinci/ Mauro Carvalho Chehab
2022-03-13 10:52 ` [PATCH 20/24] media: platform: rename omap3isp/ to ti/omap3isp/ Mauro Carvalho Chehab
2022-03-13 10:52 ` [PATCH 21/24] media: platform: rename omap/ to ti/omap/ Mauro Carvalho Chehab
2022-03-13 10:52 ` [PATCH 22/24] media: platform: rename ti-vpe/ to ti/vpe/ Mauro Carvalho Chehab
2022-03-13 10:52 ` [PATCH 23/24] media: platform: Create vendor/{Makefile,Kconfig} files Mauro Carvalho Chehab
2022-03-13 10:52   ` [PATCH 23/24] media: platform: Create vendor/{Makefile, Kconfig} files Mauro Carvalho Chehab
2022-03-13 10:52   ` Mauro Carvalho Chehab
2022-03-13 10:52   ` Mauro Carvalho Chehab
2022-03-13 10:52 ` [PATCH 24/24] media: platform/*/Kconfig: make menus more uniform Mauro Carvalho Chehab
2022-03-13 10:52   ` Mauro Carvalho Chehab
2022-03-13 10:52   ` Mauro Carvalho Chehab
2022-03-13 10:52   ` Mauro Carvalho Chehab
2022-03-13 10:52   ` Mauro Carvalho Chehab
2022-03-13 11:00 ` [PATCH 00/24] Organize media platform drivers per manufacturer Mauro Carvalho Chehab
2022-03-13 11:00   ` Mauro Carvalho Chehab
2022-03-13 11:00   ` Mauro Carvalho Chehab
2022-03-13 11:00   ` Mauro Carvalho Chehab
2022-03-13 11:00   ` Mauro Carvalho Chehab
2022-03-13 11:00   ` Mauro Carvalho Chehab
2022-03-13 11:00   ` Mauro Carvalho Chehab

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20220314141008.13e32acd@coco.lan \
    --to=mchehab@kernel.org \
    --cc=alain.volmat@foss.st.com \
    --cc=alexandre.torgue@foss.st.com \
    --cc=digetx@gmail.com \
    --cc=dillon.minfei@gmail.com \
    --cc=eugen.hristev@microchip.com \
    --cc=hugues.fruchet@foss.st.com \
    --cc=hverkuil-cisco@xs4all.nl \
    --cc=jacopo@jmondi.org \
    --cc=jernej.skrabec@gmail.com \
    --cc=khoroshilov@ispras.ru \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=leon@kernel.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-stm32@st-md-mailman.stormreply.com \
    --cc=mcoquelin.stm32@gmail.com \
    --cc=ming.qian@nxp.com \
    --cc=niklas.soderlund+renesas@ragnatech.se \
    --cc=prabhakar.mahadev-lad.rj@bp.renesas.com \
    --cc=sakari.ailus@linux.intel.com \
    --cc=tomi.valkeinen@ideasonboard.com \
    --cc=ulitin@ispras.ru \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.