Linux Tegra architecture development
 help / color / mirror / Atom feed
* Re: [PATCH 5.15 00/55] 5.15.182-rc1 review
       [not found] <20250507183759.048732653@linuxfoundation.org>
@ 2025-05-08  9:44 ` Jon Hunter
  2025-05-08 11:23   ` Greg Kroah-Hartman
  0 siblings, 1 reply; 3+ messages in thread
From: Jon Hunter @ 2025-05-08  9:44 UTC (permalink / raw)
  To: Greg Kroah-Hartman, stable
  Cc: patches, linux-kernel, torvalds, akpm, linux, shuah, patches,
	lkft-triage, pavel, f.fainelli, sudipm.mukherjee, srw, rwarsow,
	conor, hargar, broonie, linux-tegra@vger.kernel.org

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@linaro.org>
>      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!
Jon

-- 
nvpublic


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 5.15 00/55] 5.15.182-rc1 review
  2025-05-08  9:44 ` [PATCH 5.15 00/55] 5.15.182-rc1 review Jon Hunter
@ 2025-05-08 11:23   ` Greg Kroah-Hartman
  2025-05-08 12:22     ` Jon Hunter
  0 siblings, 1 reply; 3+ messages in thread
From: Greg Kroah-Hartman @ 2025-05-08 11:23 UTC (permalink / raw)
  To: Jon Hunter
  Cc: stable, patches, linux-kernel, torvalds, akpm, linux, shuah,
	patches, lkft-triage, pavel, f.fainelli, sudipm.mukherjee, srw,
	rwarsow, conor, hargar, broonie, linux-tegra@vger.kernel.org

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@linaro.org>
> >      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 :(

greg k-h

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH 5.15 00/55] 5.15.182-rc1 review
  2025-05-08 11:23   ` Greg Kroah-Hartman
@ 2025-05-08 12:22     ` Jon Hunter
  0 siblings, 0 replies; 3+ messages in thread
From: Jon Hunter @ 2025-05-08 12:22 UTC (permalink / raw)
  To: Greg Kroah-Hartman
  Cc: stable, patches, linux-kernel, torvalds, akpm, linux, shuah,
	patches, lkft-triage, pavel, f.fainelli, sudipm.mukherjee, srw,
	rwarsow, conor, hargar, broonie, linux-tegra@vger.kernel.org


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@linaro.org>
>>>       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


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2025-05-08 12:23 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <20250507183759.048732653@linuxfoundation.org>
2025-05-08  9:44 ` [PATCH 5.15 00/55] 5.15.182-rc1 review Jon Hunter
2025-05-08 11:23   ` Greg Kroah-Hartman
2025-05-08 12:22     ` Jon Hunter

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox