Hi Russell, I emphatically second Grant's opinion here: This is a good patch series and is a much-needed improvement. I am in the middle of a board-port for an A9 system, and this series will greatly simplify things for me *despite* the fact that I will need to convert to the new write_sec() form of L2C control on PSCI-enabled non-secure platforms. To reiterate Grant's request, please re-consider applying this L2C patch series! Thank you, jdl On Thu, May 1, 2014 at 10:12 AM, Grant Likely <grant.likely@xxxxxxxxxxxx> wrote: > On Tue, 29 Apr 2014 01:21:41 +0100, Russell King - ARM Linux <linux@xxxxxxxxxxxxxxxx> wrote: >> On Tue, Apr 29, 2014 at 09:02:27AM +0900, Simon Horman wrote: >> > On Mon, Apr 28, 2014 at 08:30:32PM +0100, Russell King wrote: >> > > Since we now automatically enable early BRESP in core L2C-310 code when >> > > we detect a Cortex-A9, we don't need platforms/SoCs to set this bit >> > > explicitly. Instead, they should seek to preserve the value of bit 30 >> > > in the auxiliary control register. >> > > >> > > Acked-by: Tony Lindgren <tony@xxxxxxxxxxx> >> > > Signed-off-by: Russell King <rmk+kernel@xxxxxxxxxxxxxxxx> >> > >> > I would prefer if this patch was broken out into individual patches >> > for each board or SoC file and that they were then picked up >> > by their respective platform maintainers. >> > >> > Likewise for patch 66/97. Although it is only for shmobile >> > I would prefer it broken out. >> >> Oh fuck that. >> >> Okay, I'm dropping the whole patch set right now and forgetting the whole >> damned thing. The L2 cache code can damned well stay as it is and remain >> an unmaintainable mess. > > FWIW, there are an awful lot of people, myself included, who do care > that you've done this work. It is 100% okay for you to say "no" to > requests to split things up because of the complexity of the series. > > I really hope you're reconsider and not give up on this series. > > g. > > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
