Hello Doug, On 08/21/2014 12:36 AM, Doug Anderson wrote: > Javier, > > On Wed, Aug 20, 2014 at 5:13 AM, Javier Martinez Canillas > <javier.martinez@xxxxxxxxxxxxxxx> wrote: >> From: Andrew Bresticker <abrestic@xxxxxxxxxxxx> >> >> Now that there's a central cros_ec_cmd_xfer(), move the locking >> out of the SPI and LPC drivers. > > Slight nit that the LPC driver doesn't exist upstream. This is in > prep for adding the LPC driver, though. > Right, the downstream commit was also touching the LPC driver and I stripped that part but forget to update the commit message, sorry about that. I'll fix it and do a re-spin. > >> Signed-off-by: Andrew Bresticker <abrestic@xxxxxxxxxxxx> >> Reviewed-by: Simon Glass <sjg@xxxxxxxxxxxx> >> Signed-off-by: Javier Martinez Canillas <javier.martinez@xxxxxxxxxxxxxxx> >> --- >> drivers/mfd/cros_ec.c | 10 +++++++++- >> drivers/mfd/cros_ec_spi.c | 11 ----------- >> 2 files changed, 9 insertions(+), 12 deletions(-) > > After comment nitfix: > > Reviewed-by: Doug Anderson <dianders@xxxxxxxxxxxx> > Thanks and best regards, Javier -- To unsubscribe from this list: send the line "unsubscribe linux-i2c" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
