Re: [PATCH 5.4 00/47] 5.4.266-rc1 review

["Daniel Díaz" <daniel.diaz@linaro.org>]

Hello!

On 05/01/24 10:04 a. m., Daniel Díaz wrote:
> Hello!
> 
> On 05/01/24 8:38 a. m., Greg Kroah-Hartman wrote:
>> This is the start of the stable review cycle for the 5.4.266 release.
>> There are 47 patches in this series, all will be posted as a response
>> to this one.  If anyone has any issues with these being applied, please
>> let me know.
>>
>> Responses should be made by Sun, 07 Jan 2024 14:38:02 +0000.
>> Anything received after that time might be too late.
>>
>> The whole patch series can be found in one patch at:
>>     https://www.kernel.org/pub/linux/kernel/v5.x/stable-review/patch-5.4.266-rc1.gz
>> or in the git tree and branch at:
>>     git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-5.4.y
>> and the diffstat can be found below.
>>
>> thanks,
>>
>> greg k-h
> 
> Quick note that there are Arm32 failures:
> 
> -----8<-----
>    /builds/linux/drivers/bus/ti-sysc.c: In function 'sysc_reset':
>    /builds/linux/drivers/bus/ti-sysc.c:1847:4: error: implicit declaration of function 'fsleep'; did you mean 'ssleep'? [-Werror=implicit-function-declaration]
>     1847 |    fsleep(ddata->cfg.srst_udelay);
>          |    ^~~~~~
>          |    ssleep
>    cc1: some warnings being treated as errors
>    make[3]: *** [/builds/linux/scripts/Makefile.build:262: drivers/bus/ti-sysc.o] Error 1
> ----->8-----

Bisection points to:

   commit bed75624164c501b39531927d07a4fadc5738847
   Author: Julien Panis <jpanis@baylibre.com>
   Date:   Mon Aug 21 16:24:18 2023 +0200

       bus: ti-sysc: Use fsleep() instead of usleep_range() in sysc_reset()
       
       [ Upstream commit d929b2b7464f95ec01e47f560b1e687482ba8929 ]

Reverting (swapping back the content of that commit, as it wasn't a clean revert) did make the build pass.

Greetings!

Daniel Díaz
daniel.diaz@linaro.org