Messages in this thread | | | Subject | Re: [PATCH] clk: mediatek: mt8186: Change I2C 4/5/6 ap clocks parent to infra | From | Stephen Boyd <> | Date | Mon, 23 Oct 2023 19:52:51 -0700 |
| |
Quoting Chen-Yu Tsai (2023-10-19 22:06:35) > On Thu, Oct 19, 2023 at 8:49 PM AngeloGioacchino Del Regno > <angelogioacchino.delregno@collabora.com> wrote: > > > > Fix the parenting of clocks imp_iic_wrap_ap_clock_i2c{4-6}, as those > > are effectively parented to infra_ao_i2c{4-6} and not to the I2C_AP. > > This permits the correct (and full) enablement and disablement of the > > I2C4, I2C5 and I2C6 bus clocks, satisfying the whole clock tree of > > those. > > > > As an example, when requesting to enable imp_iic_wrap_ap_clock_i2c4: > > > > Before: infra_ao_i2c_ap -> imp_iic_wrap_ap_clock_i2c4 > > After: infra_ao_i2c_ap -> infra_ao_i2c4 -> imp_iic_wrap_ap_clock_i2c4 > > > > Fixes: 66cd0b4b0ce5 ("clk: mediatek: Add MT8186 imp i2c wrapper clock support") > > Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> > > I'm curious about what led to discovering this error? >
Is that an acked-by?
| |