Re: [REGRESSION] spi: omap2-mcspi: not working with kernel v6.10

[Louis Chauvet <louis.chauvet@xxxxxxxxxxx>]

Le 18/06/24 - 09:22, Linux regression tracking (Thorsten Leemhuis) a écrit :
> On 17.06.24 21:53, João Paulo Gonçalves wrote:
> > 
> > On Mon, Jun 17, 2024 at 11:29:05AM +0200, Thorsten Leemhuis wrote:
> >> In that case the thing that really
> >> helps would be if you could use a git bisection to find the change that
> >> causes this.
> > 
> > For me, spi stopped working in commit d153ff4056cb ("spi: omap2-mcspi:
> > Add support for MULTI-mode").
> 
> Hmmm, Louis (which already was in the CC) posted a patch with a Fixes:
> tag referencing that commit more than a month ago:
> 
> spi: omap2-mcspi: Ensure that CS is correctly asserted when switching
> from MULTI mode to SINGLE mode
> https://lore.kernel.org/all/20240506-fix-omap2-mcspi-v2-1-d9c77ba8b9c7@xxxxxxxxxxx/
> 
> Unless I'm missing something, that patch did not make much progress
> since then. But I wonder if that might help. Louis? FWIW, this thread
> starts here with João problem description:
> https://lore.kernel.org/all/20240612170030.3qatttsgrwjg2m5s@joaog-nb/
> 
> Ciao, Thorsten (wearing his 'the Linux kernel's regression tracker' hat)
> --
> Everything you wanna know about Linux kernel regression tracking:
> https://linux-regtracking.leemhuis.info/about/#tldr
> If I did something stupid, please tell me, as explained on that page.

Hello everyone,

First of all, sorry for the delay, I was sick last week.

Regarding the timeout problem, it is probably solved by [1], can you try 
this? I will send a v3 to apply Miquel's comments.

For the other problem (spi not asserted properly at the end of the 
message), I think I know how to solve it, I misunderstood the 
property cs_change (which doesn't have the same meaning for the last 
transfer). I will test it tomorrow and if it works I will send a patch.

Louis Chauvet

-- 
Louis Chauvet, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com