On Mon, Feb 13, 2023 at 03:35:15PM +0100, Konrad Dybcio wrote: > v1 -> v2: > - reorder the cleanup for better reviewability (read: redo the > thing from scratch with the only thing being shared is the idea) > > - move parent pd patches to the end > > - clean up even more duplicates (lpicx vs lpi_cx) > > - remove the vdd prefix from pd names > > - use a single tab instead of a single space in [6/9] > > - only pick up the rb on [8/9], as the rest is essentially new > > v1: https://lore.kernel.org/linux-arm-msm/20230125032728.611640-1-konrad.dybcio@xxxxxxxxxx/ > > This series provides a big and necessary cleanup for RPMPD.. > Lots of redefined objects were trimmed as well as some stylistic > and maintainability fixes have been made. Also, support for parent > PD is added and put to use on SM6375 where VDDMX should be scaled > together with VDDGX. > Series looks good, but first patch fails to apply. Could you please updated the patch you pointed out the _ao issue and rebase this on v6.3-rc1? Thanks, Bjorn > Konrad Dybcio (9): > soc: qcom: rpmpd: Bring all definitions to the top > soc: qcom: rpmpd: Unify Low Power Island definitions > soc: qcom: rpmpd: Remove vdd* from struct names > soc: qcom: rpmpd: Expand struct definition macros > soc: qcom: rpmpd: Improve the naming > soc: qcom: rpmpd: Make bindings assignments consistent > soc: qcom: rpmpd: Add parent PD support > soc: qcom: rpmpd: Hook up VDDMX as parent of SM6375 VDDGX > soc: qcom: rpmpd: Remove useless comments > > drivers/soc/qcom/rpmpd.c | 800 +++++++++++++++++++++++++++------------ > 1 file changed, 564 insertions(+), 236 deletions(-) > > -- > 2.39.1 >