Hi Krzysztof,
On Thu, Mar 07, 2024 at 03:05:50PM +0100, Krzysztof Kozlowski wrote:
> On 07/03/2024 15:04, Andi Shyti wrote:
> > Marek hs reported that commit 51130d52b84c ("i2c: exynos5: Init
> > data before registering interrupt handler") is breaking things.
> >
> > This is a regression and until we find out what happens this
> > should be reverted.
>
> We know exactly what's wrong. This should have never been applied.
when you wrote me about the regression I hadn't read Marek's mail
yet but I rushed to prepare the revert in order to have the fix
asap first. I'm going to fix the commit quoting Marek:
"
i2c: exynos5: Revert "i2c: exynos5: Init data before registering interrupt handler"
Marek hs reported about commit 51130d52b84c ("i2c: exynos5: Init
data before registering interrupt handler"):
Just above exynos5_i2c_clr_pend_irq() the clocks have been
disabled, so any access to the i2c host registers will result in
freeze or external abort (depending on the soc/cpu).
To make things worse, this patch moved registering the interrupt
handler after the i2c_add_adapter() call. This means that all i2c
devices that will be probbed directly from i2c_add_adapter()
won't be able to access the i2c bus, as the host controller is
still not fully functional that time yet.
This breaks all Exynos5+ platforms.
Reported-by: Marek Szyprowski <m.szyprowski@xxxxxxxxxxx>
Reported-by: Markus Reichl <m.reichl@xxxxxxxxxxxxx>
Signed-off-by: Andi Shyti <andi.shyti@xxxxxxxxxx>
Cc: Jesper Nilsson <jesper.nilsson@xxxxxxxx>
Cc: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
"
Thank you very much, Marek, for reporting this.
I prefer the revert not to break the i2c/i2c-host-fixes branch,
but if Wlfram prefers, I can remove it from the pull request
which I'm going to send tomorrow in the morning.
Can anyone please ack here?
And... Jesper, how did you test this patch?
Thanks,
Andi
>
> Best regards,
> Krzysztof
>
