Re: [rtc-linux] Re: [PATCH v4 0/2] Add support for cros-ec-rtc driver.

[Alexandre Belloni]

On 13/12/2017 at 07:55:35 +, Lee Jones wrote:
> On Tue, 12 Dec 2017, Alexandre Belloni wrote:
> 
> > On 12/12/2017 at 17:05:37 +0100, Alexandre Belloni wrote:
> > > On 12/12/2017 at 16:41:39 +0100, Enric Balletbo Serra wrote:
> > > > +   Alessandro Zummo
> > > > +   Alexandre Belloni
> > > > 
> > > > 2017-11-10 22:55 GMT+01:00 Enric Balletbo i Serra
> > > > :
> > > > > Dear all,
> > > > >
> > > > > This is an attempt to revive some patches from that [1] patchset, some
> > > > > of them are still under discussion but I think there is no reason to 
> > > > > not
> > > > > have the other two in this fourth version to land upstream meanwhile 
> > > > > we
> > > > > discuss about the others.
> > > > >
> > > > > [1] https://lkml.org/lkml/2017/7/12/182
> > > > >
> > > > > Changes since v3:
> > > > > * Rebased an retested with current mainline using a Samsung 
> > > > > Chromebook Plus
> > > > > * Removed from patchset
> > > > >   * 1/4 mfd: cros_ec: Get rid of cros_ec_check_features from 
> > > > > cros_ec_dev.
> > > > >   * 4/4 mfd: cros_ec: add RTC as mfd subdevice.
> > > > >
> > > > > Changes since v2:
> > > > > - Rebase on top of mainline.
> > > > > - Removed patch 'mfd: cros-ec: Fix host command buffer size' from 
> > > > > series
> > > > > as was already picked.
> > > > >
> > > > > Changes since v1:
> > > > > - Removed patch 'iio: cros_ec_sensors: Fix return value to get raw and
> > > > > calibbias data' from series as was already picked.
> > > > > - Removed patch 'iio: cros_ec_sensors: Fix return value to get raw and
> > > > > calibbias data' from series as was already picked.
> > > > > - Patch 2/5: Acked-by: Jonathan Cameron <***@kernel.org>
> > > > >
> > > > > Best regards,
> > > > >
> > > > >  Enric
> > > > >
> > > > > Stephen Barber (2):
> > > > >   mfd: cros_ec: Introduce RTC commands and events definitions.
> > > > >   rtc: cros-ec: add cros-ec-rtc driver.
> > > > >
> > > > >  drivers/rtc/Kconfig  |  10 +
> > > > >  drivers/rtc/Makefile |   1 +
> > > > >  drivers/rtc/rtc-cros-ec.c| 413 
> > > > > +++
> > > > >  include/linux/mfd/cros_ec_commands.h |   8 +
> > > > >  4 files changed, 432 insertions(+)
> > > > >  create mode 100644 drivers/rtc/rtc-cros-ec.c
> > > > >
> > > > >
> > > > 
> > > > Sorry, I just noticed that my script to add the email recipients
> > > > failed and I did not add the RTC maintainers :( So this is a gentle
> > > > ping cc'ing the maintainers to take in consideration these patches. If
> > > > you want I resend or something else just let me know please.
> > > > 
> > > 
> > > I don't remember the specifics but I think the patches have the
> > > necessary acks from me to go through the mfd tree (didn't we agree on
> > > that ?)
> > > 
> > 
> > This was the reply from Lee a while ago:
> > 
> > > On Wed, 18 Jan 2017, Alexandre Belloni wrote:
> > > > I don't think that one has your ack, is it ok for me to take it?
> > > 
> > > This set looks MFD heavy, so it's probably best that the set goes
> > > through MFD.  Once the RTC patch has been fixed of course.
> > 
> > The series doesn't look MFD heavy anymore so it can also go through my
> > tree. I'll let Lee decide.
> 
> Fine by me.
> 

Applied and pushed now.

Lee, if you need a immutable branch at some point, I can create one.

-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
--- 
You received this message because you are subscribed to the Google Groups 
"rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to rtc-linux+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: [rtc-linux] Re: [PATCH v4 0/2] Add support for cros-ec-rtc driver.

[Lee Jones]

On Tue, 12 Dec 2017, Alexandre Belloni wrote:

> On 12/12/2017 at 17:05:37 +0100, Alexandre Belloni wrote:
> > On 12/12/2017 at 16:41:39 +0100, Enric Balletbo Serra wrote:
> > > +   Alessandro Zummo
> > > +   Alexandre Belloni
> > > 
> > > 2017-11-10 22:55 GMT+01:00 Enric Balletbo i Serra
> > > :
> > > > Dear all,
> > > >
> > > > This is an attempt to revive some patches from that [1] patchset, some
> > > > of them are still under discussion but I think there is no reason to not
> > > > have the other two in this fourth version to land upstream meanwhile we
> > > > discuss about the others.
> > > >
> > > > [1] https://lkml.org/lkml/2017/7/12/182
> > > >
> > > > Changes since v3:
> > > > * Rebased an retested with current mainline using a Samsung Chromebook 
> > > > Plus
> > > > * Removed from patchset
> > > >   * 1/4 mfd: cros_ec: Get rid of cros_ec_check_features from 
> > > > cros_ec_dev.
> > > >   * 4/4 mfd: cros_ec: add RTC as mfd subdevice.
> > > >
> > > > Changes since v2:
> > > > - Rebase on top of mainline.
> > > > - Removed patch 'mfd: cros-ec: Fix host command buffer size' from series
> > > > as was already picked.
> > > >
> > > > Changes since v1:
> > > > - Removed patch 'iio: cros_ec_sensors: Fix return value to get raw and
> > > > calibbias data' from series as was already picked.
> > > > - Removed patch 'iio: cros_ec_sensors: Fix return value to get raw and
> > > > calibbias data' from series as was already picked.
> > > > - Patch 2/5: Acked-by: Jonathan Cameron <***@kernel.org>
> > > >
> > > > Best regards,
> > > >
> > > >  Enric
> > > >
> > > > Stephen Barber (2):
> > > >   mfd: cros_ec: Introduce RTC commands and events definitions.
> > > >   rtc: cros-ec: add cros-ec-rtc driver.
> > > >
> > > >  drivers/rtc/Kconfig  |  10 +
> > > >  drivers/rtc/Makefile |   1 +
> > > >  drivers/rtc/rtc-cros-ec.c| 413 
> > > > +++
> > > >  include/linux/mfd/cros_ec_commands.h |   8 +
> > > >  4 files changed, 432 insertions(+)
> > > >  create mode 100644 drivers/rtc/rtc-cros-ec.c
> > > >
> > > >
> > > 
> > > Sorry, I just noticed that my script to add the email recipients
> > > failed and I did not add the RTC maintainers :( So this is a gentle
> > > ping cc'ing the maintainers to take in consideration these patches. If
> > > you want I resend or something else just let me know please.
> > > 
> > 
> > I don't remember the specifics but I think the patches have the
> > necessary acks from me to go through the mfd tree (didn't we agree on
> > that ?)
> > 
> 
> This was the reply from Lee a while ago:
> 
> > On Wed, 18 Jan 2017, Alexandre Belloni wrote:
> > > I don't think that one has your ack, is it ok for me to take it?
> > 
> > This set looks MFD heavy, so it's probably best that the set goes
> > through MFD.  Once the RTC patch has been fixed of course.
> 
> The series doesn't look MFD heavy anymore so it can also go through my
> tree. I'll let Lee decide.

Fine by me.

-- 
Lee Jones
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

-- 
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
--- 
You received this message because you are subscribed to the Google Groups 
"rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to rtc-linux+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Re: [rtc-linux] Re: [PATCH v4 0/2] Add support for cros-ec-rtc driver.

[Alexandre Belloni]

On 12/12/2017 at 17:05:37 +0100, Alexandre Belloni wrote:
> On 12/12/2017 at 16:41:39 +0100, Enric Balletbo Serra wrote:
> > +   Alessandro Zummo
> > +   Alexandre Belloni
> > 
> > 2017-11-10 22:55 GMT+01:00 Enric Balletbo i Serra
> > :
> > > Dear all,
> > >
> > > This is an attempt to revive some patches from that [1] patchset, some
> > > of them are still under discussion but I think there is no reason to not
> > > have the other two in this fourth version to land upstream meanwhile we
> > > discuss about the others.
> > >
> > > [1] https://lkml.org/lkml/2017/7/12/182
> > >
> > > Changes since v3:
> > > * Rebased an retested with current mainline using a Samsung Chromebook 
> > > Plus
> > > * Removed from patchset
> > >   * 1/4 mfd: cros_ec: Get rid of cros_ec_check_features from cros_ec_dev.
> > >   * 4/4 mfd: cros_ec: add RTC as mfd subdevice.
> > >
> > > Changes since v2:
> > > - Rebase on top of mainline.
> > > - Removed patch 'mfd: cros-ec: Fix host command buffer size' from series
> > > as was already picked.
> > >
> > > Changes since v1:
> > > - Removed patch 'iio: cros_ec_sensors: Fix return value to get raw and
> > > calibbias data' from series as was already picked.
> > > - Removed patch 'iio: cros_ec_sensors: Fix return value to get raw and
> > > calibbias data' from series as was already picked.
> > > - Patch 2/5: Acked-by: Jonathan Cameron <***@kernel.org>
> > >
> > > Best regards,
> > >
> > >  Enric
> > >
> > > Stephen Barber (2):
> > >   mfd: cros_ec: Introduce RTC commands and events definitions.
> > >   rtc: cros-ec: add cros-ec-rtc driver.
> > >
> > >  drivers/rtc/Kconfig  |  10 +
> > >  drivers/rtc/Makefile |   1 +
> > >  drivers/rtc/rtc-cros-ec.c| 413 
> > > +++
> > >  include/linux/mfd/cros_ec_commands.h |   8 +
> > >  4 files changed, 432 insertions(+)
> > >  create mode 100644 drivers/rtc/rtc-cros-ec.c
> > >
> > > --
> > > 2.9.3
> > >
> > 
> > Sorry, I just noticed that my script to add the email recipients
> > failed and I did not add the RTC maintainers :( So this is a gentle
> > ping cc'ing the maintainers to take in consideration these patches. If
> > you want I resend or something else just let me know please.
> > 
> 
> I don't remember the specifics but I think the patches have the
> necessary acks from me to go through the mfd tree (didn't we agree on
> that ?)
> 

This was the reply from Lee a while ago:

> On Wed, 18 Jan 2017, Alexandre Belloni wrote:
> > I don't think that one has your ack, is it ok for me to take it?
> 
> This set looks MFD heavy, so it's probably best that the set goes
> through MFD.  Once the RTC patch has been fixed of course.

The series doesn't look MFD heavy anymore so it can also go through my
tree. I'll let Lee decide.

-- 
Alexandre Belloni, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com

-- 
You received this message because you are subscribed to "rtc-linux".
Membership options at http://groups.google.com/group/rtc-linux .
Please read http://groups.google.com/group/rtc-linux/web/checklist
before submitting a driver.
--- 
You received this message because you are subscribed to the Google Groups 
"rtc-linux" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to rtc-linux+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.