[PATCH v2 0/6] arm: omap: Add back omap4 support

[Bastien Curutchet]

Hi Tom

On 6/4/26 9:03 PM, Tom Rini wrote:
> On Thu, Jun 04, 2026 at 08:29:13AM -0600, Tom Rini wrote:
>> On Thu, Jun 04, 2026 at 09:51:01AM +0200, Bastien Curutchet wrote:
>>> Hi all,
>>>
>>> On 5/21/26 8:45 AM, Bastien Curutchet wrote:
>>>> Hi all,
>>>>
>>>> This series aims to add back the omap4 support. This support was removed
>>>> by commit b0ee3fe642c ("arm: ti: Remove omap4 platform support") because
>>>> at that moment, none of the OMAP4-based boards had done the migration to
>>>> DM_I2C.
>>>> My use case is an old product based on the Variscite's omap4 system on
>>>> module. I needed to upgrade U-Boot on it for security reasons. I think
>>>> that this work could benefit to other people who may have same kind of
>>>> product to maintain.
>>>>
>>>> Patch 1 to 3 remove the omap's clock driver dependency to the AM33xx
>>>> as it is also present in omap4 platforms. I tested these changes on the
>>>> beaglebone black to ensure I didn't break the AM33xx case.
>>>>
>>>> Patch 4 & 5 revert the deletion of the omap4 support. The revert makes
>>>> checkpatch.pl angry. I fixed quite a lots of warnings already but it
>>>> remains two kinds of warnings:
>>>> - CamelCase on timings structure, I left the CamelCase because IMHO it's
>>>>     more readable this way.
>>>> - #ifdef CONFIG_XYZ shouldn't be used anymore. I left one of this because
>>>>     I didn't find a clean way to get rid of it.
>>>>
>>>> Patch 6 adds support for the Variscite's system on module. This system on
>>>> module is supported by the Linux project through
>>>> ti/omap/omap4-var-som-om44.dtsi
>>>>
>>>> Signed-off-by: Bastien Curutchet <bastien.curutchet at bootlin.com>
>>>> ---
>>>> Changes in v2:
>>>> - PATCH 5: Set myself as OMAP4 maintainer
>>>> - PATCH 6: Set SYS_CONFIG_NAME to ti_omap4_common to get rid of omap4_var_som.h
>>>> - Link to v1: https://lore.kernel.org/r/20260519-omap4-support-v1-0-b4a2a7435775@bootlin.com
>>>>
>>>
>>> Gentle ping here, has anyone had time to take a look at this ?
>>
>> It's on my list, thanks.
> 
> This breaks building on am335x_evm among others. Please make sure that
> at least all CONFIG_AM33XX platforms build, or use CI to test
> everything:
> https://docs.u-boot-project.org/en/latest/develop/ci_testing.html

Sorry, I hadn't noticed the warning I added.

I just sent a v3 that fixes it, I did a PR with this version on the 
U-Boot's GitHub to ensure it passed all tests (PR#980)


Best regards,
Bastien