Re: [PATCH v9 0/2] arm64: dts: qcom: add dts for sa8540p-ride board

From: Johan Hovold
Date: Wed Nov 16 2022 - 10:58:41 EST


On Wed, Nov 16, 2022 at 09:19:30PM +0530, Parikshit Pareek wrote:
> Changes in v9:
> - Enabled ufs_mem_hc/ufs_mem_phy nodes(John)

My name is Johan (below as well).

> - Corrected the indentation of 'regulator-allowed-modes' properties.

>
> Changes in v8:
> - Changed the schema of the regulators(Krzysztof)
> - Removed node remoteproc_adsp(Andrew)
> - Removed TODO comment for node usb_0_dwc3(Andrew)
> - Added Reviewed-by/Tested-by(Eric)
>
> Changes in v7:
> - Put the smpi bus related pmic changes in sseparate dtsi files(Konrad)
> - Mention allowed regulator modes via DT property regulator-allowed-modes
> (Konrad and Brian)
> - Remove unused ldo nodes vreg_l3c and vreg_l10c(Shazad)
>
> Changes in v6:
> - Introduced the new dts for the board, rather than moving common nodes
> between this one and SA8295 ADP board into dtsi file(Bjorn)
> - Drop 'adp' term to imply it being unrelated with ADP board(Internal
> discussion with Bjorn)
> - Removed Acked-by(Krzysztof) tag in dt-binding document, due to content
> change.
> - Not including Reviewed-by(Krzysztof), because of the content change.
>
> Changes in v5:
> - Moved the usb and ufs nodes from sa8540p-adp.dtsi file to respective
> board specific dts files: sa8295p-adp.dts and sa8540p-adp-ride.dts.
> Took inputs from Shazad Hussain in this regard(John)
> - Added more description of the board differences(John)
> - Not including Reviewed-by for Krzysztof, because of the new changes to
> be reviewed.
> - Removed Reported-by tag(John).
>
> Changes in v4:
> - Removed the ufs_card_hc node, as it is not mounted on Qdrive-3 board.
> - Removed usb_1 relared nodes, as usb1 doesn't have any port connected
> on Qdrive3 board.
> - Added Reported-by tag for Shazad(for ufs and usb_1 node removals)
>
> Changes in v3:
> - Added Acked-by tag (Krzysztof)
> - Renamed dtsi to sa8540p-adp.dtsi (John)
> - Removed copyright from sa8295-adp.dts and sa8295-adp.dtsi(John)
> - Added cover letter

Johan