Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 2 Feb 2023 06:57:14 GMT
From:      Emmanuel Vadot <manu@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: a829b880ea23 - stable/13 - sdio: Always use increment address for read/write_4
Message-ID:  <202302020657.3126vECA003688@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by manu:

URL: https://cgit.FreeBSD.org/src/commit/?id=a829b880ea234aa84843a9f3922053b61b42c990

commit a829b880ea234aa84843a9f3922053b61b42c990
Author:     Emmanuel Vadot <manu@FreeBSD.org>
AuthorDate: 2021-05-21 11:33:34 +0000
Commit:     Emmanuel Vadot <manu@FreeBSD.org>
CommitDate: 2023-02-02 06:47:33 +0000

    sdio: Always use increment address for read/write_4
    
    SDIO CMD53 (RW Extented) can either write to the same address (useful for FIFO)
    or auto increment the destination address (to write to multiple registers).
    It is more logical to have read/write_4 to use incremental mode and make other
    helper function for writing to a FIFO destination especially since most FIFO
    write/read will be 8bits based and not 32bits based.
    
    (cherry picked from commit 1ee7a8049218e6dc0a520e6e298626d11d254a2b)
---
 sys/dev/sdio/sdio_subr.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sys/dev/sdio/sdio_subr.c b/sys/dev/sdio/sdio_subr.c
index e2a6573abc6f..67703f88bf27 100644
--- a/sys/dev/sdio/sdio_subr.c
+++ b/sys/dev/sdio/sdio_subr.c
@@ -173,7 +173,7 @@ sdio_readl(struct sdio_func *f, uint32_t addr, int *err)
 	uint32_t v;
 
 	error = SDIO_READ_EXTENDED(device_get_parent(f->dev), f->fn, addr,
-	    sizeof(v), (uint8_t *)&v, false);
+	    sizeof(v), (uint8_t *)&v, true);
 	if (error) {
 		if (err != NULL)
 			*err = error;
@@ -191,7 +191,7 @@ sdio_writel(struct sdio_func *f, uint32_t val, uint32_t addr, int *err)
 	int error;
 
 	error = SDIO_WRITE_EXTENDED(device_get_parent(f->dev), f->fn, addr,
-	    sizeof(val), (uint8_t *)&val, false);
+	    sizeof(val), (uint8_t *)&val, true);
 	if (err != NULL)
 		*err = error;
 }



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202302020657.3126vECA003688>