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

[Daniel Díaz]

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@xxxxxxxxxxxx>
  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@xxxxxxxxxx