2024-05-02 10:40:38

by Shawn Sung

Subject: [PATCH v7 01/18] soc: mediatek: Disable 9-bit alpha in ETHDR

From: Hsiao Chien Sung <[email protected]>

ETHDR 9-bit alpha should be disabled by default,
otherwise alpha blending will not work.

Reviewed-by: AngeloGioacchino Del Regno <[email protected]>
Signed-off-by: Hsiao Chien Sung <[email protected]>
---
drivers/soc/mediatek/mtk-mmsys.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/soc/mediatek/mtk-mmsys.c b/drivers/soc/mediatek/mtk-mmsys.c
index f370f4ec4b888..938240714e54c 100644
--- a/drivers/soc/mediatek/mtk-mmsys.c
+++ b/drivers/soc/mediatek/mtk-mmsys.c
@@ -236,6 +236,7 @@ void mtk_mmsys_mixer_in_config(struct device *dev, int idx, bool alpha_sel, u16

mtk_mmsys_update_bits(mmsys, MT8195_VDO1_MIXER_IN1_ALPHA + (idx - 1) * 4, ~0,
alpha << 16 | alpha, cmdq_pkt);
+ mtk_mmsys_update_bits(mmsys, MT8195_VDO1_HDR_TOP_CFG, BIT(15 + idx), 0, cmdq_pkt);
mtk_mmsys_update_bits(mmsys, MT8195_VDO1_HDR_TOP_CFG, BIT(19 + idx),
alpha_sel << (19 + idx), cmdq_pkt);
mtk_mmsys_update_bits(mmsys, MT8195_VDO1_MIXER_IN1_PAD + (idx - 1) * 4,
--
2.18.0

2024-05-07 05:58:41

by CK Hu (胡俊光)

Subject: Re: [PATCH v7 01/18] soc: mediatek: Disable 9-bit alpha in ETHDR

Hi, Shawn:

On Thu, 2024-05-02 at 18:38 +0800, Shawn Sung wrote:
> From: Hsiao Chien Sung <[email protected]>
>
> ETHDR 9-bit alpha should be disabled by default,
> otherwise alpha blending will not work.

Please explain what is 9-bit alpha. Is ARGB8888 8-bit alpha? I don't
know there are 9-bit alpha. I would like to know which function would
be disabled when you disable 9-bit alpha.

Regards,
CK

>
> Reviewed-by: AngeloGioacchino Del Regno <
> [email protected]>
> Signed-off-by: Hsiao Chien Sung <[email protected]>
> ---
> drivers/soc/mediatek/mtk-mmsys.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/soc/mediatek/mtk-mmsys.c
> b/drivers/soc/mediatek/mtk-mmsys.c
> index f370f4ec4b888..938240714e54c 100644
> --- a/drivers/soc/mediatek/mtk-mmsys.c
> +++ b/drivers/soc/mediatek/mtk-mmsys.c
> @@ -236,6 +236,7 @@ void mtk_mmsys_mixer_in_config(struct device
> *dev, int idx, bool alpha_sel, u16
>
> mtk_mmsys_update_bits(mmsys, MT8195_VDO1_MIXER_IN1_ALPHA + (idx
> - 1) * 4, ~0,
> alpha << 16 | alpha, cmdq_pkt);
> + mtk_mmsys_update_bits(mmsys, MT8195_VDO1_HDR_TOP_CFG, BIT(15 +
> idx), 0, cmdq_pkt);
> mtk_mmsys_update_bits(mmsys, MT8195_VDO1_HDR_TOP_CFG, BIT(19 +
> idx),
> alpha_sel << (19 + idx), cmdq_pkt);
> mtk_mmsys_update_bits(mmsys, MT8195_VDO1_MIXER_IN1_PAD + (idx -
> 1) * 4,

2024-05-07 10:02:40

by Shawn Sung

Subject: Re: [PATCH v7 01/18] soc: mediatek: Disable 9-bit alpha in ETHDR

Hi CK,

On Tue, 2024-05-07 at 05:58 +0000, CK Hu (胡俊光) wrote:
> Hi, Shawn:
>
> On Thu, 2024-05-02 at 18:38 +0800, Shawn Sung wrote:
> > From: Hsiao Chien Sung <[email protected]>
> >
> > ETHDR 9-bit alpha should be disabled by default,
> > otherwise alpha blending will not work.
>
> Please explain what is 9-bit alpha. Is ARGB8888 8-bit alpha? I don't
> know there are 9-bit alpha. I would like to know which function would
> be disabled when you disable 9-bit alpha.
>

When 9-bit alpha is enabled, the original alpha range 0-255 will be
remapped to 0-256 (255 = N/A) for some special case calculation.
Checked with the designer, this setting should be always disabled and
there will be no side effects.

Thanks,
Shawn