Messages in this thread | | | Date | Wed, 25 Nov 2015 00:17:36 +0100 | From | Alexandre Belloni <> | Subject | Re: [PATCH 5/9] rtc-pcf2123: put the chip reset into a function |
| |
On 04/11/2015 at 07:36:36 -0800, Joshua Clayton wrote :
A commit message, even when simple is mandatory.
> Signed-off-by: Joshua Clayton <stillcompiling@gmail.com> > --- > drivers/rtc/rtc-pcf2123.c | 64 ++++++++++++++++++++++++++--------------------- > 1 file changed, 36 insertions(+), 28 deletions(-) > > diff --git a/drivers/rtc/rtc-pcf2123.c b/drivers/rtc/rtc-pcf2123.c > index 257ce7d..d3c1447 100644 > --- a/drivers/rtc/rtc-pcf2123.c > +++ b/drivers/rtc/rtc-pcf2123.c > @@ -260,6 +260,40 @@ static int pcf2123_rtc_set_time(struct device *dev, struct rtc_time *tm) > return 0; > } > > +static int pcf2123_reset(struct device *dev) > +{ > + int ret; > + u8 rxbuf[2]; > + > + ret = pcf2123_write_reg(dev, PCF2123_REG_CTRL1, CTRL1_SW_RESET); > + if (ret < 0) > + return ret; > + > + /* Stop the counter */ > + dev_dbg(dev, "stopping RTC\n"); > + ret = pcf2123_write_reg(dev, PCF2123_REG_CTRL1, CTRL1_STOP); > + if (ret < 0) > + return ret; > + > + /* See if the counter was actually stopped */ > + dev_dbg(dev, "checking for presence of RTC\n"); > + ret = pcf2123_read(dev, PCF2123_REG_CTRL1, rxbuf, sizeof(rxbuf)); > + if (ret < 0) > + return ret; > + > + dev_dbg(dev, "received data from RTC (0x%02X 0x%02X)\n", > + rxbuf[0], rxbuf[1]);
This is not properly aligned (use checkpatch --strict)
-- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
| |