public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [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-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-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
                   ` (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