On Wed, Mar 04, 2026 at 01:58:48PM +0100, Krzysztof Kozlowski wrote:
> Add configuration data and an entry to OF table for matching the Eliza
> SoC.
> 
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> 
> ---
> 
> Changes in v2:
> 1. Use data from manual (UBWC v5 as pointed out by Dmitry, but different

Why is it "but"?

>    highest bank bits).
> ---
>  drivers/soc/qcom/ubwc_config.c | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/drivers/soc/qcom/ubwc_config.c b/drivers/soc/qcom/ubwc_config.c
> index 1c25aaf55e52..633d1efa0883 100644
> --- a/drivers/soc/qcom/ubwc_config.c
> +++ b/drivers/soc/qcom/ubwc_config.c
> @@ -16,6 +16,16 @@ static const struct qcom_ubwc_cfg_data no_ubwc_data = {
>       /* no UBWC, no HBB */
>  };
>  
> +static const struct qcom_ubwc_cfg_data eliza_data = {
> +     .ubwc_enc_version = UBWC_5_0,
> +     .ubwc_dec_version = UBWC_5_0,
> +     .ubwc_swizzle = 6,

Please use defines for swizzle levels as the rest of the file does.

> +     .ubwc_bank_spread = true,
> +     /* TODO: highest_bank_bit = 14 for LP_DDR4 */
> +     .highest_bank_bit = 15,
> +     .macrotile_mode = true,
> +};
> +
>  static const struct qcom_ubwc_cfg_data kaanapali_data = {
>       .ubwc_enc_version = UBWC_6_0,
>       .ubwc_dec_version = UBWC_6_0,
> @@ -245,6 +255,7 @@ static const struct of_device_id qcom_ubwc_configs[] 
> __maybe_unused = {
>       { .compatible = "qcom,apq8074", .data = &no_ubwc_data },
>       { .compatible = "qcom,apq8096", .data = &msm8998_data },
>       { .compatible = "qcom,kaanapali", .data = &kaanapali_data, },
> +     { .compatible = "qcom,eliza", .data = &eliza_data, },
>       { .compatible = "qcom,glymur", .data = &glymur_data},
>       { .compatible = "qcom,msm8226", .data = &no_ubwc_data },
>       { .compatible = "qcom,msm8916", .data = &no_ubwc_data },
> 
> -- 
> 2.51.0
> 

-- 
With best wishes
Dmitry

Reply via email to