[PATCH v2 15/16] clk: Detect failure to set defaults

Rasmus Villemoes rasmus.villemoes at prevas.dk
Wed Oct 20 09:17:43 CEST 2021


On 20/08/2021 20.18, Simon Glass wrote:
> Hi Harm,
> 
> On Wed, 18 Aug 2021 at 08:09, Harm Berntsen <harm.berntsen at nedap.com> wrote:
>>
>> On Thu, 2021-05-13 at 19:39 -0600, Simon Glass wrote:
>>>  int clk_uclass_post_probe(struct udevice *dev)
>>>  {
>>> +       int ret;
>>> +
>>>         /*
>>>          * when a clock provider is probed. Call clk_set_defaults()
>>>          * also after the device is probed. This takes care of cases
>>>          * where the DT is used to setup default parents and rates
>>>          * using assigned-clocks
>>>          */
>>> -       clk_set_defaults(dev, 1);
>>> +       ret = clk_set_defaults(dev, 1);
>>> +       if (ret)
>>> +               return log_ret(ret);
>>>
>>>         return 0;
>>>  }
>>
>> Note that this patch broke booting my imx8mn based board on U-Boot
>> v2021.10-rc2. 

I just ran into the same issue with v2021.10 being broken for
imx8mp_evk, and git bisect pointing at this commit, symptoms being

U-Boot 2021.10 (Oct 20 2021 - 08:45:51 +0200)

CPU:   Freescale i.MX8MP[8] rev1.1 at 1200 MHz
...
MMC:   mmc at 30b50000 - probe failed: -2
mmc at 30b60000 - probe failed: -2

Reverting 92f1e9a4b on top of v2021.10 yields

U-Boot 2021.10-00001-gac2520a138 (Oct 20 2021 - 09:05:48 +0200)

CPU:   Freescale i.MX8MP[8] rev1.1 at 1200 MHz
...
MMC:   FSL_SDHC: 1, FSL_SDHC: 2

cc += imx maintainers, this should not still be broken 2 months (and a
release) after it was reported.

Rasmus


More information about the U-Boot mailing list