[PATCH] arm64: dts: ti: k3-j7200: Fix OSPI boot

OSPI boot is broken due to missing bootph property in pin mux of OSPI. So add bootph to fix OSPI boot.
Signed-off-by: Udit Kumar u-kumar1@ti.com ---
This fix is already merged into kernel with[0] series. Sending this patch to allow OSPI boot, till above fix is getting merged to u-boot.
Bootlog : https://gist.github.com/uditkumarti/4c827d6801b3fde60a4902ffefd5354c
[0]: https://lore.kernel.org/all/173021674661.3859929.13298221281337904627.b4-ty@...
dts/upstream/src/arm64/ti/k3-j7200-som-p0.dtsi | 1 + 1 file changed, 1 insertion(+)
diff --git a/dts/upstream/src/arm64/ti/k3-j7200-som-p0.dtsi b/dts/upstream/src/arm64/ti/k3-j7200-som-p0.dtsi index 21fe194a57..014cf1805f 100644 --- a/dts/upstream/src/arm64/ti/k3-j7200-som-p0.dtsi +++ b/dts/upstream/src/arm64/ti/k3-j7200-som-p0.dtsi @@ -124,6 +124,7 @@ };
mcu_fss0_ospi0_pins_default: mcu-fss0-ospi0-default-pins { + bootph-all; pinctrl-single,pins = < J721E_WKUP_IOPAD(0x0000, PIN_OUTPUT, 0) /* MCU_OSPI0_CLK */ J721E_WKUP_IOPAD(0x002c, PIN_OUTPUT, 0) /* MCU_OSPI0_CSn0 */

On Tue, Nov 19, 2024 at 03:37:20PM +0530, Udit Kumar wrote:
OSPI boot is broken due to missing bootph property in pin mux of OSPI. So add bootph to fix OSPI boot.
Signed-off-by: Udit Kumar u-kumar1@ti.com
Applied to u-boot/master, thanks!

Hi Tom,
On Sat, 23 Nov 2024 at 08:12, Tom Rini trini@konsulko.com wrote:
On Tue, Nov 19, 2024 at 03:37:20PM +0530, Udit Kumar wrote:
OSPI boot is broken due to missing bootph property in pin mux of OSPI. So add bootph to fix OSPI boot.
Signed-off-by: Udit Kumar u-kumar1@ti.com
Applied to u-boot/master, thanks!
Please don't merge direct contributions to dts/upstream subtree. It should rather have been a change via *-u-boot.dtsi or a cherry pick from dts/upstream subtree. Otherwise, it is very likely it's going to cause conflicts.
-Sumit
-- Tom

On Fri, Nov 29, 2024 at 12:33:04PM +0530, Sumit Garg wrote:
Hi Tom,
On Sat, 23 Nov 2024 at 08:12, Tom Rini trini@konsulko.com wrote:
On Tue, Nov 19, 2024 at 03:37:20PM +0530, Udit Kumar wrote:
OSPI boot is broken due to missing bootph property in pin mux of OSPI. So add bootph to fix OSPI boot.
Signed-off-by: Udit Kumar u-kumar1@ti.com
Applied to u-boot/master, thanks!
Please don't merge direct contributions to dts/upstream subtree. It should rather have been a change via *-u-boot.dtsi or a cherry pick from dts/upstream subtree. Otherwise, it is very likely it's going to cause conflicts.
I thought this was a cherry-pick, sorry.

On 11/29/2024 7:06 PM, Tom Rini wrote:
On Fri, Nov 29, 2024 at 12:33:04PM +0530, Sumit Garg wrote:
Hi Tom,
On Sat, 23 Nov 2024 at 08:12, Tom Rini trini@konsulko.com wrote:
On Tue, Nov 19, 2024 at 03:37:20PM +0530, Udit Kumar wrote:
OSPI boot is broken due to missing bootph property in pin mux of OSPI. So add bootph to fix OSPI boot.
Signed-off-by: Udit Kumar u-kumar1@ti.com
Applied to u-boot/master, thanks!
Please don't merge direct contributions to dts/upstream subtree. It should rather have been a change via *-u-boot.dtsi or a cherry pick from dts/upstream subtree. Otherwise, it is very likely it's going to cause conflicts.
I thought this was a cherry-pick, sorry.
Sorry for confusion , it was not cherry-pick,
Let me know, if i need to send patch to revert above commit.
Thanks
Udit

On Fri, Nov 29, 2024 at 07:18:15PM +0530, Kumar, Udit wrote:
On 11/29/2024 7:06 PM, Tom Rini wrote:
On Fri, Nov 29, 2024 at 12:33:04PM +0530, Sumit Garg wrote:
Hi Tom,
On Sat, 23 Nov 2024 at 08:12, Tom Rini trini@konsulko.com wrote:
On Tue, Nov 19, 2024 at 03:37:20PM +0530, Udit Kumar wrote:
OSPI boot is broken due to missing bootph property in pin mux of OSPI. So add bootph to fix OSPI boot.
Signed-off-by: Udit Kumar u-kumar1@ti.com
Applied to u-boot/master, thanks!
Please don't merge direct contributions to dts/upstream subtree. It should rather have been a change via *-u-boot.dtsi or a cherry pick from dts/upstream subtree. Otherwise, it is very likely it's going to cause conflicts.
I thought this was a cherry-pick, sorry.
Sorry for confusion , it was not cherry-pick,
Let me know, if i need to send patch to revert above commit.
I'll just do the revert and push it out with other changes.

On Fri, 29 Nov 2024 at 19:18, Kumar, Udit u-kumar1@ti.com wrote:
On 11/29/2024 7:06 PM, Tom Rini wrote:
On Fri, Nov 29, 2024 at 12:33:04PM +0530, Sumit Garg wrote:
Hi Tom,
On Sat, 23 Nov 2024 at 08:12, Tom Rini trini@konsulko.com wrote:
On Tue, Nov 19, 2024 at 03:37:20PM +0530, Udit Kumar wrote:
OSPI boot is broken due to missing bootph property in pin mux of OSPI. So add bootph to fix OSPI boot.
Signed-off-by: Udit Kumar u-kumar1@ti.com
Applied to u-boot/master, thanks!
Please don't merge direct contributions to dts/upstream subtree. It should rather have been a change via *-u-boot.dtsi or a cherry pick from dts/upstream subtree. Otherwise, it is very likely it's going to cause conflicts.
I thought this was a cherry-pick, sorry.
Sorry for confusion , it was not cherry-pick,
Let me know, if i need to send patch to revert above commit.
I hope subtree automerge will take care of it for v6.12-dts uprev otherwise Tom may need to revert it and do the sync afterwards.
-Sumit
Thanks
Udit
participants (4)
-
Kumar, Udit
-
Sumit Garg
-
Tom Rini
-
Udit Kumar