[U-Boot] [U-Boot, 2/2] rockchip: Drop call to rockchip_dnl_mode_check() for now

Philipp Tomsich philipp.tomsich at theobroma-systems.com
Fri Feb 1 08:39:58 UTC 2019



> On 01.02.2019, at 09:38, Philipp Tomsich <philipp.tomsich at theobroma-systems.com> wrote:
> 
> Andy,
> 
>> On 01.02.2019, at 02:32, andy.yan at rock-chips.com wrote:
>> 
>> Hi:
>> 
>>> This function causes a 5-second delay and stops the display working on
>>> minnie. This code should be in a driver and should only be enabled by
>>> a device-tree property, so that it does not affect devices which do not
>>> have this feature.
>>> 
>>> Signed-off-by: Simon Glass <sjg at chromium.org>
>>> Reviewed-by: Philipp Tomsich <philipp.tomsich at theobroma-systems.com>
>>> ---
>>> 
>>> arch/arm/mach-rockchip/boot_mode.c | 8 +++++++-
>>> 1 file changed, 7 insertions(+), 1 deletion(-)
>>> 
>> 
>> 》Applied to u-boot-rockchip, thanks!
>> 
>> 
>> According to the previous discussing, shouldn‘t this be fixed by  by setting CONFIG_BOOT_MODE_REG=0 ?
>> All rockchip products need this function,we really don't want it be turned off.
> 
> The actual decision was that this needed to be converted to the driver-model.
> Setting CONFIG_BOOT_MODE_REG=0 was a workaround that we used only
> for 2019.1, so I could delay applying Simon’s patch until this cycle.

Just for the record, this is a verbatim quote from the original review:
> If this works for you, we can avoid removing this feature this late in the cycle and
> defeature/remove it in the next release, unless moved into a DTS-controlled driver.

> 
> Please convert this into a DM driver, so we can add the driver for 2019.4.
> 
> Thanks,
> Philipp.
> 
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot



More information about the U-Boot mailing list