diff mbox

Fixed ds1390_get_reg returned value

Message ID CAO1575Wk_O6RYMGJSx2=54OYj-_fZXSGum5bZa8fF=cpf5d5Gw@mail.gmail.com
State Superseded
Headers show

Commit Message

Ivan Grimaldi Sept. 16, 2015, 3:51 p.m. UTC
spi_write_then_read puts in rx_buf the received data starting from the
first byte of the rx_buf for n_rx bytes.

Signed-off-by: Ivan Grimaldi <grimaldi.ivan@gmail.com>
---
 }

Comments

Alexandre Belloni Sept. 16, 2015, 6:10 p.m. UTC | #1
Hi,

Your patch is not properly fomatted, it as probably been mangled by your
mailer can you resend it properly?

Thanks,

On 16/09/2015 at 17:51:17 +0200, Ivan Grimaldi wrote :
> spi_write_then_read puts in rx_buf the received data starting from the
> first byte of the rx_buf for n_rx bytes.
> 
> Signed-off-by: Ivan Grimaldi <grimaldi.ivan@gmail.com>
> ---
> diff --git a/drivers/rtc/rtc-ds1390.c b/drivers/rtc/rtc-ds1390.c
> index e67bfcb..a4303b4 100644
> --- a/drivers/rtc/rtc-ds1390.c
> +++ b/drivers/rtc/rtc-ds1390.c
> @@ -62,7 +62,7 @@ static int ds1390_get_reg(struct device *dev, unsigned
> char address,
>         if (status != 0)
>                 return status;
> 
> -       *data = chip->txrx_buf[1];
> +       *data = chip->txrx_buf[0];
> 
>         return 0;
>  }
> 
> -- 
> -- 
> 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+unsubscribe@googlegroups.com.
> For more options, visit https://groups.google.com/d/optout.
diff mbox

Patch

diff --git a/drivers/rtc/rtc-ds1390.c b/drivers/rtc/rtc-ds1390.c
index e67bfcb..a4303b4 100644
--- a/drivers/rtc/rtc-ds1390.c
+++ b/drivers/rtc/rtc-ds1390.c
@@ -62,7 +62,7 @@  static int ds1390_get_reg(struct device *dev, unsigned
char address,
        if (status != 0)
                return status;

-       *data = chip->txrx_buf[1];
+       *data = chip->txrx_buf[0];

        return 0;