On Wed, 19 Jan 2022 at 01:11, Linus Walleij <linus.walleij@xxxxxxxxxx> wrote:
>
> Convert the S3C64xx SPI host to use GPIO descriptors.
>
> Provide GPIO descriptor tables for the one user with CS
> 0 and 1.
>
> Cc: linux-samsung-soc@xxxxxxxxxxxxxxx
> Cc: Sylwester Nawrocki <snawrocki@xxxxxxxxxx>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxxxxx>
> Reviewed-by: Sam Protsenko <semen.protsenko@xxxxxxxxxx>
R-b tags for 1/3 and 2/3 seem to be lost.
> Signed-off-by: Linus Walleij <linus.walleij@xxxxxxxxxx>
> ---
> ChangeLog v2->v3:
> - Make sure to write a zero to the fb_delay if unused.
> - Drop changes to chipselect data comments.
> - Collect Review tags.
> ChangeLog v1->v2:
> - Split off code cleaning to separate patches
> ---
[snip]
> @@ -656,7 +654,11 @@ static int s3c64xx_spi_prepare_message(struct spi_master *master,
> struct s3c64xx_spi_csinfo *cs = spi->controller_data;
>
> /* Configure feedback delay */
> - writel(cs->fb_delay & 0x3, sdd->regs + S3C64XX_SPI_FB_CLK);
> + if (!cs)
> + /* No delay if not defined */
> + writel(0, sdd->regs + S3C64XX_SPI_FB_CLK);
> + else
> + writel(cs->fb_delay & 0x3, sdd->regs + S3C64XX_SPI_FB_CLK);
>
Looks good to me. I'd add {} braces and change "if (!cs)" to "if
(cs)", but that's hair splitting and not worth v4, it's fine as it is.
[snip]
