From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751878AbdISPZP (ORCPT ); Tue, 19 Sep 2017 11:25:15 -0400 Received: from mail-wm0-f48.google.com ([74.125.82.48]:46426 "EHLO mail-wm0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751862AbdISPZJ (ORCPT ); Tue, 19 Sep 2017 11:25:09 -0400 X-Google-Smtp-Source: AOwi7QBi6n4lc4LDz9UsKpFBc++szWHphAVzaqPFVmDr1y/UWgAOy58CzJ/IOLZkJx3yzYTVy424VQ== Date: Tue, 19 Sep 2017 16:25:04 +0100 From: Lee Jones To: Nicolas Ferre Cc: Romain Izard , Alexandre Belloni , 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, linux-kernel@vger.kernel.org, linux-mtd@lists.infradead.org, linux-pwm@vger.kernel.org, linux-serial@vger.kernel.org, linux-usb@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH v2 8/9] atmel_flexcom: Support backup mode Message-ID: <20170919152504.nhp5i734u75gb5gq@dell> References: <20170915140411.31716-1-romain.izard.pro@gmail.com> <20170915140411.31716-9-romain.izard.pro@gmail.com> <53aa7783-0ecf-1e68-1bf9-c470ba2f79eb@microchip.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <53aa7783-0ecf-1e68-1bf9-c470ba2f79eb@microchip.com> User-Agent: NeoMutt/20170113 (1.7.2) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org 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? -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH v2 8/9] atmel_flexcom: Support backup mode Date: Tue, 19 Sep 2017 16:25:04 +0100 Message-ID: <20170919152504.nhp5i734u75gb5gq@dell> References: <20170915140411.31716-1-romain.izard.pro@gmail.com> <20170915140411.31716-9-romain.izard.pro@gmail.com> <53aa7783-0ecf-1e68-1bf9-c470ba2f79eb@microchip.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit Return-path: Content-Disposition: inline In-Reply-To: <53aa7783-0ecf-1e68-1bf9-c470ba2f79eb@microchip.com> Sender: linux-kernel-owner@vger.kernel.org To: Nicolas Ferre Cc: Romain Izard , Alexandre Belloni , 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, linux-kernel@vger.kernel.org List-Id: linux-pwm@vger.kernel.org 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? -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog From mboxrd@z Thu Jan 1 00:00:00 1970 From: lee.jones@linaro.org (Lee Jones) Date: Tue, 19 Sep 2017 16:25:04 +0100 Subject: [PATCH v2 8/9] atmel_flexcom: Support backup mode In-Reply-To: <53aa7783-0ecf-1e68-1bf9-c470ba2f79eb@microchip.com> References: <20170915140411.31716-1-romain.izard.pro@gmail.com> <20170915140411.31716-9-romain.izard.pro@gmail.com> <53aa7783-0ecf-1e68-1bf9-c470ba2f79eb@microchip.com> Message-ID: <20170919152504.nhp5i734u75gb5gq@dell> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org 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? -- Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org ? Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog