Subject: Re: [PATCH] soc: mediatek: pwrap: Remove obsolete NEED CONFIRM comments

From: AngeloGioacchino Del Regno <[email-masked]>
Date: Mon, 25 May 2026 10:46:01 +0200

On 5/22/26 15:45, Akari Tsuyukusa wrote:
> Remove the obsolete "/* NEED CONFIRM */" comments from the MT8196

s/mt8196/mt8195/g

> configuration. These values were previously confirmed during review,
> but the placeholder comments were accidentally left behind.
> 
> Link: https://lore.kernel.org/all/[email-masked]/
> Signed-off-by: Akari Tsuyukusa <[email-masked]>

after which...
Reviewed-by: AngeloGioacchino Del Regno <[email-masked]>

> ---
>   drivers/soc/mediatek/mtk-pmic-wrap.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/soc/mediatek/mtk-pmic-wrap.c b/drivers/soc/mediatek/mtk-pmic-wrap.c
> index 0bcd85826375..87bcbfa2d5f1 100644
> --- a/drivers/soc/mediatek/mtk-pmic-wrap.c
> +++ b/drivers/soc/mediatek/mtk-pmic-wrap.c
> @@ -2396,8 +2396,8 @@ static const struct pmic_wrapper_type pwrap_mt8183 = {
>   static const struct pmic_wrapper_type pwrap_mt8195 = {
>   	.regs = mt8195_regs,
>   	.type = PWRAP_MT8195,
> -	.arb_en_all = 0x777f, /* NEED CONFIRM */
> -	.int_en_all = 0x180000, /* NEED CONFIRM */
> +	.arb_en_all = 0x777f,
> +	.int_en_all = 0x180000,
>   	.int1_en_all = 0,
>   	.spi_w = PWRAP_MAN_CMD_SPI_WRITE,
>   	.wdt_src = PWRAP_WDT_SRC_MASK_ALL,