LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Lee Jones <lee.jones@linaro.org>
To: Kevin Hilman <khilman@baylibre.com>
Cc: Marc Zyngier <maz@kernel.org>,
Neil Armstrong <narmstrong@baylibre.com>,
Jerome Brunet <jbrunet@baylibre.com>,
linux-amlogic@lists.infradead.org,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>,
open list <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 1/2] irqchip: irq-meson-gpio: make it possible to build as a module
Date: Tue, 13 Jul 2021 10:05:23 +0100 [thread overview]
Message-ID: <YO1XU2ySZL3d03RH@google.com> (raw)
In-Reply-To: <7h7diwgjup.fsf@baylibre.com>
On Mon, 14 Jun 2021, Kevin Hilman wrote:
> Marc Zyngier <maz@kernel.org> writes:
>
> > On Fri, 21 May 2021 10:47:48 +0100,
> > Lee Jones <lee.jones@linaro.org> wrote:
> >>
> >> [1 <text/plain; UTF-8 (quoted-printable)>]
> >> On Tue, 20 Oct 2020 at 19:23, Kevin Hilman <khilman@baylibre.com> wrote:
> >>
> >> > Neil Armstrong <narmstrong@baylibre.com> writes:
> >> >
> >> > > In order to reduce the kernel Image size on multi-platform distributions,
> >> > > make it possible to build the Amlogic GPIO IRQ controller as a module
> >> > > by switching it to a platform driver.
> >> > >
> >> > > Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> >> >
> >> > Reviewed-by: Kevin Hilman <khilman@baylibre.com>
> >> > Tested-by: Kevin Hilman <khilman@baylibre.com>
> >> >
> >> > Tested as a module on meson-sm1-khadas-vim3l where the wired networking
> >> > uses GPIO IRQs.
> >> >
> >>
> >> Good morning Neil, Kevin,
> >>
> >> What happened to this set in the end? I still don't see it in Mainline.
> >
> > Last time I tried this patch, it broke my test setup in non-obvious
> > ways. Has someone checked that the issue I reported back then has been
> > resolved now that fw_devlink is more usable?
>
> OK, after much anticipation (and much delay due to me forgetting about
> this), I just gave this series a spin again on top of v5.13-rc6, and it
> seems to work fine with `fw_devlink=on`
>
> I started with your config[1] and accepting all the defaults of any new
> configs. IOW, I ran: yes '' | make oldconfig after copying your config
> to .config.
>
> With that it seems to be working fine for me.
>
> Right after boot (and before network probes) I see module loaded, but no
> users yet in /proc/interrupts:
>
> / # uname -a
> Linux buildroot 5.13.0-rc6-00002-g679c8e852942 #5 SMP PREEMPT Mon Jun 14 15:08:40 PDT 2021 aarch64 GNU/Linux
> / # lsmod |grep gpio
> irq_meson_gpio 20480 0
> leds_gpio 16384 0
> / # cat /proc/interrupts
> CPU0 CPU1 CPU2 CPU3
> 9: 0 0 0 0 GICv2 25 Level vgic
> 11: 0 0 0 0 GICv2 30 Level kvm guest ptimer
> 12: 0 0 0 0 GICv2 27 Level kvm guest vtimer
> 13: 1416 916 534 1421 GICv2 26 Level arch_timer
> 15: 5 0 0 0 GICv2 89 Edge dw_hdmi_top_irq, ff600000.hdmi-tx
> 22: 38 0 0 0 GICv2 225 Edge ttyAML0
> 23: 20 0 0 0 GICv2 227 Edge ff805000.i2c
> 25: 2 0 0 0 GICv2 232 Edge ff809000.adc
> 28: 322 0 0 0 GICv2 35 Edge meson
> 31: 0 0 0 0 GICv2 222 Edge ffe05000.sd
> 32: 787 0 0 0 GICv2 223 Edge ffe07000.mmc
> 34: 0 0 0 0 GICv2 194 Level panfrost-job
> 35: 0 0 0 0 GICv2 193 Level panfrost-mmu
> 36: 3 0 0 0 GICv2 192 Level panfrost-gpu
> 39: 0 0 0 0 GICv2 63 Level ff400000.usb, ff400000.usb
> 40: 32 0 0 0 GICv2 62 Level xhci-hcd:usb1
> IPI0: 425 544 664 925 Rescheduling interrupts
> IPI1: 86 166 269 136 Function call interrupts
> IPI2: 0 0 0 0 CPU stop interrupts
> IPI3: 0 0 0 0 CPU stop (for crash dump) interrupts
> IPI4: 0 0 0 0 Timer broadcast interrupts
> IPI5: 0 0 0 0 IRQ work interrupts
> IPI6: 0 0 0 0 CPU wake-up interrupts
> Err: 0
>
> So then I init the network interface and PHY works, DHCP works etc.
>
> / # udhcpc
> udhcpc: started, v1.31.1
> [ 102.250449] meson8b-dwmac ff3f0000.ethernet eth0: PHY [0.0:00] driver [RTL8211F Gigabit Ethernet] (irq=37)
> [ 102.256413] meson8b-dwmac ff3f0000.ethernet eth0: Register MEM_TYPE_PAGE_POOL RxQ-0
> [ 102.269433] meson8b-dwmac ff3f0000.ethernet eth0: No Safety Features support found
> [ 102.271357] meson8b-dwmac ff3f0000.ethernet eth0: PTP not supported by HW
> [ 102.278493] meson8b-dwmac ff3f0000.ethernet eth0: configuring for phy/rgmii link mode
> udhcpc: sending discover
> [ 104.743301] meson8b-dwmac ff3f0000.ethernet eth0: Link is Up - 100Mbps/Full - flow control rx/tx
> [ 104.746470] IPv6: ADDRCONF(NETDEV_CHANGE): eth0: link becomes ready
> udhcpc: sending discover
> udhcpc: sending select for 192.168.0.122
> udhcpc: lease of 192.168.0.122 obtained, lease time 600
> deleting routers
> adding dns 192.168.0.254
> adding dns 192.168.0.254
> / # cat /proc/interrupts
> CPU0 CPU1 CPU2 CPU3
> 9: 0 0 0 0 GICv2 25 Level vgic
> 11: 0 0 0 0 GICv2 30 Level kvm guest ptimer
> 12: 0 0 0 0 GICv2 27 Level kvm guest vtimer
> 13: 1575 1018 604 1588 GICv2 26 Level arch_timer
> 14: 8 0 0 0 GICv2 40 Level eth0
> 15: 5 0 0 0 GICv2 89 Edge dw_hdmi_top_irq, ff600000.hdmi-tx
> 22: 132 0 0 0 GICv2 225 Edge ttyAML0
> 23: 20 0 0 0 GICv2 227 Edge ff805000.i2c
> 25: 2 0 0 0 GICv2 232 Edge ff809000.adc
> 28: 322 0 0 0 GICv2 35 Edge meson
> 31: 0 0 0 0 GICv2 222 Edge ffe05000.sd
> 32: 787 0 0 0 GICv2 223 Edge ffe07000.mmc
> 34: 0 0 0 0 GICv2 194 Level panfrost-job
> 35: 0 0 0 0 GICv2 193 Level panfrost-mmu
> 36: 3 0 0 0 GICv2 192 Level panfrost-gpu
> 37: 2 0 0 0 meson-gpio-irqchip 26 Level 0.0:00
> 39: 0 0 0 0 GICv2 63 Level ff400000.usb, ff400000.usb
> 40: 32 0 0 0 GICv2 62 Level xhci-hcd:usb1
> IPI0: 476 567 720 956 Rescheduling interrupts
> IPI1: 93 166 270 137 Function call interrupts
> IPI2: 0 0 0 0 CPU stop interrupts
> IPI3: 0 0 0 0 CPU stop (for crash dump) interrupts
> IPI4: 0 0 0 0 Timer broadcast interrupts
> IPI5: 0 0 0 0 IRQ work interrupts
> IPI6: 0 0 0 0 CPU wake-up interrupts
> Err: 0
> / #
>
> Kevin
>
> [1] http://www.loen.fr/tmp/Config.full-arm64
Thanks Kevin.
Now that -rc1 is out, hopefully Marc can assign some time to this.
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
next prev parent reply other threads:[~2021-07-13 9:05 UTC|newest]
Thread overview: 41+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-20 7:25 [PATCH 0/2] " Neil Armstrong
2020-10-20 7:25 ` [PATCH 1/2] irqchip: " Neil Armstrong
2020-10-20 18:23 ` Kevin Hilman
[not found] ` <CAF2Aj3g6c8FEZb3e1by6sd8LpKLaeN5hsKrrQkZUvh8hosiW9A@mail.gmail.com>
2021-05-24 10:11 ` Marc Zyngier
2021-05-25 16:17 ` Kevin Hilman
2021-05-25 16:30 ` Lee Jones
2021-06-14 22:30 ` Kevin Hilman
2021-07-13 9:05 ` Lee Jones [this message]
2021-08-03 9:44 ` Marc Zyngier
2021-08-03 9:51 ` Marc Zyngier
2021-08-04 2:11 ` Saravana Kannan
2021-08-03 9:51 ` Neil Armstrong
2021-08-04 1:36 ` Saravana Kannan
2021-08-04 8:50 ` Marc Zyngier
2021-08-04 18:20 ` Saravana Kannan
2021-08-04 21:47 ` Saravana Kannan
2021-08-05 6:31 ` Neil Armstrong
2021-08-06 23:55 ` Saravana Kannan
2021-08-05 7:57 ` Lee Jones
2021-08-16 12:47 ` Lee Jones
2021-08-16 20:27 ` Saravana Kannan
2021-08-16 20:46 ` Andrew Lunn
2021-08-16 21:02 ` Saravana Kannan
2021-08-16 21:18 ` Andrew Lunn
2021-08-17 7:24 ` Lee Jones
2021-08-17 18:12 ` Saravana Kannan
2021-08-18 11:19 ` Marc Zyngier
2021-09-02 9:28 ` Neil Armstrong
2020-10-20 7:25 ` [PATCH 2/2] arm64: meson: remove MESON_IRQ_GPIO selection Neil Armstrong
2020-10-20 23:18 ` Kevin Hilman
2020-10-25 11:51 ` [PATCH 0/2] irq-meson-gpio: make it possible to build as a module Marc Zyngier
2020-10-26 16:18 ` Kevin Hilman
2020-10-26 17:00 ` Marc Zyngier
2020-10-26 17:28 ` Kevin Hilman
2020-10-26 17:33 ` Kevin Hilman
2020-10-26 18:30 ` Marc Zyngier
2020-10-26 23:45 ` Kevin Hilman
2021-09-02 13:49 Neil Armstrong
2021-09-02 13:49 ` [PATCH 1/2] irqchip: " Neil Armstrong
2021-09-02 17:00 ` Saravana Kannan
2021-09-28 10:45 ` Lee Jones
2021-09-28 23:27 ` Kevin Hilman
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=YO1XU2ySZL3d03RH@google.com \
--to=lee.jones@linaro.org \
--cc=jbrunet@baylibre.com \
--cc=khilman@baylibre.com \
--cc=linux-amlogic@lists.infradead.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=maz@kernel.org \
--cc=narmstrong@baylibre.com \
--subject='Re: [PATCH 1/2] irqchip: irq-meson-gpio: make it possible to build as a module' \
/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
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).