Re: [PATCH 5.15 01/93] soc: sifive: ccache: Rename SiFive L2 cache to Composable cache.

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

 



On Wed, Apr 12, 2023 at 10:36:45AM +0100, Conor Dooley wrote:
> On Wed, Apr 12, 2023 at 10:33:02AM +0200, Greg Kroah-Hartman wrote:
> > From: Greentime Hu <greentime.hu@xxxxxxxxxx>
> > 
> > [ Upstream commit ca120a79cf5a3323172c82e77efd70ae10d120ef ]
> > 
> > Since composable cache may be L3 cache if there is a L2 cache, we should
> > use its original name composable cache to prevent confusion.
> > 
> > There are some new lines were generated due to adding the compatible
> > "sifive,ccache0" into ID table and indent requirement.
> > 
> > The sifive L2 has been renamed to sifive CCACHE, EDAC driver needs to
> > apply the change as well.
> > 
> > Signed-off-by: Greentime Hu <greentime.hu@xxxxxxxxxx>
> > Signed-off-by: Zong Li <zong.li@xxxxxxxxxx>
> > Co-developed-by: Zong Li <zong.li@xxxxxxxxxx>
> > Reviewed-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx>
> > Link: https://lore.kernel.org/r/20220913061817.22564-3-zong.li@xxxxxxxxxx
> > Signed-off-by: Palmer Dabbelt <palmer@xxxxxxxxxxxx>
> > Stable-dep-of: 73e770f08502 ("soc: sifive: ccache: fix missing iounmap() in error path in sifive_ccache_init()")
> > Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>
> 
> Did I just miss AUTOSEL emails for this stuff?

No, there was none as it was pulled in as a dependancy.

> NAK. This seems waay too invasive to me, and changing the Kconfig symbol
> for the driver in stable kernels sounds like a bit of a nasty surprise?
> 
> The two actual fixes that this is a dep of should be backported
> individually, please drop patches 1-7 (inclusive) & I'll give you less
> invasive backports for 6 & 7.

Ok, I'll drop all 7 now, thanks.

greg k-h



[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