[PATCH v8 00/28] Add DM support for omap PWM backlight

Lokesh Vutla lokeshvutla at ti.com
Wed Dec 30 19:14:53 CET 2020



On 30/12/20 4:36 am, Dario Binacchi wrote:
> 
> The series was born from the need to manage the PWM backlight of the
> display connected to my beaglebone board. To hit the target, I had to
> develop drivers for PWM management which in turn relied on drivers for
> managing timers and clocks, all developed according to the driver model.
> My intention was to use the SoC-specific API only at strictly necessary
> points in the code. My previous patches for migrating the AM335x display
> driver to the driver model had required the implementation of additional
> functions outside the concerns of the driver, (settings for dividing the
> pixel clock rate, configuring the display DPLL rate, ....) not being
> able to use the API of the related clock drivers. This series shouldn't
> have repeated the same kind of mistake. Furthermore, I also wanted to fix
> that kind of forced choice. Almost everything should have been accessible
> via the driver model API. In the series there are also some patches that
> could be submitted separately, but which I have however inserted to avoid
> applying future patches to incorporate them.
> With this last consideration, I hope I have convincingly justified the
> large number of patches in the series.
> 
> The patch enabling address translation into a CPU physical address from
> device-tree even in case of crossing levels with #size-cells = <0>, is
> crucial for the series. The previous implementation was unable to
> perform the address translation required by the am33xx device tree.
> I tried to apply in a conservative way as few changes as possible and
> to verify the execution of all the tests already developed, as well as
> the new ones I added for the new feature.
> 
> The patch series can be cleanly applied to the HEAD of the master which
> at the time of release points to ab865a8ee5c1a069f72a171270c02c99ccda7bfa
> commit (Merge tag 'u-boot-imx-20201227' of https://gitlab.denx.de/u-boot/custodians/u-boot-imx).
> 
> Changes in v8:
> - Imply CONFIG_TI_SYSC only if CONFIG_DM and CONFIG_OF_CONTROL are enabled.
> - Revert change on cm_t335_defconfig added on version 7 of the series.
> - Revert change on da850-evm-u-boot.dtsi. It was wrong and generated building
>   errors.


Applied to u-boot-ti/for-next branch.

Thanks and regards,
Lokesh


More information about the U-Boot mailing list