Re: [PATCH v2 1/2] sysreset: Implement PSCI based reset to EDL mode for QCOM SoCs

2025-05-05 Thread Casey Connolly
On 5/5/25 07:26, Varadarajan Narayanan wrote: 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

Re: [PATCH v2 1/2] sysreset: Implement PSCI based reset to EDL mode for QCOM SoCs

2025-05-04 Thread Varadarajan Narayanan
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 tec

Re: [PATCH v2 1/2] sysreset: Implement PSCI based reset to EDL mode for QCOM SoCs

2025-05-02 Thread Casey Connolly
Hi Varadarajan, You have almost entirely ignored my feedback on the previous revision. 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

Re: [PATCH v2 1/2] sysreset: Implement PSCI based reset to EDL mode for QCOM SoCs

2025-04-30 Thread neil . armstrong
On 30/04/2025 10:54, Varadarajan Narayanan wrote: The SYSRESET_PSCI interface presently available doesn't support booting into Emergency DownLoad (EDL) mode. The EDL mode is used to flash the board with different firmware. Since EDL is Qcom specific, implement a Qcom specific SYSRESET_PSCI handl

[PATCH v2 1/2] sysreset: Implement PSCI based reset to EDL mode for QCOM SoCs

2025-04-30 Thread Varadarajan Narayanan
The SYSRESET_PSCI interface presently available doesn't support booting into Emergency DownLoad (EDL) mode. The EDL mode is used to flash the board with different firmware. Since EDL is Qcom specific, implement a Qcom specific SYSRESET_PSCI handler that handles resetting to EDL in addition to the