diff mbox series

[3/5] MAINTAINERS: Add entry for allegrodvt Gen 3 drivers

Message ID 20250523134207.68481-4-yassine.ouaissa@allegrodvt.com
State New
Headers show
Series media: Add Gen 3 IP stateful decoder driver | expand

Commit Message

Yassine Ouaissa May 23, 2025, 1:41 p.m. UTC
Add my self as maintainer of the allegrodvt Gen drivers

Signed-off-by: Yassine Ouaissa <yassine.ouaissa@allegrodvt.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)

Comments

Krzysztof Kozlowski May 25, 2025, 4:40 a.m. UTC | #1
On 23/05/2025 15:41, Yassine Ouaissa wrote:
> Add my self as maintainer of the allegrodvt Gen drivers
> 
> Signed-off-by: Yassine Ouaissa <yassine.ouaissa@allegrodvt.com>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index e59011a36e6b..9285bb2f43d9 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -802,6 +802,7 @@ F:	drivers/platform/x86/dell/alienware-wmi*
>  
>  ALLEGRO DVT VIDEO IP CORE DRIVER
>  M:	Michael Tretter <m.tretter@pengutronix.de>
> +M:	Yassine OUAISSA <yassine.ouaissa@allegrodvt.com>
I already disagreed. I commented on this and nothing improved. I
provided arguments that you do not know the process, thus you need to
first read the docs.

THEN you sent it again still not reading the docs and not learning
anything here from previous comments.

You sent four times the same version, ignoring review and not
understanding basic versioning concept.

You need to first understand how the process works, what is a bug
report, what is the review process, what is the reviewer's statement of
oversight. I suggest sending several patches and contributions prior
asking to be maintainer.

So still disagree, but this is now disappointing: NAK

Best regards,
Krzysztof
Nicolas Dufresne May 25, 2025, 9:50 p.m. UTC | #2
Hi Yassine,

Le vendredi 23 mai 2025 à 15:41 +0200, Yassine Ouaissa a écrit :
> Add my self as maintainer of the allegrodvt Gen drivers
> 
> Signed-off-by: Yassine Ouaissa <yassine.ouaissa@allegrodvt.com>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index e59011a36e6b..9285bb2f43d9 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -802,6 +802,7 @@ F:	drivers/platform/x86/dell/alienware-wmi*
>  
>  ALLEGRO DVT VIDEO IP CORE DRIVER
>  M:	Michael Tretter <m.tretter@pengutronix.de>
> +M:	Yassine OUAISSA <yassine.ouaissa@allegrodvt.com>
>  R:	Pengutronix Kernel Team <kernel@pengutronix.de>
>  L:	linux-media@vger.kernel.org
>  S:	Maintained

Be aware that I do not endorse Krzysztof style of communication, and this
does not reflect Linux Media values. We strongly encourage both new comers and
contributions coming from the hardware companies. Please, don't get discourage,
simply focus on the facts and the way forward. DT maintainers don't usually deal
with pre-silicon drivers, so we'll have to see what this means for
bindings. But having drivers contributed before the hardware is a clear win
for the Linux kernel, so we should all encourage this and find a way.

In general, don't assume any of the above is known and document it. Its quite possible
your reviewers so far have been thinking this driver is for existing hardware already
running in a known SoC. They cannot guess, you have to make things really clear and
transparent.

Meanwhile, a better approach to maintenance, and the one I expected initially, is to
place yourself under Michael in the hierarchy, and remove yourself from the bindings
path. Bindings should really come from the SoC vendor in practice, so perhaps we should
not provide a generic one. Hopefully we can get proper feedback from DT maintainers on
that aspect.

I'd like to see a focus move onto the driver code, which is at this stage much more
important. In parallel, spend time to re-read the guidelines for submissions and
check some automation tools. 'b4' is really my goto, and will help you avoid some
of the common mistakes.

regards,
Nicolas
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index e59011a36e6b..9285bb2f43d9 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -802,6 +802,7 @@  F:	drivers/platform/x86/dell/alienware-wmi*
 
 ALLEGRO DVT VIDEO IP CORE DRIVER
 M:	Michael Tretter <m.tretter@pengutronix.de>
+M:	Yassine OUAISSA <yassine.ouaissa@allegrodvt.com>
 R:	Pengutronix Kernel Team <kernel@pengutronix.de>
 L:	linux-media@vger.kernel.org
 S:	Maintained