On Fri, May 02, 2025 at 02:32:52PM +0200, Casey Connolly wrote: > Hi Varadarajan, > > You have almost entirely ignored my feedback on the previous revision.
Sorry. Since you had said "For now I'm fine with this new sysreset-qcom.c driver", in [1], I assumed you were ok with just addressing the technical issues alone. Will add a new op to sysreset_ops and post a new version. Thanks Varada 1 - https://lore.kernel.org/u-boot/1e0e9a72-40a8-4ce3-a390-edef1d6a0...@linaro.org/ > If there's some technical disagreement or you don't understand my feedback > then please let me know (in private if you prefer). > > This approach is fundamentally wrong, full of layering violations and > totally unsustainable (imagine if every vendor started polluting core code > with their custom stuff). > > These abstractions need to be improved to add this feature properly and > there's no getting around it. > > Kind regards, > Casey (she/they) > [ . . . ]
