From: Marc Zyngier <marc.zyngier@arm.com> To: "Radosław Pietrzyk" <radoslaw.pietrzyk@gmail.com> Cc: Thomas Gleixner <tglx@linutronix.de>, Jason Cooper <jason@lakedaemon.net>, Maxime Coquelin <mcoquelin.stm32@gmail.com>, Alexandre Torgue <alexandre.torgue@st.com>, Linus Walleij <linus.walleij@linaro.org>, Benjamin Gaignard <benjamin.gaignard@linaro.org>, Philipp Zabel <p.zabel@pengutronix.de>, open list <linux-kernel@vger.kernel.org>, "moderated list:ARM/STM32 ARCHITECTURE" <linux-arm-kernel@lists.infradead.org>, "open list:GPIO SUBSYSTEM" <linux-gpio@vger.kernel.org>, Ludovic BARRE <ludovic.barre@st.com> Subject: Re: [PATCH v2 1/2] irqchip: stm32: Optimizes and cleans up stm32-exti irq_domain Date: Wed, 14 Mar 2018 12:04:50 +0000 [thread overview] Message-ID: <118e66b4-3137-7af5-b93d-b1949fad575c@arm.com> (raw) In-Reply-To: <CAFvLkMSZjgFMAe0VRc0AEpG89BgXkfMb9+ivJ8V+PFighOoDRA@mail.gmail.com> On 14/03/18 11:46, Radosław Pietrzyk wrote: > Hi Marc, > We had a quite fruitful discussion in this mail thread regarding this > topic and Ludovic acked it so recently I have asked Thomas if he still > needs this v3 patch with detailed explanation especially as v2 version > of stm32-gpio patch has been already taken by Linus. However if you > require I can resend v3 of this patch only with this detailed explanation. That'd be useful. The changelog is the only thing that will be left from this discussion, so it'd better be complete and accurate. If you quickly send a v3 for this single patch, I'll queue it right away. Thanks, M. > > 2018-03-14 12:09 GMT+01:00 Marc Zyngier <marc.zyngier@arm.com > <mailto:marc.zyngier@arm.com>>: > > Radoslaw, > > On 23/02/18 08:42, Thomas Gleixner wrote: > > Radoslaw, > > > > On Fri, 23 Feb 2018, Radoslaw Pietrzyk wrote: > > > >> - discards setting handle_simple_irq handler for hierarchy interrupts > >> - removes acking in chained irq handler as this is done by > >> irq_chip itself inside handle_edge_irq > >> - removes unneeded irq_domain_ops.xlate callback > > > > if that's all functionally correct, then this is a nice cleanup. Though > > from the above changelog its hard to tell because it merily tells WHAT the > > patch does, but not WHY. The WHY is the important information for a > > reviewer who is not familiar with the particular piece of code/hardware. > > > > Can you please amend the changelog with proper explanations why a > > particular piece of code is not needed or has to be changed to something > > else? > > Any update on this? I'd like to queue this for 4.17, but Thomas' > comments should be addressed before that happens. Ca you please respin a > version with a better change log and the various review tags? > > Thanks, > > M. > -- > Jazz is not dead. It just smells funny... > > -- Jazz is not dead. It just smells funny...
WARNING: multiple messages have this Message-ID (diff)
From: marc.zyngier@arm.com (Marc Zyngier) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH v2 1/2] irqchip: stm32: Optimizes and cleans up stm32-exti irq_domain Date: Wed, 14 Mar 2018 12:04:50 +0000 [thread overview] Message-ID: <118e66b4-3137-7af5-b93d-b1949fad575c@arm.com> (raw) In-Reply-To: <CAFvLkMSZjgFMAe0VRc0AEpG89BgXkfMb9+ivJ8V+PFighOoDRA@mail.gmail.com> On 14/03/18 11:46, Rados?aw Pietrzyk wrote: > Hi Marc, > We had a quite fruitful discussion in this mail thread regarding this > topic and Ludovic acked it so recently I have asked Thomas if he still > needs this v3 patch with detailed explanation especially as v2 version > of stm32-gpio patch has been already taken by Linus. However if you > require I can resend v3 of this patch only with this detailed explanation. That'd be useful. The changelog is the only thing that will be left from this discussion, so it'd better be complete and accurate. If you quickly send a v3 for this single patch, I'll queue it right away. Thanks, M. > > 2018-03-14 12:09 GMT+01:00 Marc Zyngier <marc.zyngier@arm.com > <mailto:marc.zyngier@arm.com>>: > > Radoslaw, > > On 23/02/18 08:42, Thomas Gleixner wrote: > > Radoslaw, > > > > On Fri, 23 Feb 2018, Radoslaw Pietrzyk wrote: > > > >> - discards setting handle_simple_irq handler for hierarchy interrupts > >> - removes acking in chained irq handler as this is done by > >> irq_chip itself inside handle_edge_irq > >> - removes unneeded irq_domain_ops.xlate callback > > > > if that's all functionally correct, then this is a nice cleanup. Though > > from the above changelog its hard to tell because it merily tells WHAT the > > patch does, but not WHY. The WHY is the important information for a > > reviewer who is not familiar with the particular piece of code/hardware. > > > > Can you please amend the changelog with proper explanations why a > > particular piece of code is not needed or has to be changed to something > > else? > > Any update on this? I'd like to queue this for 4.17, but Thomas' > comments should be addressed before that happens. Ca you please respin a > version with a better change log and the various review tags? > > Thanks, > > ? ? ? ? M. > -- > Jazz is not dead. It just smells funny... > > -- Jazz is not dead. It just smells funny...
next prev parent reply other threads:[~2018-03-14 12:04 UTC|newest] Thread overview: 69+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-02-21 13:50 [PATCH 0/2] Optimizes, cleans up and enhances stm32-exti irq hierarchy Radoslaw Pietrzyk 2018-02-21 13:50 ` Radoslaw Pietrzyk 2018-02-21 13:50 ` [PATCH 1/2] ARM: irqchip: stm32: Optimizes and cleans up stm32-exti irq_domain Radoslaw Pietrzyk 2018-02-21 13:50 ` Radoslaw Pietrzyk 2018-02-22 10:55 ` [1/2] " Ludovic BARRE 2018-02-22 10:55 ` Ludovic BARRE 2018-02-22 10:55 ` Ludovic BARRE 2018-02-22 11:01 ` Radosław Pietrzyk 2018-02-22 11:01 ` Radosław Pietrzyk 2018-02-22 11:01 ` Radosław Pietrzyk 2018-02-23 8:34 ` Radosław Pietrzyk 2018-02-23 8:34 ` Radosław Pietrzyk 2018-02-23 8:34 ` Radosław Pietrzyk 2018-02-21 13:50 ` [PATCH 2/2] ARM: pinctrl: stm32: Optimizes and enhances stm32gpio irqchip Radoslaw Pietrzyk 2018-02-21 13:50 ` Radoslaw Pietrzyk 2018-02-21 15:11 ` Alexandre Torgue 2018-02-21 15:11 ` Alexandre Torgue 2018-02-21 15:11 ` Alexandre Torgue 2018-02-21 15:12 ` Alexandre Torgue 2018-02-21 15:12 ` Alexandre Torgue 2018-02-21 15:12 ` Alexandre Torgue 2018-03-01 21:04 ` Linus Walleij 2018-03-01 21:04 ` Linus Walleij 2018-03-01 21:24 ` Radosław Pietrzyk 2018-03-01 21:24 ` Radosław Pietrzyk 2018-03-02 8:14 ` Alexandre Torgue 2018-03-02 8:14 ` Alexandre Torgue 2018-03-02 8:14 ` Alexandre Torgue 2018-03-02 8:37 ` Radosław Pietrzyk 2018-03-02 8:37 ` Radosław Pietrzyk 2018-03-02 8:49 ` Alexandre Torgue 2018-03-02 8:49 ` Alexandre Torgue 2018-02-23 8:31 ` [PATCH v2 0/2] v2 patches for stm32-exti irq hierarchy Radoslaw Pietrzyk 2018-02-23 8:31 ` Radoslaw Pietrzyk 2018-02-23 8:31 ` [PATCH v2 1/2] irqchip: stm32: Optimizes and cleans up stm32-exti irq_domain Radoslaw Pietrzyk 2018-02-23 8:31 ` Radoslaw Pietrzyk 2018-02-23 8:42 ` Thomas Gleixner 2018-02-23 8:42 ` Thomas Gleixner 2018-02-23 8:42 ` Thomas Gleixner 2018-02-23 9:05 ` Radosław Pietrzyk 2018-02-23 9:05 ` Radosław Pietrzyk 2018-02-23 9:05 ` Radosław Pietrzyk 2018-03-14 11:09 ` Marc Zyngier 2018-03-14 11:09 ` Marc Zyngier [not found] ` <CAFvLkMSZjgFMAe0VRc0AEpG89BgXkfMb9+ivJ8V+PFighOoDRA@mail.gmail.com> 2018-03-14 12:04 ` Marc Zyngier [this message] 2018-03-14 12:04 ` Marc Zyngier 2018-04-19 13:24 ` Ludovic BARRE 2018-04-19 13:24 ` Ludovic BARRE [not found] ` <CAFvLkMQv3AajhFaRxsBt+UzdGFeix_FA-0mhtYqwMo2x0zgrnA@mail.gmail.com> 2018-04-20 7:18 ` Ludovic BARRE 2018-04-20 7:18 ` Ludovic BARRE 2018-02-23 15:46 ` Ludovic BARRE 2018-02-23 15:46 ` Ludovic BARRE 2018-02-23 15:46 ` Ludovic BARRE 2018-02-23 16:16 ` Ludovic BARRE 2018-02-23 16:16 ` Ludovic BARRE 2018-02-23 16:16 ` Ludovic BARRE 2018-02-23 17:37 ` Radosław Pietrzyk 2018-02-23 17:37 ` Radosław Pietrzyk 2018-02-23 17:37 ` Radosław Pietrzyk 2018-02-26 15:23 ` Ludovic BARRE 2018-02-26 15:23 ` Ludovic BARRE 2018-02-26 15:23 ` Ludovic BARRE 2018-02-23 8:31 ` [PATCH v2 2/2] pinctrl: stm32: Optimizes and enhances stm32gpio irqchip Radoslaw Pietrzyk 2018-02-23 8:31 ` Radoslaw Pietrzyk 2018-02-26 15:25 ` Ludovic BARRE 2018-02-26 15:25 ` Ludovic BARRE 2018-02-26 15:25 ` Ludovic BARRE 2018-03-01 22:41 ` Linus Walleij 2018-03-01 22:41 ` Linus Walleij
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=118e66b4-3137-7af5-b93d-b1949fad575c@arm.com \ --to=marc.zyngier@arm.com \ --cc=alexandre.torgue@st.com \ --cc=benjamin.gaignard@linaro.org \ --cc=jason@lakedaemon.net \ --cc=linus.walleij@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-gpio@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=ludovic.barre@st.com \ --cc=mcoquelin.stm32@gmail.com \ --cc=p.zabel@pengutronix.de \ --cc=radoslaw.pietrzyk@gmail.com \ --cc=tglx@linutronix.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: linkBe 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.