Re: [PATCH v2 8/9] atmel_flexcom: Support backup mode
From: Alexandre Belloni
Date: Wed Sep 20 2017 - 05:18:41 EST
On 20/09/2017 at 10:30:31 +0200, Romain Izard wrote:
> 2017-09-19 17:25 GMT+02:00 Lee Jones <lee.jones@xxxxxxxxxx>:
> > 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 <romain.izard.pro@xxxxxxxxx>
> >> Tested-by: Nicolas Ferre <nicolas.ferre@xxxxxxxxxxxxx>
> >> On sama5d2 Xplained board (i2c0 from flexcom 4).
> >> and obviously:
> >> Acked-by: Nicolas Ferre <nicolas.ferre@xxxxxxxxxxxxx>
> >> 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