mbox series

[v2,0/2] Try to fix the sg_mitet bugs in SH MMCIF

Message ID 20240221-fix-sh-mmcif-v2-0-5e521eb25ae4@linaro.org
Headers
Series Try to fix the sg_mitet bugs in SH MMCIF |

Message

Linus Walleij Feb. 21, 2024, 9:22 p.m. UTC 
  This adresses some bugs found after the introduction of the
memory iterator sg_miter to the sh mmcif driver.

This was first just one patch for fixing the atomic bug, but
now also a second patch is needed to fix a semantic issue.

Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
Changes in v2:
- Collect Geerts test tag on patch 1
- Add a second patch fixing the problem with advancing to
  the next sglist entry before reading/writing the first block and
  after reading/writing each block in a multiblock operation.
- Link to v1: https://lore.kernel.org/r/20240220-fix-sh-mmcif-v1-1-b9d08a787c1f@linaro.org

---
Linus Walleij (2):
      mmc: sh_mmcif: sg_miter does not need to be atomic
      mmc: sh_mmcif: Advance sg_miter before reading blocks

 drivers/mmc/host/sh_mmcif.c | 46 +++++++++++++++++++++++++++++----------------
 1 file changed, 30 insertions(+), 16 deletions(-)
---
base-commit: 2d5c7b7eb345249cb34d42cbc2b97b4c57ea944e
change-id: 20240220-fix-sh-mmcif-49c1de70c5b7

Best regards,

Comments

Ulf Hansson Feb. 28, 2024, 1:03 p.m. UTC | #1 
On Wed, 21 Feb 2024 at 22:23, Linus Walleij <linus.walleij@linaro.org> wrote:
>
> This adresses some bugs found after the introduction of the
> memory iterator sg_miter to the sh mmcif driver.
>
> This was first just one patch for fixing the atomic bug, but
> now also a second patch is needed to fix a semantic issue.
>
> Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
> ---
> Changes in v2:
> - Collect Geerts test tag on patch 1
> - Add a second patch fixing the problem with advancing to
>   the next sglist entry before reading/writing the first block and
>   after reading/writing each block in a multiblock operation.
> - Link to v1: https://lore.kernel.org/r/20240220-fix-sh-mmcif-v1-1-b9d08a787c1f@linaro.org
>
> ---
> Linus Walleij (2):
>       mmc: sh_mmcif: sg_miter does not need to be atomic
>       mmc: sh_mmcif: Advance sg_miter before reading blocks
>
>  drivers/mmc/host/sh_mmcif.c | 46 +++++++++++++++++++++++++++++----------------
>  1 file changed, 30 insertions(+), 16 deletions(-)
> ---

Applied for next and by amending the commit message header as pointed
out by Geert, thanks!

Kind regards
Uffe