From: Krzysztof Kozlowski <krzk@kernel.org>
To: j0sh1x <aljoshua.hell@gmail.com>, tglx@kernel.org
Cc: linux-kernel@vger.kernel.org, subsystem@vger.kernel.org,
dominikkobinski314@gmail.com
Subject: Re: [PATCH 2/5] irqchip: Add Qualcomm MSM VIC driver
Date: Mon, 16 Mar 2026 09:30:31 +0100 [thread overview]
Message-ID: <bb196674-b1ad-4f75-b64e-35ce23284dee@kernel.org> (raw)
In-Reply-To: <20260315111705.118544-2-aljoshua.hell@gmail.com>
On 15/03/2026 12:17, j0sh1x wrote:
> Signed-off-by: j0sh1x <aljoshua.hell@gmail.com>
You must use full name.
> ---
> drivers/irqchip/irq-msm-vic.c | 361 ++++++++++++++++++++++++++++++++++
> 1 file changed, 361 insertions(+)
> create mode 100644 drivers/irqchip/irq-msm-vic.c
>
> diff --git a/drivers/irqchip/irq-msm-vic.c b/drivers/irqchip/irq-msm-vic.c
> new file mode 100644
> index 000000000000..7e7f7be3ad6d
> --- /dev/null
> +++ b/drivers/irqchip/irq-msm-vic.c
> @@ -0,0 +1,361 @@
> +// SPDX-License-Identifier: GPL-2.0 OR MIT
> +/*
> + * Copyright (C) 2007 Google, Inc.
> + * Copyright (c) 2009, Code Aurora Forum. All rights reserved.
> + * Copyright (c) 2024, Htc Leo Revival Project
> + *
> + * This software is licensed under the terms of the GNU General Public
> + * License version 2, as published by the Free Software Foundation, and
> + * may be copied, distributed, and modified under those terms.
> + *
> + * This program is distributed in the hope that it will be useful,
> + * but WITHOUT ANY WARRANTY; without even the implied warranty of
> + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
> + * GNU General Public License for more details.
We don't take 19 years old code. You need to fix it to match current
Linux kernel style, which means most likely all this has to be
completely rewritten from scratch. Take latest reviewed drivers as
starting point, not something from 2007.
Best regards,
Krzysztof
next prev parent reply other threads:[~2026-03-16 8:30 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-03-15 11:17 [PATCH 1/5] dt-bindings: irq: Add Qualcomm MSM VIC binding j0sh1x
2026-03-15 11:17 ` [PATCH 2/5] irqchip: Add Qualcomm MSM VIC driver j0sh1x
2026-03-16 8:30 ` Krzysztof Kozlowski [this message]
2026-03-20 9:51 ` Thomas Gleixner
2026-03-15 11:17 ` [PATCH 3/5] irqchip: Add Kconfig and Makefile entries for MSM VIC j0sh1x
2026-03-30 4:45 ` kernel test robot
2026-03-30 9:04 ` kernel test robot
2026-03-15 11:17 ` [PATCH 4/5] dt-bindings: irq: Add Qualcomm MSM VIC header j0sh1x
2026-03-16 8:30 ` Krzysztof Kozlowski
2026-03-15 11:17 ` [PATCH 5/5] MAINTAINERS: Add Maintainer entry for MSM VIC j0sh1x
2026-03-16 8:29 ` [PATCH 1/5] dt-bindings: irq: Add Qualcomm MSM VIC binding Krzysztof Kozlowski
2026-03-16 17:56 ` Krzysztof Kozlowski
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=bb196674-b1ad-4f75-b64e-35ce23284dee@kernel.org \
--to=krzk@kernel.org \
--cc=aljoshua.hell@gmail.com \
--cc=dominikkobinski314@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=subsystem@vger.kernel.org \
--cc=tglx@kernel.org \
/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.