Hello Quentin,
On 2024-02-01 18:46, Quentin Schulz wrote:
On 2/1/24 05:02, Dragan Simic wrote:
On 2024-02-01 03:48, Kever Yang wrote:
On 2024/1/23 22:49, Quentin Schulz wrote:
From: Quentin Schulz quentin.schulz@theobroma-systems.com
Compared to the original misc_init_r from Rockchip mach code, setup_iodomain() is added and rockchip_setup_macaddr() is not called.
It is assumed adding rockchip_setup_macaddr() back is fine. Let's use rockchip_early_misc_init_r instead of reimplementing the whole misc_init_r from Rockchip (the side effect being that rockchip_setup_macaddr() is back).
We might actually introduce some issues with this change. I'll get back later with a more detailed explanation, together with a proposed fix, after I check it all in detail.
This applies to some other patches in this series as well.
I assume this is specifically about the migration to rockchip_early_misc_init_r?
What's the issue? And for which devices?
I don't really need this patch, it's just to avoid code duplication in a later patch but I'd rather not have this block this patch series. We can always revisit this in a separate patch series which can take as long as we want.
I should be able to provide all the details later today or tomorrow, together with the proposed fix. Perhaps such a timeline should be acceptable, by not introducing some significant delays.