Re: [PATCH v2 2/2] clk: mediatek: reset: add infra_ao reset support for MT8188

Runyang Chen (陈润洋)
2023-05-25T07:29:05Z

On Wed, 2023-05-10 at 14:20 -0700, Stephen Boyd wrote:
> External email : Please do not click links or open attachments until
> you have verified the sender or the content.
>
>
> Quoting Runyang Chen (2023-05-05 06:13:08)
> > @@ -176,9 +177,32 @@ static const struct mtk_gate infra_ao_clks[] =
> > {
> > "infra_ao_aes_msdcfde_0p",
> > "top_aes_msdcfde", 18),
> > };
> >
> > +static u16 infra_ao_rst_ofs[] = {
>
> const?

The infra_ao_rst_ofs and infra_ao_idx_map will be modified by the reset
controller driver, so it could not be constified. And previous code did
not set it as const.

>
> > + INFRA_RST0_SET_OFFSET,
> > + INFRA_RST1_SET_OFFSET,
> > + INFRA_RST2_SET_OFFSET,
> > + INFRA_RST3_SET_OFFSET,
> > + INFRA_RST4_SET_OFFSET,
> > +};
> > +
> > +static u16 infra_ao_idx_map[] = {
>
> const?
>
> > + [MT8188_INFRA_RST1_THERMAL_MCU_RST] = 1 * RST_NR_PER_BANK +
> > 2,
> > + [MT8188_INFRA_RST1_THERMAL_CTRL_RST] = 1 * RST_NR_PER_BANK
> > + 4,
> > + [MT8188_INFRA_RST3_PTP_CTRL_RST] = 3 * RST_NR_PER_BANK + 5,
> > +};
> > +
> > +static struct mtk_clk_rst_desc infra_ao_rst_desc = {

infra_ao_rst_desc will be constified at V4.

> > + .version = MTK_RST_SET_CLR,
> > + .rst_bank_ofs = infra_ao_rst_ofs,
> > + .rst_bank_nr = ARRAY_SIZE(infra_ao_rst_ofs),
> > + .rst_idx_map = infra_ao_idx_map,
> > + .rst_idx_map_nr = ARRAY_SIZE(infra_ao_idx_map),
> > +};
> > +
> > static const struct mtk_clk_desc infra_ao_desc = {
> > .clks = infra_ao_clks,
> > .num_clks = ARRAY_SIZE(infra_ao_clks),
> > + .rst_desc = &infra_ao_rst_desc,
> > };