Re: [PATCH v5 07/17] soc: qcom: minidump: Add pending region registration

2023-09-11 Thread Jeff Johnson
On 9/9/2023 1:16 PM, Mukesh Ojha wrote: +/** + * struct minidump_pregion - Minidump pending region + * @list : Pending region list pointer + * @region : APSS minidump client region does kernel-doc parse this correctly? should not be whitespace between @ID and ":" refer to

[REBASE PATCH v5 07/17] soc: qcom: minidump: Add pending region registration

2023-09-11 Thread Mukesh Ojha
Pending regions are those apss minidump regions which came for registration before minidump was initialized or ready to do register the region. We can add regions to pending region list and register all of them from apss minidump driver probe in one go. Signed-off-by: Mukesh Ojha --- drivers/so

Re: [PATCH v5 07/17] soc: qcom: minidump: Add pending region registration

2023-09-11 Thread Krzysztof Kozlowski
On 09/09/2023 22:16, Mukesh Ojha wrote: > static int qcom_apss_minidump_probe(struct platform_device *pdev) > { > struct minidump_global_toc *mdgtoc; > @@ -571,7 +688,10 @@ static int qcom_apss_minidump_probe(struct > platform_device *pdev) > return ret; > } > > +

[PATCH v5 07/17] soc: qcom: minidump: Add pending region registration

2023-09-09 Thread Mukesh Ojha
Pending regions are those apss minidump regions which came for registration before minidump was initialized or ready to do register the region. We can add regions to pending region list and register all of them from apss minidump driver probe in one go. Signed-off-by: Mukesh Ojha --- drivers/so