From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932978AbcIUHwt (ORCPT ); Wed, 21 Sep 2016 03:52:49 -0400 Received: from Galois.linutronix.de ([146.0.238.70]:33451 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932754AbcIUHwq (ORCPT ); Wed, 21 Sep 2016 03:52:46 -0400 Date: Wed, 21 Sep 2016 09:50:11 +0200 (CEST) From: Thomas Gleixner To: Alexandre Torgue cc: Maxime Coquelin , Jason Cooper , Marc Zyngier , Linus Walleij , Mark Rutland , Rob Herring , linux-gpio@vger.kernel.org, arnd@arndb.de, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, Daniel Thompson , bruherrera@gmail.com, lee.jones@linaro.org Subject: Re: [PATCH v6 2/4] drivers: irqchip: Add STM32 external interrupts support In-Reply-To: Message-ID: References: <1474387259-18926-1-git-send-email-alexandre.torgue@st.com> <1474387259-18926-3-git-send-email-alexandre.torgue@st.com> User-Agent: Alpine 2.20 (DEB 67 2015-01-07) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 21 Sep 2016, Alexandre Torgue wrote: > Hi Thomas, > > On 09/20/2016 10:16 PM, Thomas Gleixner wrote: > > Alexandre, > > > > On Tue, 20 Sep 2016, Alexandre TORGUE wrote: > > > > > The STM32 external interrupt controller consists of edge detectors that > > > generate interrupts requests or wake-up events. > > > > > > Each line can be independently configured as interrupt or wake-up source, > > > and triggers either on rising, falling or both edges. Each line can also > > > be masked independently. > > > > > > Signed-off-by: Maxime Coquelin > > > Signed-off-by: Alexandre TORGUE > > > > That all looks very reasonable now. The only remaining question is your SOB > > chain. Who is the author of these patches? You or Maxime? If it's Maxime, > > then the changelog misses a From: tag. If it's you then Maximes SOB is > > bogus. > > Actually Maxime wrote the main part of this driver and sent version 1 and 2 of > the series. After Linus W. reviews, rework was required to use hierarchical > domain. According to Maxime, I coded the rework (adaptation to hierarchical > domain) and sent other version of the series. So I replace Maximes SOB with Originally-by: Does that apply to all four patches? Thanks, tglx