Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751673AbdITJSh (ORCPT ); Wed, 20 Sep 2017 05:18:37 -0400 Received: from mail.free-electrons.com ([62.4.15.54]:49698 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751554AbdITJSe (ORCPT ); Wed, 20 Sep 2017 05:18:34 -0400 Date: Wed, 20 Sep 2017 11:18:29 +0200 From: Alexandre Belloni To: Romain Izard Cc: Lee Jones , Nicolas Ferre , Boris Brezillon , Michael Turquette , Stephen Boyd , Ludovic Desroches , Wenyou Yang , Josh Wu , David Woodhouse , Brian Norris , Marek Vasut , Cyrille Pitchen , Thierry Reding , Richard Genoud , Greg Kroah-Hartman , Alan Stern , linux-clk@vger.kernel.org, LKML , linux-mtd , linux-pwm@vger.kernel.org, linux-serial@vger.kernel.org, linux-usb@vger.kernel.org, linux-arm-kernel Subject: Re: [PATCH v2 8/9] atmel_flexcom: Support backup mode Message-ID: <20170920091829.wftbjshog3ixvpmu@piout.net> References: <20170915140411.31716-1-romain.izard.pro@gmail.com> <20170915140411.31716-9-romain.izard.pro@gmail.com> <53aa7783-0ecf-1e68-1bf9-c470ba2f79eb@microchip.com> <20170919152504.nhp5i734u75gb5gq@dell> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: NeoMutt/20170609 (1.8.3) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1422 Lines: 39 On 20/09/2017 at 10:30:31 +0200, Romain Izard wrote: > 2017-09-19 17:25 GMT+02:00 Lee Jones : > > On Tue, 19 Sep 2017, Nicolas Ferre wrote: > > > >> On 15/09/2017 at 16:04, Romain Izard wrote: > >> > The controller used by a flexcom module is configured at boot, and left > >> > alone after this. As the configuration will be lost after backup mode, > >> > restore the state of the flexcom driver on resume. > >> > > >> > Signed-off-by: Romain Izard > >> > >> Tested-by: Nicolas Ferre > >> On sama5d2 Xplained board (i2c0 from flexcom 4). > >> and obviously: > >> Acked-by: Nicolas Ferre > >> > >> Thanks Romain! > >> > >> Regards, > >> > >> > --- > >> > drivers/mfd/atmel-flexcom.c | 65 ++++++++++++++++++++++++++++++++++----------- > >> > 1 file changed, 50 insertions(+), 15 deletions(-) > > > > This is the first time I've seen this patch. Why's that? > > > > As the patchset covers many subsystems, get_maintainers.pl provided a > very long list of both developpers and mailing lists (28). I thought it > was a good idea to shorten it a little. Bad idea. Sorry. > I think the correct way of handling that would have been to send each patch to the proper subsystem as there are no dependency here. -- Alexandre Belloni, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com