* [PATCH v2 0/2] Update DSS EDP integration configuration register
@ 2025-09-07 18:28 Harikrishna Shenoy
2025-09-07 18:28 ` [PATCH v2 1/2] arm64: dts: ti: k3-j721e-main: " Harikrishna Shenoy
2025-09-07 18:28 ` [PATCH v2 2/2] arm64: dts: ti: k3-j784s4-j742s2-main-common: " Harikrishna Shenoy
0 siblings, 2 replies; 12+ messages in thread
From: Harikrishna Shenoy @ 2025-09-07 18:28 UTC (permalink / raw)
To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt, r-ravikumar,
m-chawdhry, b-padhi, u-kumar1, linux-arm-kernel, devicetree,
linux-kernel, tomi.valkeinen, aradhya.bhatia, devarsht, s-jain1
This series fixes the size of EDP integration connfiguration register space
for K3 J7-family of SoC's.
Changelog v1 -> v2:
- Removed extra line from commit message.
- Added Reviewed-by tag.
- Added fix for j784s4 as well.
Link v1: https://lore.kernel.org/all/20250904050940.2913567-1-h-shenoy@ti.com
Harikrishna Shenoy (2):
arm64: dts: ti: k3-j721e-main: Update DSS EDP integration
configuration register
arm64: dts: ti: k3-j784s4-j742s2-main-common: Update DSS EDP
integration configuration register
arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 2 +-
arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
--
2.34.1
^ permalink raw reply [flat|nested] 12+ messages in thread* [PATCH v2 1/2] arm64: dts: ti: k3-j721e-main: Update DSS EDP integration configuration register 2025-09-07 18:28 [PATCH v2 0/2] Update DSS EDP integration configuration register Harikrishna Shenoy @ 2025-09-07 18:28 ` Harikrishna Shenoy 2025-09-08 4:34 ` Beleswar Prasad Padhi 2025-09-08 6:16 ` Tomi Valkeinen 2025-09-07 18:28 ` [PATCH v2 2/2] arm64: dts: ti: k3-j784s4-j742s2-main-common: " Harikrishna Shenoy 1 sibling, 2 replies; 12+ messages in thread From: Harikrishna Shenoy @ 2025-09-07 18:28 UTC (permalink / raw) To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt, r-ravikumar, m-chawdhry, b-padhi, u-kumar1, linux-arm-kernel, devicetree, linux-kernel, tomi.valkeinen, aradhya.bhatia, devarsht, s-jain1 Fix size of DSS_EDP0_INT_CFG_VP to 256B as stated in TRM Table 2-1 MAIN Domain Memory Map. Link: https://www.ti.com/lit/zip/spruil1/SPRUIL_DRA829_TDA4VM Fixes: 92c996f4ceab ("arm64: dts: ti: k3-j721e-*: add DP & DP PHY") Reviewed-by: Devarsh Thakkar <devarsht@ti.com> Signed-off-by: Harikrishna Shenoy <h-shenoy@ti.com> --- arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi index ab3666ff4297..3fa7537d5414 100644 --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi @@ -1863,7 +1863,7 @@ mhdp: dp-bridge@a000000 { * the PHY driver. */ reg = <0x00 0x0a000000 0x00 0x030a00>, /* DSS_EDP0_V2A_CORE_VP_REGS_APB */ - <0x00 0x04f40000 0x00 0x20>; /* DSS_EDP0_INTG_CFG_VP */ + <0x00 0x04f40000 0x00 0x100>; /* DSS_EDP0_INTG_CFG_VP */ reg-names = "mhdptx", "j721e-intg"; clocks = <&k3_clks 151 36>; -- 2.34.1 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH v2 1/2] arm64: dts: ti: k3-j721e-main: Update DSS EDP integration configuration register 2025-09-07 18:28 ` [PATCH v2 1/2] arm64: dts: ti: k3-j721e-main: " Harikrishna Shenoy @ 2025-09-08 4:34 ` Beleswar Prasad Padhi 2025-09-08 4:51 ` Harikrishna Shenoy 2025-09-08 6:16 ` Tomi Valkeinen 1 sibling, 1 reply; 12+ messages in thread From: Beleswar Prasad Padhi @ 2025-09-08 4:34 UTC (permalink / raw) To: Harikrishna Shenoy, nm, vigneshr, kristo, robh, krzk+dt, conor+dt, r-ravikumar, m-chawdhry, u-kumar1, linux-arm-kernel, devicetree, linux-kernel, tomi.valkeinen, aradhya.bhatia, devarsht, s-jain1 On 07/09/25 23:58, Harikrishna Shenoy wrote: > Fix size of DSS_EDP0_INT_CFG_VP to 256B as stated in > TRM Table 2-1 MAIN Domain Memory Map. > Link: https://www.ti.com/lit/zip/spruil1/SPRUIL_DRA829_TDA4VM Gives a 404 on the above link? Thanks, Beleswar > > Fixes: 92c996f4ceab ("arm64: dts: ti: k3-j721e-*: add DP & DP PHY") > Reviewed-by: Devarsh Thakkar <devarsht@ti.com> > Signed-off-by: Harikrishna Shenoy <h-shenoy@ti.com> > --- > arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > index ab3666ff4297..3fa7537d5414 100644 > --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > @@ -1863,7 +1863,7 @@ mhdp: dp-bridge@a000000 { > * the PHY driver. > */ > reg = <0x00 0x0a000000 0x00 0x030a00>, /* DSS_EDP0_V2A_CORE_VP_REGS_APB */ > - <0x00 0x04f40000 0x00 0x20>; /* DSS_EDP0_INTG_CFG_VP */ > + <0x00 0x04f40000 0x00 0x100>; /* DSS_EDP0_INTG_CFG_VP */ > reg-names = "mhdptx", "j721e-intg"; > > clocks = <&k3_clks 151 36>; ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 1/2] arm64: dts: ti: k3-j721e-main: Update DSS EDP integration configuration register 2025-09-08 4:34 ` Beleswar Prasad Padhi @ 2025-09-08 4:51 ` Harikrishna Shenoy 0 siblings, 0 replies; 12+ messages in thread From: Harikrishna Shenoy @ 2025-09-08 4:51 UTC (permalink / raw) To: Beleswar Prasad Padhi, nm, vigneshr, kristo, robh, krzk+dt, conor+dt, r-ravikumar, m-chawdhry, u-kumar1, linux-arm-kernel, devicetree, linux-kernel, tomi.valkeinen, aradhya.bhatia, devarsht, s-jain1 On 9/8/25 10:04, Beleswar Prasad Padhi wrote: > On 07/09/25 23:58, Harikrishna Shenoy wrote: >> Fix size of DSS_EDP0_INT_CFG_VP to 256B as stated in >> TRM Table 2-1 MAIN Domain Memory Map. >> Link: https://www.ti.com/lit/zip/spruil1/SPRUIL_DRA829_TDA4VM > > Gives a 404 on the above link? https://www.ti.com/lit/zip/spruil1 this downalods the zip, refer SPRUIL_DRA829_TDA4VM file. > > Thanks, > Beleswar > >> Fixes: 92c996f4ceab ("arm64: dts: ti: k3-j721e-*: add DP & DP PHY") >> Reviewed-by: Devarsh Thakkar <devarsht@ti.com> >> Signed-off-by: Harikrishna Shenoy <h-shenoy@ti.com> >> --- >> arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >> index ab3666ff4297..3fa7537d5414 100644 >> --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >> +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >> @@ -1863,7 +1863,7 @@ mhdp: dp-bridge@a000000 { >> * the PHY driver. >> */ >> reg = <0x00 0x0a000000 0x00 0x030a00>, /* DSS_EDP0_V2A_CORE_VP_REGS_APB */ >> - <0x00 0x04f40000 0x00 0x20>; /* DSS_EDP0_INTG_CFG_VP */ >> + <0x00 0x04f40000 0x00 0x100>; /* DSS_EDP0_INTG_CFG_VP */ >> reg-names = "mhdptx", "j721e-intg"; >> >> clocks = <&k3_clks 151 36>; ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 1/2] arm64: dts: ti: k3-j721e-main: Update DSS EDP integration configuration register 2025-09-07 18:28 ` [PATCH v2 1/2] arm64: dts: ti: k3-j721e-main: " Harikrishna Shenoy 2025-09-08 4:34 ` Beleswar Prasad Padhi @ 2025-09-08 6:16 ` Tomi Valkeinen 2025-09-08 6:20 ` Harikrishna Shenoy 1 sibling, 1 reply; 12+ messages in thread From: Tomi Valkeinen @ 2025-09-08 6:16 UTC (permalink / raw) To: Harikrishna Shenoy Cc: nm, vigneshr, kristo, robh, krzk+dt, conor+dt, r-ravikumar, m-chawdhry, b-padhi, u-kumar1, linux-arm-kernel, devicetree, linux-kernel, aradhya.bhatia, devarsht, s-jain1 Hi, On 07/09/2025 21:28, Harikrishna Shenoy wrote: > Fix size of DSS_EDP0_INT_CFG_VP to 256B as stated in > TRM Table 2-1 MAIN Domain Memory Map. > Link: https://www.ti.com/lit/zip/spruil1/SPRUIL_DRA829_TDA4VM What issues does this cause? Tomi > Fixes: 92c996f4ceab ("arm64: dts: ti: k3-j721e-*: add DP & DP PHY") > Reviewed-by: Devarsh Thakkar <devarsht@ti.com> > Signed-off-by: Harikrishna Shenoy <h-shenoy@ti.com> > --- > arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > index ab3666ff4297..3fa7537d5414 100644 > --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi > @@ -1863,7 +1863,7 @@ mhdp: dp-bridge@a000000 { > * the PHY driver. > */ > reg = <0x00 0x0a000000 0x00 0x030a00>, /* DSS_EDP0_V2A_CORE_VP_REGS_APB */ > - <0x00 0x04f40000 0x00 0x20>; /* DSS_EDP0_INTG_CFG_VP */ > + <0x00 0x04f40000 0x00 0x100>; /* DSS_EDP0_INTG_CFG_VP */ > reg-names = "mhdptx", "j721e-intg"; > > clocks = <&k3_clks 151 36>; ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 1/2] arm64: dts: ti: k3-j721e-main: Update DSS EDP integration configuration register 2025-09-08 6:16 ` Tomi Valkeinen @ 2025-09-08 6:20 ` Harikrishna Shenoy 2025-09-08 6:28 ` Tomi Valkeinen 0 siblings, 1 reply; 12+ messages in thread From: Harikrishna Shenoy @ 2025-09-08 6:20 UTC (permalink / raw) To: Tomi Valkeinen Cc: nm, vigneshr, kristo, robh, krzk+dt, conor+dt, r-ravikumar, m-chawdhry, b-padhi, u-kumar1, linux-arm-kernel, devicetree, linux-kernel, aradhya.bhatia, devarsht, s-jain1 On 9/8/25 11:46, Tomi Valkeinen wrote: > Hi, > > On 07/09/2025 21:28, Harikrishna Shenoy wrote: >> Fix size of DSS_EDP0_INT_CFG_VP to 256B as stated in >> TRM Table 2-1 MAIN Domain Memory Map. >> Link: https://www.ti.com/lit/zip/spruil1/SPRUIL_DRA829_TDA4VM > What issues does this cause? > > Tomi Hi Tomi, No issues seen in driver functionality yet, but it might when we enable HDCP,MST which driver supports but not enabled yet. so thought it is better to align as per TRM, hence aligning the size as per TRM. >> Fixes: 92c996f4ceab ("arm64: dts: ti: k3-j721e-*: add DP & DP PHY") >> Reviewed-by: Devarsh Thakkar <devarsht@ti.com> >> Signed-off-by: Harikrishna Shenoy <h-shenoy@ti.com> >> --- >> arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >> index ab3666ff4297..3fa7537d5414 100644 >> --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >> +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >> @@ -1863,7 +1863,7 @@ mhdp: dp-bridge@a000000 { >> * the PHY driver. >> */ >> reg = <0x00 0x0a000000 0x00 0x030a00>, /* DSS_EDP0_V2A_CORE_VP_REGS_APB */ >> - <0x00 0x04f40000 0x00 0x20>; /* DSS_EDP0_INTG_CFG_VP */ >> + <0x00 0x04f40000 0x00 0x100>; /* DSS_EDP0_INTG_CFG_VP */ >> reg-names = "mhdptx", "j721e-intg"; >> >> clocks = <&k3_clks 151 36>; ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 1/2] arm64: dts: ti: k3-j721e-main: Update DSS EDP integration configuration register 2025-09-08 6:20 ` Harikrishna Shenoy @ 2025-09-08 6:28 ` Tomi Valkeinen 2025-09-09 1:54 ` Kumar, Udit 0 siblings, 1 reply; 12+ messages in thread From: Tomi Valkeinen @ 2025-09-08 6:28 UTC (permalink / raw) To: Harikrishna Shenoy Cc: nm, vigneshr, kristo, robh, krzk+dt, conor+dt, r-ravikumar, m-chawdhry, b-padhi, u-kumar1, linux-arm-kernel, devicetree, linux-kernel, aradhya.bhatia, devarsht, s-jain1 On 08/09/2025 09:20, Harikrishna Shenoy wrote: > > On 9/8/25 11:46, Tomi Valkeinen wrote: >> Hi, >> >> On 07/09/2025 21:28, Harikrishna Shenoy wrote: >>> Fix size of DSS_EDP0_INT_CFG_VP to 256B as stated in >>> TRM Table 2-1 MAIN Domain Memory Map. >>> Link: https://www.ti.com/lit/zip/spruil1/SPRUIL_DRA829_TDA4VM >> What issues does this cause? >> >> Tomi > > Hi Tomi, > > No issues seen in driver functionality yet, but it might when we enable > HDCP,MST which driver supports but not enabled yet. > > so thought it is better to align as per TRM, hence aligning the size as > per TRM. You need to explain this in the patch message. What is the current behavior/issue, how does this fix it, what are the effects, etc... Also, here I think it's good to consider what this means for the future HDCP, MST work you refer to. Is HDCP/MST just adding driver code, no new DT properties or such needed? If so, what happens when a user runs the new code with the old dtb, which contains a too small register range? Tomi > >>> Fixes: 92c996f4ceab ("arm64: dts: ti: k3-j721e-*: add DP & DP PHY") >>> Reviewed-by: Devarsh Thakkar <devarsht@ti.com> >>> Signed-off-by: Harikrishna Shenoy <h-shenoy@ti.com> >>> --- >>> arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/ >>> boot/dts/ti/k3-j721e-main.dtsi >>> index ab3666ff4297..3fa7537d5414 100644 >>> --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >>> +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >>> @@ -1863,7 +1863,7 @@ mhdp: dp-bridge@a000000 { >>> * the PHY driver. >>> */ >>> reg = <0x00 0x0a000000 0x00 0x030a00>, /* >>> DSS_EDP0_V2A_CORE_VP_REGS_APB */ >>> - <0x00 0x04f40000 0x00 0x20>; /* >>> DSS_EDP0_INTG_CFG_VP */ >>> + <0x00 0x04f40000 0x00 0x100>; /* >>> DSS_EDP0_INTG_CFG_VP */ >>> reg-names = "mhdptx", "j721e-intg"; >>> clocks = <&k3_clks 151 36>; ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 1/2] arm64: dts: ti: k3-j721e-main: Update DSS EDP integration configuration register 2025-09-08 6:28 ` Tomi Valkeinen @ 2025-09-09 1:54 ` Kumar, Udit 0 siblings, 0 replies; 12+ messages in thread From: Kumar, Udit @ 2025-09-09 1:54 UTC (permalink / raw) To: Tomi Valkeinen, Harikrishna Shenoy Cc: nm, vigneshr, kristo, robh, krzk+dt, conor+dt, r-ravikumar, m-chawdhry, b-padhi, linux-arm-kernel, devicetree, linux-kernel, aradhya.bhatia, devarsht, s-jain1, u-kumar1 On 9/8/2025 11:58 AM, Tomi Valkeinen wrote: > On 08/09/2025 09:20, Harikrishna Shenoy wrote: >> On 9/8/25 11:46, Tomi Valkeinen wrote: >>> Hi, >>> >>> On 07/09/2025 21:28, Harikrishna Shenoy wrote: >>>> Fix size of DSS_EDP0_INT_CFG_VP to 256B as stated in >>>> TRM Table 2-1 MAIN Domain Memory Map. >>>> Link: https://www.ti.com/lit/zip/spruil1/SPRUIL_DRA829_TDA4VM >>> What issues does this cause? >>> >>> Tomi >> Hi Tomi, >> >> No issues seen in driver functionality yet, but it might when we enable >> HDCP,MST which driver supports but not enabled yet. >> >> so thought it is better to align as per TRM, hence aligning the size as >> per TRM. > You need to explain this in the patch message. What is the current > behavior/issue, how does this fix it, what are the effects, etc... > > Also, here I think it's good to consider what this means for the future > HDCP, MST work you refer to. Is HDCP/MST just adding driver code, no new > DT properties or such needed? If so, what happens when a user runs the > new code with the old dtb, which contains a too small register range? we can keep to 256 bytes for completeness, but i don't think registers are defined beyond 0x20. > > Tomi > >>>> Fixes: 92c996f4ceab ("arm64: dts: ti: k3-j721e-*: add DP & DP PHY") >>>> Reviewed-by: Devarsh Thakkar <devarsht@ti.com> >>>> Signed-off-by: Harikrishna Shenoy <h-shenoy@ti.com> >>>> --- >>>> arch/arm64/boot/dts/ti/k3-j721e-main.dtsi | 2 +- >>>> 1 file changed, 1 insertion(+), 1 deletion(-) >>>> >>>> diff --git a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi b/arch/arm64/ >>>> boot/dts/ti/k3-j721e-main.dtsi >>>> index ab3666ff4297..3fa7537d5414 100644 >>>> --- a/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >>>> +++ b/arch/arm64/boot/dts/ti/k3-j721e-main.dtsi >>>> @@ -1863,7 +1863,7 @@ mhdp: dp-bridge@a000000 { >>>> * the PHY driver. >>>> */ >>>> reg = <0x00 0x0a000000 0x00 0x030a00>, /* >>>> DSS_EDP0_V2A_CORE_VP_REGS_APB */ >>>> - <0x00 0x04f40000 0x00 0x20>; /* >>>> DSS_EDP0_INTG_CFG_VP */ >>>> + <0x00 0x04f40000 0x00 0x100>; /* >>>> DSS_EDP0_INTG_CFG_VP */ >>>> reg-names = "mhdptx", "j721e-intg"; >>>> clocks = <&k3_clks 151 36>; ^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH v2 2/2] arm64: dts: ti: k3-j784s4-j742s2-main-common: Update DSS EDP integration configuration register 2025-09-07 18:28 [PATCH v2 0/2] Update DSS EDP integration configuration register Harikrishna Shenoy 2025-09-07 18:28 ` [PATCH v2 1/2] arm64: dts: ti: k3-j721e-main: " Harikrishna Shenoy @ 2025-09-07 18:28 ` Harikrishna Shenoy 2025-09-08 4:37 ` Beleswar Prasad Padhi 1 sibling, 1 reply; 12+ messages in thread From: Harikrishna Shenoy @ 2025-09-07 18:28 UTC (permalink / raw) To: nm, vigneshr, kristo, robh, krzk+dt, conor+dt, r-ravikumar, m-chawdhry, b-padhi, u-kumar1, linux-arm-kernel, devicetree, linux-kernel, tomi.valkeinen, aradhya.bhatia, devarsht, s-jain1 Fix size of DSS_EDP0_INT_CFG_VP to 256B as stated in TRM Table 2-1 MAIN Domain Memory Map. Link:https://www.ti.com/lit/zip/spruj52/SPRUJ52-J84S4 AM69A TRM Fixes: 9cc161a4509c ("arm64: dts: ti: Refactor J784s4 SoC files to a common file") Signed-off-by: Harikrishna Shenoy <h-shenoy@ti.com> --- arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi index fbbe768e7a30..f0cda14c2530 100644 --- a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi +++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi @@ -2573,7 +2573,7 @@ port@1 { mhdp: bridge@a000000 { compatible = "ti,j721e-mhdp8546"; reg = <0x0 0xa000000 0x0 0x30a00>, - <0x0 0x4f40000 0x0 0x20>; + <0x0 0x4f40000 0x0 0x100>; reg-names = "mhdptx", "j721e-intg"; clocks = <&k3_clks 217 11>; interrupt-parent = <&gic500>; -- 2.34.1 ^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: ti: k3-j784s4-j742s2-main-common: Update DSS EDP integration configuration register 2025-09-07 18:28 ` [PATCH v2 2/2] arm64: dts: ti: k3-j784s4-j742s2-main-common: " Harikrishna Shenoy @ 2025-09-08 4:37 ` Beleswar Prasad Padhi 2025-09-08 4:50 ` Harikrishna Shenoy 0 siblings, 1 reply; 12+ messages in thread From: Beleswar Prasad Padhi @ 2025-09-08 4:37 UTC (permalink / raw) To: Harikrishna Shenoy, nm, vigneshr, kristo, robh, krzk+dt, conor+dt, r-ravikumar, m-chawdhry, u-kumar1, linux-arm-kernel, devicetree, linux-kernel, tomi.valkeinen, aradhya.bhatia, devarsht, s-jain1 On 07/09/25 23:58, Harikrishna Shenoy wrote: > Fix size of DSS_EDP0_INT_CFG_VP to 256B as stated in > TRM Table 2-1 MAIN Domain Memory Map. > Link:https://www.ti.com/lit/zip/spruj52/SPRUJ52-J84S4 AM69A TRM Broken link > > Fixes: 9cc161a4509c ("arm64: dts: ti: Refactor J784s4 SoC files to a common file") Above commit only refactored the changes and carried the bug. The commit that introduced this bug should be: 603669b16701 arm64: dts: ti: k3-j784s4-main: Add DSS and DP-bridge node Thanks, Beleswar > Signed-off-by: Harikrishna Shenoy <h-shenoy@ti.com> > --- > arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi > index fbbe768e7a30..f0cda14c2530 100644 > --- a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi > +++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi > @@ -2573,7 +2573,7 @@ port@1 { > mhdp: bridge@a000000 { > compatible = "ti,j721e-mhdp8546"; > reg = <0x0 0xa000000 0x0 0x30a00>, > - <0x0 0x4f40000 0x0 0x20>; > + <0x0 0x4f40000 0x0 0x100>; > reg-names = "mhdptx", "j721e-intg"; > clocks = <&k3_clks 217 11>; > interrupt-parent = <&gic500>; ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: ti: k3-j784s4-j742s2-main-common: Update DSS EDP integration configuration register 2025-09-08 4:37 ` Beleswar Prasad Padhi @ 2025-09-08 4:50 ` Harikrishna Shenoy 2025-09-08 4:59 ` Beleswar Prasad Padhi 0 siblings, 1 reply; 12+ messages in thread From: Harikrishna Shenoy @ 2025-09-08 4:50 UTC (permalink / raw) To: Beleswar Prasad Padhi, nm, vigneshr, kristo, robh, krzk+dt, conor+dt, r-ravikumar, m-chawdhry, u-kumar1, linux-arm-kernel, devicetree, linux-kernel, tomi.valkeinen, aradhya.bhatia, devarsht, s-jain1 On 9/8/25 10:07, Beleswar Prasad Padhi wrote: > On 07/09/25 23:58, Harikrishna Shenoy wrote: >> Fix size of DSS_EDP0_INT_CFG_VP to 256B as stated in >> TRM Table 2-1 MAIN Domain Memory Map. >> Link:https://www.ti.com/lit/zip/spruj52/SPRUJ52-J84S4 AM69A TRM > > Broken link https://www.ti.com/lit/zip/spruj52 , this link will download the zip and refer the SPRUJ52-J84S4 AM69A TRM. > >> Fixes: 9cc161a4509c ("arm64: dts: ti: Refactor J784s4 SoC files to a common file") > > Above commit only refactored the changes and carried the bug. > The commit that introduced this bug should be: > > 603669b16701 arm64: dts: ti: k3-j784s4-main: Add DSS and DP-bridge node > > Thanks, > Beleswar Noted >> Signed-off-by: Harikrishna Shenoy <h-shenoy@ti.com> >> --- >> arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi >> index fbbe768e7a30..f0cda14c2530 100644 >> --- a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi >> +++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi >> @@ -2573,7 +2573,7 @@ port@1 { >> mhdp: bridge@a000000 { >> compatible = "ti,j721e-mhdp8546"; >> reg = <0x0 0xa000000 0x0 0x30a00>, >> - <0x0 0x4f40000 0x0 0x20>; >> + <0x0 0x4f40000 0x0 0x100>; >> reg-names = "mhdptx", "j721e-intg"; >> clocks = <&k3_clks 217 11>; >> interrupt-parent = <&gic500>; ^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH v2 2/2] arm64: dts: ti: k3-j784s4-j742s2-main-common: Update DSS EDP integration configuration register 2025-09-08 4:50 ` Harikrishna Shenoy @ 2025-09-08 4:59 ` Beleswar Prasad Padhi 0 siblings, 0 replies; 12+ messages in thread From: Beleswar Prasad Padhi @ 2025-09-08 4:59 UTC (permalink / raw) To: Harikrishna Shenoy, nm, vigneshr, kristo, robh, krzk+dt, conor+dt, r-ravikumar, m-chawdhry, u-kumar1, linux-arm-kernel, devicetree, linux-kernel, tomi.valkeinen, aradhya.bhatia, devarsht, s-jain1 On 08/09/25 10:20, Harikrishna Shenoy wrote: > > On 9/8/25 10:07, Beleswar Prasad Padhi wrote: >> On 07/09/25 23:58, Harikrishna Shenoy wrote: >>> Fix size of DSS_EDP0_INT_CFG_VP to 256B as stated in >>> TRM Table 2-1 MAIN Domain Memory Map. >>> Link:https://www.ti.com/lit/zip/spruj52/SPRUJ52-J84S4 AM69A TRM >> >> Broken link > > https://www.ti.com/lit/zip/spruj52 , this link will download the zip Then maybe put that part only in the link. Otherwise it just shows 404 > and refer the SPRUJ52-J84S4 AM69A TRM. and put this in the commit message. Thanks, Beleswar > >> >>> Fixes: 9cc161a4509c ("arm64: dts: ti: Refactor J784s4 SoC files to a common file") >> >> Above commit only refactored the changes and carried the bug. >> The commit that introduced this bug should be: >> >> 603669b16701 arm64: dts: ti: k3-j784s4-main: Add DSS and DP-bridge node >> >> Thanks, >> Beleswar > Noted >>> Signed-off-by: Harikrishna Shenoy <h-shenoy@ti.com> >>> --- >>> arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi >>> index fbbe768e7a30..f0cda14c2530 100644 >>> --- a/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi >>> +++ b/arch/arm64/boot/dts/ti/k3-j784s4-j742s2-main-common.dtsi >>> @@ -2573,7 +2573,7 @@ port@1 { >>> mhdp: bridge@a000000 { >>> compatible = "ti,j721e-mhdp8546"; >>> reg = <0x0 0xa000000 0x0 0x30a00>, >>> - <0x0 0x4f40000 0x0 0x20>; >>> + <0x0 0x4f40000 0x0 0x100>; >>> reg-names = "mhdptx", "j721e-intg"; >>> clocks = <&k3_clks 217 11>; >>> interrupt-parent = <&gic500>; ^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2025-09-09 1:54 UTC | newest] Thread overview: 12+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2025-09-07 18:28 [PATCH v2 0/2] Update DSS EDP integration configuration register Harikrishna Shenoy 2025-09-07 18:28 ` [PATCH v2 1/2] arm64: dts: ti: k3-j721e-main: " Harikrishna Shenoy 2025-09-08 4:34 ` Beleswar Prasad Padhi 2025-09-08 4:51 ` Harikrishna Shenoy 2025-09-08 6:16 ` Tomi Valkeinen 2025-09-08 6:20 ` Harikrishna Shenoy 2025-09-08 6:28 ` Tomi Valkeinen 2025-09-09 1:54 ` Kumar, Udit 2025-09-07 18:28 ` [PATCH v2 2/2] arm64: dts: ti: k3-j784s4-j742s2-main-common: " Harikrishna Shenoy 2025-09-08 4:37 ` Beleswar Prasad Padhi 2025-09-08 4:50 ` Harikrishna Shenoy 2025-09-08 4:59 ` Beleswar Prasad Padhi
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).