Re: [PATCH 6.1 455/611] spi: bcm63xx: remove PM_SLEEP based conditional compilation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, May 09, 2023 at 04:54:45AM +0200, Greg Kroah-Hartman wrote:
> On Mon, May 08, 2023 at 11:27:42PM +0100, Conor Dooley wrote:
> > On Mon, May 08, 2023 at 03:25:11PM +0200, Greg Kroah-Hartman wrote:
> > > On Mon, May 08, 2023 at 03:15:15PM +0200, Greg Kroah-Hartman wrote:
> > > > On Mon, May 08, 2023 at 05:55:50PM +0530, Dhruva Gole wrote:

> > > > > > -#endif
> > > > > 
> > > > > This patch may cause build failures with some of the configs that disable
> > > > > CONFIG_PM I understand,
> > > > > So to fix that I had sent another patch:
> > > > > https://lore.kernel.org/all/CAOiHx==anPTqXNJNG7zap1XP2zKUp5SbaVJdyUsUvvitKRUHZw@xxxxxxxxxxxxxx/
> > > > > 
> > > > > However missed out adding the fixes tag.
> > > > > 
> > > > > I humbly request you to add
> > > > > https://lore.kernel.org/all/20230424102546.1604484-1-d-gole@xxxxxx/
> > > > > 
> > > > > this patch to fix this patch throughout the stable fixes trees.
> > > > > 
> > > > > It can also be found on Linus' master branch here:
> > > > > 
> > > > > https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/drivers/spi/spi-bcm63xx.c?id=cc5f6fa4f6590e3b9eb8d34302ea43af4a3cfed7
> > > > > >   static const struct dev_pm_ops bcm63xx_spi_pm_ops = {
> > > > > >   	SET_SYSTEM_SLEEP_PM_OPS(bcm63xx_spi_suspend, bcm63xx_spi_resume)
> > > > 
> > > > Sure, now queued up, thanks!
> > > 
> > > 
> > > Nope, sorry, that broke the build in many places, so I've dropped it now
> > > from kernels 5.15.y and older.
> > 
> > I retriggered my CI after seeing these mails, but the build is still
> > broken on RISC-V.
> 
> Now dropped, thanks.

All green now, thanks.

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux Kernel]     [Kernel Development Newbies]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Hiking]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux