Re: [PATCH 5.15 00/55] 5.15.182-rc1 review

[Jon Hunter]

On 08/05/2025 12:23, Greg Kroah-Hartman wrote:
> On Thu, May 08, 2025 at 10:44:45AM +0100, Jon Hunter wrote:
> > Hi Greg,
> >
> > On 07/05/2025 19:39, Greg Kroah-Hartman wrote:
> > > This is the start of the stable review cycle for the 5.15.182 release.
> > > There are 55 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 Fri, 09 May 2025 18:37:41 +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.15.182-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.15.y
> > > and the diffstat can be found below.
> > >
> > > thanks,
> > >
> > > greg k-h
> > >
> > > -------------
> > > Pseudo-Shortlog of commits:
> >
> > ...
> > > Stephan Gerhold <stephan.gerhold@xxxxxxxxxx>
> > >       serial: msm: Configure correct working mode before starting earlycon
> >
> > The above commit is breaking the build for ARM64 and I am seeing
> > the following build error ...
> >
> > drivers/tty/serial/msm_serial.c: In function ‘msm_serial_early_console_setup_dm’:
> > drivers/tty/serial/msm_serial.c:1737:34: error: ‘MSM_UART_CR_CMD_RESET_RX’ undeclared (first use in this function); did you mean ‘UART_CR_CMD_RESET_RX’?
> >   1737 |         msm_write(&device->port, MSM_UART_CR_CMD_RESET_RX, MSM_UART_CR);
> >        |                                  ^~~~~~~~~~~~~~~~~~~~~~~~
> >        |                                  UART_CR_CMD_RESET_RX
> > drivers/tty/serial/msm_serial.c:1737:34: note: each undeclared identifier is reported only once for each function it appears in
> > drivers/tty/serial/msm_serial.c:1737:60: error: ‘MSM_UART_CR’ undeclared (first use in this function); did you mean ‘UART_CR’?
> >   1737 |         msm_write(&device->port, MSM_UART_CR_CMD_RESET_RX, MSM_UART_CR);
> >        |                                                            ^~~~~~~~~~~
> >        |                                                            UART_CR
> > drivers/tty/serial/msm_serial.c:1738:34: error: ‘MSM_UART_CR_CMD_RESET_TX’ undeclared (first use in this function); did you mean ‘UART_CR_CMD_RESET_TX’?
> >   1738 |         msm_write(&device->port, MSM_UART_CR_CMD_RESET_TX, MSM_UART_CR);
> >        |                                  ^~~~~~~~~~~~~~~~~~~~~~~~
> >        |                                  UART_CR_CMD_RESET_TX
> >    CC      drivers/ata/libata-transport.o
> > drivers/tty/serial/msm_serial.c:1739:34: error: ‘MSM_UART_CR_TX_ENABLE’ undeclared (first use in this function); did you mean ‘UART_CR_TX_ENABLE’?
> >   1739 |         msm_write(&device->port, MSM_UART_CR_TX_ENABLE, MSM_UART_CR);
> >        |                                  ^~~~~~~~~~~~~~~~~~~~~
> >        |                                  UART_CR_TX_ENABLE
> >
> >
> > After reverting this, the build is passing again.
>
> Thanks, commit is now dropped.  Odd it didn't show up on my arm64
> allmodconfig builds :(

I saw this just building the stock ARM64 defconfig.

Jon

--
nvpublic