Message ID | 20201229230654.8609-1-dariobin@libero.it |
---|---|
Headers | show |
Series | Add DM support for omap PWM backlight | expand |
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