From: Daniel Lezcano <daniel.lezcano@linaro.org>
To: afzal mohammed <afzal.mohd.ma@gmail.com>,
linux-kernel@vger.kernel.org,
linux-rpi-kernel@lists.infradead.org,
linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org,
linux-amlogic@lists.infradead.org
Cc: "Thomas Gleixner" <tglx@linutronix.de>,
"Florian Fainelli" <f.fainelli@gmail.com>,
"Ray Jui" <rjui@broadcom.com>,
"Scott Branden" <sbranden@broadcom.com>,
bcm-kernel-feedback-list@broadcom.com,
"Nicolas Saenz Julienne" <nsaenzjulienne@suse.de>,
"Kukjin Kim" <kgene@kernel.org>,
"Krzysztof Kozlowski" <krzk@kernel.org>,
"Shawn Guo" <shawnguo@kernel.org>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"Fabio Estevam" <festevam@gmail.com>,
"NXP Linux Team" <linux-imx@nxp.com>,
"Barry Song" <baohua@kernel.org>,
"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
"Kevin Hilman" <khilman@baylibre.com>,
"Linus Walleij" <linus.walleij@linaro.org>,
"Tony Prisk" <linux@prisktech.co.nz>,
"Allison Randal" <allison@lohutok.net>,
"Enrico Weigelt" <info@metux.net>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Kate Stewart" <kstewart@linuxfoundation.org>
Subject: Re: [PATCH 16/18] clocksource: Replace setup_irq() by request_irq()
Date: Thu, 20 Feb 2020 12:13:41 +0100 [thread overview]
Message-ID: <7ebf8cbc-0186-c8ea-fc0d-979c762b526b@linaro.org> (raw)
In-Reply-To: <109d17402bc75ed186a2e151dfda1edf05463b5a.1581478324.git.afzal.mohd.ma@gmail.com>
On 12/02/2020 09:05, afzal mohammed wrote:
> request_irq() is preferred over setup_irq(). Existing callers of
> setup_irq() reached mostly via 'init_IRQ()' & 'time_init()', while
> memory allocators are ready by 'mm_init()'.
>
> Per tglx[1], setup_irq() existed in olden days when allocators were not
> ready by the time early interrupts were initialized.
>
> Hence replace setup_irq() by request_irq().
>
> Seldom remove_irq() usage has been observed coupled with setup_irq(),
> wherever that has been found, it too has been replaced by free_irq().
>
> [1] https://lkml.kernel.org/r/alpine.DEB.2.20.1710191609480.1971@nanos
>
> Signed-off-by: afzal mohammed <afzal.mohd.ma@gmail.com>
Nice, thanks for this cleanup
Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org>
[ ... ]
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
WARNING: multiple messages have this Message-ID (diff)
From: Daniel Lezcano <daniel.lezcano@linaro.org>
To: afzal mohammed <afzal.mohd.ma@gmail.com>,
linux-kernel@vger.kernel.org,
linux-rpi-kernel@lists.infradead.org,
linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org,
linux-amlogic@lists.infradead.org
Cc: "Kate Stewart" <kstewart@linuxfoundation.org>,
"Linus Walleij" <linus.walleij@linaro.org>,
"Shawn Guo" <shawnguo@kernel.org>,
"Fabio Estevam" <festevam@gmail.com>,
"Florian Fainelli" <f.fainelli@gmail.com>,
"Kevin Hilman" <khilman@baylibre.com>,
"Krzysztof Kozlowski" <krzk@kernel.org>,
"Kukjin Kim" <kgene@kernel.org>,
bcm-kernel-feedback-list@broadcom.com,
"NXP Linux Team" <linux-imx@nxp.com>,
"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
"Ray Jui" <rjui@broadcom.com>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"Thomas Gleixner" <tglx@linutronix.de>,
"Allison Randal" <allison@lohutok.net>,
"Barry Song" <baohua@kernel.org>,
"Scott Branden" <sbranden@broadcom.com>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Tony Prisk" <linux@prisktech.co.nz>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"Enrico Weigelt" <info@metux.net>,
"Nicolas Saenz Julienne" <nsaenzjulienne@suse.de>
Subject: Re: [PATCH 16/18] clocksource: Replace setup_irq() by request_irq()
Date: Thu, 20 Feb 2020 12:13:41 +0100 [thread overview]
Message-ID: <7ebf8cbc-0186-c8ea-fc0d-979c762b526b@linaro.org> (raw)
In-Reply-To: <109d17402bc75ed186a2e151dfda1edf05463b5a.1581478324.git.afzal.mohd.ma@gmail.com>
On 12/02/2020 09:05, afzal mohammed wrote:
> request_irq() is preferred over setup_irq(). Existing callers of
> setup_irq() reached mostly via 'init_IRQ()' & 'time_init()', while
> memory allocators are ready by 'mm_init()'.
>
> Per tglx[1], setup_irq() existed in olden days when allocators were not
> ready by the time early interrupts were initialized.
>
> Hence replace setup_irq() by request_irq().
>
> Seldom remove_irq() usage has been observed coupled with setup_irq(),
> wherever that has been found, it too has been replaced by free_irq().
>
> [1] https://lkml.kernel.org/r/alpine.DEB.2.20.1710191609480.1971@nanos
>
> Signed-off-by: afzal mohammed <afzal.mohd.ma@gmail.com>
Nice, thanks for this cleanup
Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org>
[ ... ]
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
WARNING: multiple messages have this Message-ID (diff)
From: Daniel Lezcano <daniel.lezcano@linaro.org>
To: afzal mohammed <afzal.mohd.ma@gmail.com>,
linux-kernel@vger.kernel.org,
linux-rpi-kernel@lists.infradead.org,
linux-arm-kernel@lists.infradead.org,
linux-samsung-soc@vger.kernel.org,
linux-amlogic@lists.infradead.org
Cc: "Kate Stewart" <kstewart@linuxfoundation.org>,
"Linus Walleij" <linus.walleij@linaro.org>,
"Shawn Guo" <shawnguo@kernel.org>,
"Fabio Estevam" <festevam@gmail.com>,
"Florian Fainelli" <f.fainelli@gmail.com>,
"Kevin Hilman" <khilman@baylibre.com>,
"Krzysztof Kozlowski" <krzk@kernel.org>,
"Kukjin Kim" <kgene@kernel.org>,
bcm-kernel-feedback-list@broadcom.com,
"NXP Linux Team" <linux-imx@nxp.com>,
"Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>,
"Ray Jui" <rjui@broadcom.com>,
"Sascha Hauer" <s.hauer@pengutronix.de>,
"Thomas Gleixner" <tglx@linutronix.de>,
"Allison Randal" <allison@lohutok.net>,
"Barry Song" <baohua@kernel.org>,
"Scott Branden" <sbranden@broadcom.com>,
"Greg Kroah-Hartman" <gregkh@linuxfoundation.org>,
"Tony Prisk" <linux@prisktech.co.nz>,
"Pengutronix Kernel Team" <kernel@pengutronix.de>,
"Enrico Weigelt" <info@metux.net>,
"Nicolas Saenz Julienne" <nsaenzjulienne@suse.de>
Subject: Re: [PATCH 16/18] clocksource: Replace setup_irq() by request_irq()
Date: Thu, 20 Feb 2020 12:13:41 +0100 [thread overview]
Message-ID: <7ebf8cbc-0186-c8ea-fc0d-979c762b526b@linaro.org> (raw)
In-Reply-To: <109d17402bc75ed186a2e151dfda1edf05463b5a.1581478324.git.afzal.mohd.ma@gmail.com>
On 12/02/2020 09:05, afzal mohammed wrote:
> request_irq() is preferred over setup_irq(). Existing callers of
> setup_irq() reached mostly via 'init_IRQ()' & 'time_init()', while
> memory allocators are ready by 'mm_init()'.
>
> Per tglx[1], setup_irq() existed in olden days when allocators were not
> ready by the time early interrupts were initialized.
>
> Hence replace setup_irq() by request_irq().
>
> Seldom remove_irq() usage has been observed coupled with setup_irq(),
> wherever that has been found, it too has been replaced by free_irq().
>
> [1] https://lkml.kernel.org/r/alpine.DEB.2.20.1710191609480.1971@nanos
>
> Signed-off-by: afzal mohammed <afzal.mohd.ma@gmail.com>
Nice, thanks for this cleanup
Acked-by: Daniel Lezcano <daniel.lezcano@linaro.org>
[ ... ]
--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs
Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog
_______________________________________________
linux-amlogic mailing list
linux-amlogic@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-amlogic
next prev parent reply other threads:[~2020-02-20 11:13 UTC|newest]
Thread overview: 93+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-02-12 8:01 [PATCH 00/18] genirq: Remove setup_irq() afzal mohammed
2020-02-12 8:13 ` afzal mohammed
2020-02-12 8:01 ` afzal mohammed
2020-02-12 8:01 ` afzal mohammed
2020-02-12 8:02 ` [PATCH 01/18] alpha: replace setup_irq() by request_irq() afzal mohammed
2020-02-12 8:02 ` [PATCH 02/18] ARM: " afzal mohammed
2020-02-12 8:02 ` afzal mohammed
2020-02-12 8:10 ` Viresh Kumar
2020-02-12 8:10 ` Viresh Kumar
2020-02-12 8:10 ` Viresh Kumar
2020-02-12 23:27 ` Alexander Sverdlin
2020-02-12 23:27 ` Alexander Sverdlin
2020-02-12 23:27 ` Alexander Sverdlin
2020-02-12 8:02 ` [PATCH 03/18] c6x: " afzal mohammed
2020-02-13 17:37 ` Mark Salter
2020-02-12 8:03 ` [PATCH 04/18] hexagon: " afzal mohammed
2020-02-12 8:03 ` [PATCH 05/18] ia64: " afzal mohammed
2020-02-12 8:15 ` afzal mohammed
2020-02-12 8:03 ` [PATCH 06/18] m68k: Replace " afzal mohammed
2020-02-12 22:25 ` Finn Thain
2020-02-13 2:03 ` afzal mohammed
2020-02-13 7:11 ` Greg Ungerer
2020-02-14 13:07 ` afzal mohammed
2020-02-12 8:03 ` [PATCH 07/18] microblaze: " afzal mohammed
2020-02-12 8:04 ` [PATCH 08/18] MIPS: " afzal mohammed
2020-02-12 8:04 ` afzal mohammed
2020-02-12 8:04 ` [PATCH 09/18] parisc: " afzal mohammed
2020-02-12 8:04 ` [PATCH 10/18] powerpc: " afzal mohammed
2020-02-12 8:04 ` afzal mohammed
2020-02-13 10:59 ` Christophe Leroy
2020-02-13 10:59 ` Christophe Leroy
2020-02-12 8:04 ` [PATCH 11/18] s390: replace " afzal mohammed
2020-02-12 8:04 ` [PATCH 12/18] sh: " afzal mohammed
2020-02-12 8:16 ` afzal mohammed
2020-02-12 8:05 ` [PATCH 13/18] unicore32: " afzal mohammed
2020-02-12 8:05 ` [PATCH 14/18] x86: Replace " afzal mohammed
2020-02-27 10:49 ` Thomas Gleixner
2020-02-27 11:36 ` afzal mohammed
2020-02-27 13:29 ` Thomas Gleixner
2020-02-27 14:26 ` afzal mohammed
2020-02-12 8:05 ` [PATCH 15/18] xtensa: replace " afzal mohammed
2020-02-12 9:10 ` Max Filippov
2020-02-12 8:05 ` [PATCH 16/18] clocksource: Replace " afzal mohammed
2020-02-12 8:05 ` afzal mohammed
2020-02-12 8:05 ` afzal mohammed
2020-02-20 11:13 ` Daniel Lezcano [this message]
2020-02-20 11:13 ` Daniel Lezcano
2020-02-20 11:13 ` Daniel Lezcano
2020-02-20 14:48 ` Linus Walleij
2020-02-20 14:48 ` Linus Walleij
2020-02-20 14:48 ` Linus Walleij
2020-02-12 8:05 ` [PATCH 17/18] irqchip: " afzal mohammed
[not found] ` <8e00874d072f32496c2d0da05423bda1cadd6975.1581478324.git.afzal.mohd.ma@gmail. com>
2020-02-13 16:02 ` Paul Cercueil
2020-02-12 8:06 ` [PATCH 18/18] genirq: Remove setup_irq() and remove_irq() afzal mohammed
2020-02-20 14:49 ` Linus Walleij
2020-02-27 10:31 ` [PATCH 00/18] genirq: Remove setup_irq() Thomas Gleixner
2020-02-27 10:31 ` Thomas Gleixner
2020-02-27 10:31 ` Thomas Gleixner
2020-02-27 10:31 ` Thomas Gleixner
2020-02-27 11:07 ` afzal mohammed
2020-02-27 11:19 ` afzal mohammed
2020-02-27 11:07 ` afzal mohammed
2020-02-27 11:07 ` afzal mohammed
2020-03-21 17:43 ` afzal mohammed
2020-03-21 17:55 ` afzal mohammed
2020-03-21 17:43 ` afzal mohammed
2020-03-21 17:43 ` afzal mohammed
2020-03-27 16:08 ` [PATCH 0/6] Kill setup_irq() afzal mohammed
2020-03-27 16:20 ` afzal mohammed
2020-03-27 16:08 ` afzal mohammed
2020-03-27 16:08 ` afzal mohammed
2020-03-27 16:09 ` [PATCH v5 1/6] alpha: Replace setup_irq() by request_irq() afzal mohammed
2020-03-27 16:09 ` [PATCH v5 2/6] c6x: replace " afzal mohammed
2020-03-27 16:09 ` [PATCH v5 3/6] hexagon: " afzal mohammed
2020-03-27 16:10 ` [PATCH v5 4/6] sh: " afzal mohammed
2020-03-27 16:22 ` afzal mohammed
2020-03-27 16:10 ` [PATCH v5 5/6] unicore32: " afzal mohammed
2020-03-27 16:11 ` [PATCH v5 6/6] genirq: Remove setup_irq() and remove_irq() afzal mohammed
2020-04-11 14:10 ` afzal mohammed
2020-03-28 2:48 ` [PATCH 0/6] Kill setup_irq() Brian Cain
2020-03-28 2:48 ` Brian Cain
2020-03-28 2:48 ` Brian Cain
2020-03-28 2:48 ` Brian Cain
2020-03-28 2:48 ` Brian Cain
2020-03-28 7:32 ` afzal mohammed
2020-03-28 7:44 ` afzal mohammed
2020-03-28 7:32 ` afzal mohammed
2020-03-28 7:32 ` afzal mohammed
2020-04-02 15:03 ` Brian Cain
2020-04-02 15:03 ` Brian Cain
2020-04-02 15:03 ` Brian Cain
2020-04-02 15:03 ` Brian Cain
2020-04-02 15:03 ` Brian Cain
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=7ebf8cbc-0186-c8ea-fc0d-979c762b526b@linaro.org \
--to=daniel.lezcano@linaro.org \
--cc=afzal.mohd.ma@gmail.com \
--cc=allison@lohutok.net \
--cc=baohua@kernel.org \
--cc=bcm-kernel-feedback-list@broadcom.com \
--cc=f.fainelli@gmail.com \
--cc=festevam@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=info@metux.net \
--cc=kernel@pengutronix.de \
--cc=kgene@kernel.org \
--cc=khilman@baylibre.com \
--cc=krzk@kernel.org \
--cc=kstewart@linuxfoundation.org \
--cc=linus.walleij@linaro.org \
--cc=linux-amlogic@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-imx@nxp.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-rpi-kernel@lists.infradead.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=linux@prisktech.co.nz \
--cc=nsaenzjulienne@suse.de \
--cc=rjui@broadcom.com \
--cc=s.hauer@pengutronix.de \
--cc=sbranden@broadcom.com \
--cc=shawnguo@kernel.org \
--cc=tglx@linutronix.de \
--cc=u.kleine-koenig@pengutronix.de \
/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.