* [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform
@ 2026-02-28 18:34 Dmitry Baryshkov
2026-02-28 18:48 ` Abel Vesa
` (4 more replies)
0 siblings, 5 replies; 11+ messages in thread
From: Dmitry Baryshkov @ 2026-02-28 18:34 UTC (permalink / raw)
To: Bjorn Andersson, Konrad Dybcio, Rob Clark, Akhil P Oommen,
Abel Vesa
Cc: linux-arm-msm, linux-kernel
Due to the way the DDR controller is organized on Glymur, hardware
engineers strongly recommended disabling UBWC bank swizzling on Glymur.
Follow that recommendation.
Fixes: 9b21c3bd2480 ("soc: qcom: ubwc: Add configuration Glymur platform")
Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com>
---
Changes in v2:
- Fix the syntax error...
- Link to v1: https://lore.kernel.org/r/20260228-fix-glymur-ubwc-v1-1-d80e3fe0dcc7@oss.qualcomm.com
---
drivers/soc/qcom/ubwc_config.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/soc/qcom/ubwc_config.c b/drivers/soc/qcom/ubwc_config.c
index 1c25aaf55e52..8304463f238a 100644
--- a/drivers/soc/qcom/ubwc_config.c
+++ b/drivers/soc/qcom/ubwc_config.c
@@ -231,8 +231,7 @@ static const struct qcom_ubwc_cfg_data x1e80100_data = {
static const struct qcom_ubwc_cfg_data glymur_data = {
.ubwc_enc_version = UBWC_5_0,
.ubwc_dec_version = UBWC_5_0,
- .ubwc_swizzle = UBWC_SWIZZLE_ENABLE_LVL2 |
- UBWC_SWIZZLE_ENABLE_LVL3,
+ .ubwc_swizzle = 0,
.ubwc_bank_spread = true,
/* TODO: highest_bank_bit = 15 for LP_DDR4 */
.highest_bank_bit = 16,
---
base-commit: 6de23f81a5e08be8fbf5e8d7e9febc72a5b5f27f
change-id: 20260228-fix-glymur-ubwc-f673d5ca0581
Best regards,
--
With best wishes
Dmitry
^ permalink raw reply related [flat|nested] 11+ messages in thread* Re: [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform 2026-02-28 18:34 [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform Dmitry Baryshkov @ 2026-02-28 18:48 ` Abel Vesa 2026-03-02 10:53 ` Konrad Dybcio ` (3 subsequent siblings) 4 siblings, 0 replies; 11+ messages in thread From: Abel Vesa @ 2026-02-28 18:48 UTC (permalink / raw) To: Dmitry Baryshkov Cc: Bjorn Andersson, Konrad Dybcio, Rob Clark, Akhil P Oommen, Abel Vesa, linux-arm-msm, linux-kernel On 26-02-28 20:34:27, Dmitry Baryshkov wrote: > Due to the way the DDR controller is organized on Glymur, hardware > engineers strongly recommended disabling UBWC bank swizzling on Glymur. > Follow that recommendation. > > Fixes: 9b21c3bd2480 ("soc: qcom: ubwc: Add configuration Glymur platform") > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> Reviewed-by: Abel Vesa <abel.vesa@oss.qualcomm.com> ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform 2026-02-28 18:34 [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform Dmitry Baryshkov 2026-02-28 18:48 ` Abel Vesa @ 2026-03-02 10:53 ` Konrad Dybcio 2026-03-06 15:15 ` Dmitry Baryshkov ` (2 subsequent siblings) 4 siblings, 0 replies; 11+ messages in thread From: Konrad Dybcio @ 2026-03-02 10:53 UTC (permalink / raw) To: Dmitry Baryshkov, Bjorn Andersson, Konrad Dybcio, Rob Clark, Akhil P Oommen, Abel Vesa Cc: linux-arm-msm, linux-kernel On 2/28/26 7:34 PM, Dmitry Baryshkov wrote: > Due to the way the DDR controller is organized on Glymur, hardware > engineers strongly recommended disabling UBWC bank swizzling on Glymur. > Follow that recommendation. > > Fixes: 9b21c3bd2480 ("soc: qcom: ubwc: Add configuration Glymur platform") > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > --- Reviewed-by: Konrad Dybcio <konrad.dybcio@oss.qualcomm.com> Konrad ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform 2026-02-28 18:34 [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform Dmitry Baryshkov 2026-02-28 18:48 ` Abel Vesa 2026-03-02 10:53 ` Konrad Dybcio @ 2026-03-06 15:15 ` Dmitry Baryshkov 2026-03-06 16:07 ` Akhil P Oommen 2026-03-16 1:41 ` Bjorn Andersson 2026-03-06 22:19 ` Rob Clark 2026-03-23 14:38 ` Bjorn Andersson 4 siblings, 2 replies; 11+ messages in thread From: Dmitry Baryshkov @ 2026-03-06 15:15 UTC (permalink / raw) To: Bjorn Andersson, Konrad Dybcio, Rob Clark, Akhil P Oommen, Abel Vesa Cc: linux-arm-msm, linux-kernel On Sat, Feb 28, 2026 at 08:34:27PM +0200, Dmitry Baryshkov wrote: > Due to the way the DDR controller is organized on Glymur, hardware > engineers strongly recommended disabling UBWC bank swizzling on Glymur. > Follow that recommendation. > > Fixes: 9b21c3bd2480 ("soc: qcom: ubwc: Add configuration Glymur platform") > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > --- > Changes in v2: > - Fix the syntax error... > - Link to v1: https://lore.kernel.org/r/20260228-fix-glymur-ubwc-v1-1-d80e3fe0dcc7@oss.qualcomm.com > --- > drivers/soc/qcom/ubwc_config.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/soc/qcom/ubwc_config.c b/drivers/soc/qcom/ubwc_config.c > index 1c25aaf55e52..8304463f238a 100644 > --- a/drivers/soc/qcom/ubwc_config.c > +++ b/drivers/soc/qcom/ubwc_config.c > @@ -231,8 +231,7 @@ static const struct qcom_ubwc_cfg_data x1e80100_data = { > static const struct qcom_ubwc_cfg_data glymur_data = { > .ubwc_enc_version = UBWC_5_0, > .ubwc_dec_version = UBWC_5_0, > - .ubwc_swizzle = UBWC_SWIZZLE_ENABLE_LVL2 | > - UBWC_SWIZZLE_ENABLE_LVL3, > + .ubwc_swizzle = 0, > .ubwc_bank_spread = true, > /* TODO: highest_bank_bit = 15 for LP_DDR4 */ > .highest_bank_bit = 16, Carrying over from v1 discussion. Reviewed-by: Akhil P Oommen <quic_akhilpo@quicinc.com> It depends on the fix which is currently a part of msm-fixes for the device to function correctly. Raised the question on IRC regarding the immutable tag or any other proper way to merge it. > > --- > base-commit: 6de23f81a5e08be8fbf5e8d7e9febc72a5b5f27f > change-id: 20260228-fix-glymur-ubwc-f673d5ca0581 > > Best regards, > -- > With best wishes > Dmitry > -- With best wishes Dmitry ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform 2026-03-06 15:15 ` Dmitry Baryshkov @ 2026-03-06 16:07 ` Akhil P Oommen 2026-03-16 1:41 ` Bjorn Andersson 1 sibling, 0 replies; 11+ messages in thread From: Akhil P Oommen @ 2026-03-06 16:07 UTC (permalink / raw) To: Dmitry Baryshkov, Bjorn Andersson, Konrad Dybcio, Rob Clark, Abel Vesa Cc: linux-arm-msm, linux-kernel On 3/6/2026 8:45 PM, Dmitry Baryshkov wrote: > On Sat, Feb 28, 2026 at 08:34:27PM +0200, Dmitry Baryshkov wrote: >> Due to the way the DDR controller is organized on Glymur, hardware >> engineers strongly recommended disabling UBWC bank swizzling on Glymur. >> Follow that recommendation. >> >> Fixes: 9b21c3bd2480 ("soc: qcom: ubwc: Add configuration Glymur platform") >> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> >> --- >> Changes in v2: >> - Fix the syntax error... >> - Link to v1: https://lore.kernel.org/r/20260228-fix-glymur-ubwc-v1-1-d80e3fe0dcc7@oss.qualcomm.com >> --- >> drivers/soc/qcom/ubwc_config.c | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/drivers/soc/qcom/ubwc_config.c b/drivers/soc/qcom/ubwc_config.c >> index 1c25aaf55e52..8304463f238a 100644 >> --- a/drivers/soc/qcom/ubwc_config.c >> +++ b/drivers/soc/qcom/ubwc_config.c >> @@ -231,8 +231,7 @@ static const struct qcom_ubwc_cfg_data x1e80100_data = { >> static const struct qcom_ubwc_cfg_data glymur_data = { >> .ubwc_enc_version = UBWC_5_0, >> .ubwc_dec_version = UBWC_5_0, >> - .ubwc_swizzle = UBWC_SWIZZLE_ENABLE_LVL2 | >> - UBWC_SWIZZLE_ENABLE_LVL3, >> + .ubwc_swizzle = 0, >> .ubwc_bank_spread = true, >> /* TODO: highest_bank_bit = 15 for LP_DDR4 */ >> .highest_bank_bit = 16, > > Carrying over from v1 discussion. > > Reviewed-by: Akhil P Oommen <quic_akhilpo@quicinc.com> Sorry, please use my oss email instead: Reviewed-by: Akhil P Oommen <akhilpo@oss.qualcomm.com> -Akhil. > > It depends on the fix which is currently a part of msm-fixes for the > device to function correctly. Raised the question on IRC regarding the > immutable tag or any other proper way to merge it. > >> >> --- >> base-commit: 6de23f81a5e08be8fbf5e8d7e9febc72a5b5f27f >> change-id: 20260228-fix-glymur-ubwc-f673d5ca0581 >> >> Best regards, >> -- >> With best wishes >> Dmitry >> > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform 2026-03-06 15:15 ` Dmitry Baryshkov 2026-03-06 16:07 ` Akhil P Oommen @ 2026-03-16 1:41 ` Bjorn Andersson 2026-03-16 2:51 ` Dmitry Baryshkov 1 sibling, 1 reply; 11+ messages in thread From: Bjorn Andersson @ 2026-03-16 1:41 UTC (permalink / raw) To: Dmitry Baryshkov Cc: Konrad Dybcio, Rob Clark, Akhil P Oommen, Abel Vesa, linux-arm-msm, linux-kernel On Fri, Mar 06, 2026 at 05:15:32PM +0200, Dmitry Baryshkov wrote: > On Sat, Feb 28, 2026 at 08:34:27PM +0200, Dmitry Baryshkov wrote: > > Due to the way the DDR controller is organized on Glymur, hardware > > engineers strongly recommended disabling UBWC bank swizzling on Glymur. > > Follow that recommendation. > > > > Fixes: 9b21c3bd2480 ("soc: qcom: ubwc: Add configuration Glymur platform") > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > > --- > > Changes in v2: > > - Fix the syntax error... > > - Link to v1: https://lore.kernel.org/r/20260228-fix-glymur-ubwc-v1-1-d80e3fe0dcc7@oss.qualcomm.com > > --- > > drivers/soc/qcom/ubwc_config.c | 3 +-- > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > diff --git a/drivers/soc/qcom/ubwc_config.c b/drivers/soc/qcom/ubwc_config.c > > index 1c25aaf55e52..8304463f238a 100644 > > --- a/drivers/soc/qcom/ubwc_config.c > > +++ b/drivers/soc/qcom/ubwc_config.c > > @@ -231,8 +231,7 @@ static const struct qcom_ubwc_cfg_data x1e80100_data = { > > static const struct qcom_ubwc_cfg_data glymur_data = { > > .ubwc_enc_version = UBWC_5_0, > > .ubwc_dec_version = UBWC_5_0, > > - .ubwc_swizzle = UBWC_SWIZZLE_ENABLE_LVL2 | > > - UBWC_SWIZZLE_ENABLE_LVL3, > > + .ubwc_swizzle = 0, > > .ubwc_bank_spread = true, > > /* TODO: highest_bank_bit = 15 for LP_DDR4 */ > > .highest_bank_bit = 16, > > Carrying over from v1 discussion. > > Reviewed-by: Akhil P Oommen <quic_akhilpo@quicinc.com> > > It depends on the fix which is currently a part of msm-fixes for the > device to function correctly. Raised the question on IRC regarding the > immutable tag or any other proper way to merge it. > Sorry, I did see your question on IRC, but didn't follow up and forgot to ask about it. What do you mean with "depends on"? Why do we need an immutable tag? Regards, Bjorn > > > > --- > > base-commit: 6de23f81a5e08be8fbf5e8d7e9febc72a5b5f27f > > change-id: 20260228-fix-glymur-ubwc-f673d5ca0581 > > > > Best regards, > > -- > > With best wishes > > Dmitry > > > > -- > With best wishes > Dmitry ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform 2026-03-16 1:41 ` Bjorn Andersson @ 2026-03-16 2:51 ` Dmitry Baryshkov 2026-03-19 3:23 ` Bjorn Andersson 0 siblings, 1 reply; 11+ messages in thread From: Dmitry Baryshkov @ 2026-03-16 2:51 UTC (permalink / raw) To: Bjorn Andersson Cc: Konrad Dybcio, Rob Clark, Akhil P Oommen, Abel Vesa, linux-arm-msm, linux-kernel On Sun, Mar 15, 2026 at 08:41:02PM -0500, Bjorn Andersson wrote: > On Fri, Mar 06, 2026 at 05:15:32PM +0200, Dmitry Baryshkov wrote: > > On Sat, Feb 28, 2026 at 08:34:27PM +0200, Dmitry Baryshkov wrote: > > > Due to the way the DDR controller is organized on Glymur, hardware > > > engineers strongly recommended disabling UBWC bank swizzling on Glymur. > > > Follow that recommendation. > > > > > > Fixes: 9b21c3bd2480 ("soc: qcom: ubwc: Add configuration Glymur platform") > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > > > --- > > > Changes in v2: > > > - Fix the syntax error... > > > - Link to v1: https://lore.kernel.org/r/20260228-fix-glymur-ubwc-v1-1-d80e3fe0dcc7@oss.qualcomm.com > > > --- > > > drivers/soc/qcom/ubwc_config.c | 3 +-- > > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > > > diff --git a/drivers/soc/qcom/ubwc_config.c b/drivers/soc/qcom/ubwc_config.c > > > index 1c25aaf55e52..8304463f238a 100644 > > > --- a/drivers/soc/qcom/ubwc_config.c > > > +++ b/drivers/soc/qcom/ubwc_config.c > > > @@ -231,8 +231,7 @@ static const struct qcom_ubwc_cfg_data x1e80100_data = { > > > static const struct qcom_ubwc_cfg_data glymur_data = { > > > .ubwc_enc_version = UBWC_5_0, > > > .ubwc_dec_version = UBWC_5_0, > > > - .ubwc_swizzle = UBWC_SWIZZLE_ENABLE_LVL2 | > > > - UBWC_SWIZZLE_ENABLE_LVL3, > > > + .ubwc_swizzle = 0, > > > .ubwc_bank_spread = true, > > > /* TODO: highest_bank_bit = 15 for LP_DDR4 */ > > > .highest_bank_bit = 16, > > > > Carrying over from v1 discussion. > > > > Reviewed-by: Akhil P Oommen <quic_akhilpo@quicinc.com> > > > > It depends on the fix which is currently a part of msm-fixes for the > > device to function correctly. Raised the question on IRC regarding the > > immutable tag or any other proper way to merge it. > > > > Sorry, I did see your question on IRC, but didn't follow up and forgot > to ask about it. > > What do you mean with "depends on"? Why do we need an immutable tag? Disabling LVL2 / LVL3 swizzling on the GPU side requires the patch from msm-fixes. Otherwise the GPU driver ignores passed swizzle, making the hardware use incorrect memory layout in case of Glymur (A8xx). -- With best wishes Dmitry ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform 2026-03-16 2:51 ` Dmitry Baryshkov @ 2026-03-19 3:23 ` Bjorn Andersson 2026-03-19 3:50 ` Dmitry Baryshkov 0 siblings, 1 reply; 11+ messages in thread From: Bjorn Andersson @ 2026-03-19 3:23 UTC (permalink / raw) To: Dmitry Baryshkov Cc: Konrad Dybcio, Rob Clark, Akhil P Oommen, Abel Vesa, linux-arm-msm, linux-kernel On Mon, Mar 16, 2026 at 04:51:46AM +0200, Dmitry Baryshkov wrote: > On Sun, Mar 15, 2026 at 08:41:02PM -0500, Bjorn Andersson wrote: > > On Fri, Mar 06, 2026 at 05:15:32PM +0200, Dmitry Baryshkov wrote: > > > On Sat, Feb 28, 2026 at 08:34:27PM +0200, Dmitry Baryshkov wrote: > > > > Due to the way the DDR controller is organized on Glymur, hardware > > > > engineers strongly recommended disabling UBWC bank swizzling on Glymur. > > > > Follow that recommendation. > > > > > > > > Fixes: 9b21c3bd2480 ("soc: qcom: ubwc: Add configuration Glymur platform") > > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > > > > --- > > > > Changes in v2: > > > > - Fix the syntax error... > > > > - Link to v1: https://lore.kernel.org/r/20260228-fix-glymur-ubwc-v1-1-d80e3fe0dcc7@oss.qualcomm.com > > > > --- > > > > drivers/soc/qcom/ubwc_config.c | 3 +-- > > > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > > > > > diff --git a/drivers/soc/qcom/ubwc_config.c b/drivers/soc/qcom/ubwc_config.c > > > > index 1c25aaf55e52..8304463f238a 100644 > > > > --- a/drivers/soc/qcom/ubwc_config.c > > > > +++ b/drivers/soc/qcom/ubwc_config.c > > > > @@ -231,8 +231,7 @@ static const struct qcom_ubwc_cfg_data x1e80100_data = { > > > > static const struct qcom_ubwc_cfg_data glymur_data = { > > > > .ubwc_enc_version = UBWC_5_0, > > > > .ubwc_dec_version = UBWC_5_0, > > > > - .ubwc_swizzle = UBWC_SWIZZLE_ENABLE_LVL2 | > > > > - UBWC_SWIZZLE_ENABLE_LVL3, > > > > + .ubwc_swizzle = 0, > > > > .ubwc_bank_spread = true, > > > > /* TODO: highest_bank_bit = 15 for LP_DDR4 */ > > > > .highest_bank_bit = 16, > > > > > > Carrying over from v1 discussion. > > > > > > Reviewed-by: Akhil P Oommen <quic_akhilpo@quicinc.com> > > > > > > It depends on the fix which is currently a part of msm-fixes for the > > > device to function correctly. Raised the question on IRC regarding the > > > immutable tag or any other proper way to merge it. > > > > > > > Sorry, I did see your question on IRC, but didn't follow up and forgot > > to ask about it. > > > > What do you mean with "depends on"? Why do we need an immutable tag? > > Disabling LVL2 / LVL3 swizzling on the GPU side requires the patch from > msm-fixes. Otherwise the GPU driver ignores passed swizzle, making the > hardware use incorrect memory layout in case of Glymur (A8xx). > So this patch must appear after the change in msm-fixes? If you have already gotten the msm-fixes merged upstream, in which case ordering is already achieved, without the immutable branch... If you haven't, the immutable branch is not going to help, as it doesn't guarantee that the immutable changes arrives in torvalds/master only after the msm-fixes. That said, this seems somewhat theoretical, as there's no GPU enabled in the upstream DTSI afaict. Regards, Bjorn > -- > With best wishes > Dmitry ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform 2026-03-19 3:23 ` Bjorn Andersson @ 2026-03-19 3:50 ` Dmitry Baryshkov 0 siblings, 0 replies; 11+ messages in thread From: Dmitry Baryshkov @ 2026-03-19 3:50 UTC (permalink / raw) To: Bjorn Andersson Cc: Konrad Dybcio, Rob Clark, Akhil P Oommen, Abel Vesa, linux-arm-msm, linux-kernel On Wed, Mar 18, 2026 at 10:23:15PM -0500, Bjorn Andersson wrote: > On Mon, Mar 16, 2026 at 04:51:46AM +0200, Dmitry Baryshkov wrote: > > On Sun, Mar 15, 2026 at 08:41:02PM -0500, Bjorn Andersson wrote: > > > On Fri, Mar 06, 2026 at 05:15:32PM +0200, Dmitry Baryshkov wrote: > > > > On Sat, Feb 28, 2026 at 08:34:27PM +0200, Dmitry Baryshkov wrote: > > > > > Due to the way the DDR controller is organized on Glymur, hardware > > > > > engineers strongly recommended disabling UBWC bank swizzling on Glymur. > > > > > Follow that recommendation. > > > > > > > > > > Fixes: 9b21c3bd2480 ("soc: qcom: ubwc: Add configuration Glymur platform") > > > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > > > > > --- > > > > > Changes in v2: > > > > > - Fix the syntax error... > > > > > - Link to v1: https://lore.kernel.org/r/20260228-fix-glymur-ubwc-v1-1-d80e3fe0dcc7@oss.qualcomm.com > > > > > --- > > > > > drivers/soc/qcom/ubwc_config.c | 3 +-- > > > > > 1 file changed, 1 insertion(+), 2 deletions(-) > > > > > > > > > > diff --git a/drivers/soc/qcom/ubwc_config.c b/drivers/soc/qcom/ubwc_config.c > > > > > index 1c25aaf55e52..8304463f238a 100644 > > > > > --- a/drivers/soc/qcom/ubwc_config.c > > > > > +++ b/drivers/soc/qcom/ubwc_config.c > > > > > @@ -231,8 +231,7 @@ static const struct qcom_ubwc_cfg_data x1e80100_data = { > > > > > static const struct qcom_ubwc_cfg_data glymur_data = { > > > > > .ubwc_enc_version = UBWC_5_0, > > > > > .ubwc_dec_version = UBWC_5_0, > > > > > - .ubwc_swizzle = UBWC_SWIZZLE_ENABLE_LVL2 | > > > > > - UBWC_SWIZZLE_ENABLE_LVL3, > > > > > + .ubwc_swizzle = 0, > > > > > .ubwc_bank_spread = true, > > > > > /* TODO: highest_bank_bit = 15 for LP_DDR4 */ > > > > > .highest_bank_bit = 16, > > > > > > > > Carrying over from v1 discussion. > > > > > > > > Reviewed-by: Akhil P Oommen <quic_akhilpo@quicinc.com> > > > > > > > > It depends on the fix which is currently a part of msm-fixes for the > > > > device to function correctly. Raised the question on IRC regarding the > > > > immutable tag or any other proper way to merge it. > > > > > > > > > > Sorry, I did see your question on IRC, but didn't follow up and forgot > > > to ask about it. > > > > > > What do you mean with "depends on"? Why do we need an immutable tag? > > > > Disabling LVL2 / LVL3 swizzling on the GPU side requires the patch from > > msm-fixes. Otherwise the GPU driver ignores passed swizzle, making the > > hardware use incorrect memory layout in case of Glymur (A8xx). > > > > So this patch must appear after the change in msm-fixes? > > If you have already gotten the msm-fixes merged upstream, in which case > ordering is already achieved, without the immutable branch... Yes, it is merged. However I'm not sure, which branch you are basing upon. > If you haven't, the immutable branch is not going to help, as it doesn't > guarantee that the immutable changes arrives in torvalds/master only > after the msm-fixes. > > That said, this seems somewhat theoretical, as there's no GPU enabled in > the upstream DTSI afaict. Ack. -- With best wishes Dmitry ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform 2026-02-28 18:34 [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform Dmitry Baryshkov ` (2 preceding siblings ...) 2026-03-06 15:15 ` Dmitry Baryshkov @ 2026-03-06 22:19 ` Rob Clark 2026-03-23 14:38 ` Bjorn Andersson 4 siblings, 0 replies; 11+ messages in thread From: Rob Clark @ 2026-03-06 22:19 UTC (permalink / raw) To: Dmitry Baryshkov Cc: Bjorn Andersson, Konrad Dybcio, Akhil P Oommen, Abel Vesa, linux-arm-msm, linux-kernel On Sat, Feb 28, 2026 at 10:34 AM Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> wrote: > > Due to the way the DDR controller is organized on Glymur, hardware > engineers strongly recommended disabling UBWC bank swizzling on Glymur. > Follow that recommendation. > > Fixes: 9b21c3bd2480 ("soc: qcom: ubwc: Add configuration Glymur platform") > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@oss.qualcomm.com> > --- > Changes in v2: > - Fix the syntax error... > - Link to v1: https://lore.kernel.org/r/20260228-fix-glymur-ubwc-v1-1-d80e3fe0dcc7@oss.qualcomm.com > --- > drivers/soc/qcom/ubwc_config.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > > diff --git a/drivers/soc/qcom/ubwc_config.c b/drivers/soc/qcom/ubwc_config.c > index 1c25aaf55e52..8304463f238a 100644 > --- a/drivers/soc/qcom/ubwc_config.c > +++ b/drivers/soc/qcom/ubwc_config.c > @@ -231,8 +231,7 @@ static const struct qcom_ubwc_cfg_data x1e80100_data = { > static const struct qcom_ubwc_cfg_data glymur_data = { > .ubwc_enc_version = UBWC_5_0, > .ubwc_dec_version = UBWC_5_0, > - .ubwc_swizzle = UBWC_SWIZZLE_ENABLE_LVL2 | > - UBWC_SWIZZLE_ENABLE_LVL3, > + .ubwc_swizzle = 0, > .ubwc_bank_spread = true, > /* TODO: highest_bank_bit = 15 for LP_DDR4 */ > .highest_bank_bit = 16, dEQP-VK.image.host_image_copy.* is happy with this plus Akhil's patch (drm/msm/a8xx: Fix ubwc config related to swizzling) in msm-fixes Acked-by: Rob Clark <rob.clark@oss.qualcomm.com> > --- > base-commit: 6de23f81a5e08be8fbf5e8d7e9febc72a5b5f27f > change-id: 20260228-fix-glymur-ubwc-f673d5ca0581 > > Best regards, > -- > With best wishes > Dmitry > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform 2026-02-28 18:34 [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform Dmitry Baryshkov ` (3 preceding siblings ...) 2026-03-06 22:19 ` Rob Clark @ 2026-03-23 14:38 ` Bjorn Andersson 4 siblings, 0 replies; 11+ messages in thread From: Bjorn Andersson @ 2026-03-23 14:38 UTC (permalink / raw) To: Konrad Dybcio, Rob Clark, Akhil P Oommen, Abel Vesa, Dmitry Baryshkov Cc: linux-arm-msm, linux-kernel On Sat, 28 Feb 2026 20:34:27 +0200, Dmitry Baryshkov wrote: > Due to the way the DDR controller is organized on Glymur, hardware > engineers strongly recommended disabling UBWC bank swizzling on Glymur. > Follow that recommendation. > > Applied, thanks! [1/1] soc: qcom: ubwc: disable bank swizzling for Glymur platform commit: e031e7ceac4ee04973bd77362c363734e79dd08c Best regards, -- Bjorn Andersson <andersson@kernel.org> ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2026-03-23 14:38 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2026-02-28 18:34 [PATCH v2] soc: qcom: ubwc: disable bank swizzling for Glymur platform Dmitry Baryshkov 2026-02-28 18:48 ` Abel Vesa 2026-03-02 10:53 ` Konrad Dybcio 2026-03-06 15:15 ` Dmitry Baryshkov 2026-03-06 16:07 ` Akhil P Oommen 2026-03-16 1:41 ` Bjorn Andersson 2026-03-16 2:51 ` Dmitry Baryshkov 2026-03-19 3:23 ` Bjorn Andersson 2026-03-19 3:50 ` Dmitry Baryshkov 2026-03-06 22:19 ` Rob Clark 2026-03-23 14:38 ` Bjorn Andersson
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox