On Wed, Aug 24, 2022 at 11:42 PM Geert Uytterhoeven
<geert@xxxxxxxxxxxxxx> wrote:
>
> Hi Morimoto-san,
>
> On Thu, Aug 25, 2022 at 3:11 AM Kuninori Morimoto
> <kuninori.morimoto.gx@xxxxxxxxxxx> wrote:
> > > > > To probe "Sound Card", it needs other necessary drivers.
> > > > > But it seems some necessary drivers are not probed by above patch.
> > > > > In my quick check, "HDMI" and 1 of "Sound Codec" drivers are not probed.
> > (snip)
> > > > Saravana is more familiar with devlink than I am, so he may be able to
> > > > diagnose that without further information, but otherwise it would be
> > > > helpful if you could provide some logs. Enabling the debug messages in
> > > > drivers/base/dd.c can provide some insight. The contents of the debugfs
> > > > devices_deferred file can also be useful, as well as the
> > > > consumer/supplier links in sysfs for all the devices involved with the
> > > > sound card.
> >
> > Thanks !
> >
> > This devices_deferred file (which I didn't know) helped my issue.
> > It was my fault. My .config had been missing necessary driver
> > (= CONFIG_REGULATOR_FIXED_VOLTAGE).
> >
> > ... but... why it had been working without it before... ?
Hi Morimoto-san,
Hmmm... even if you didn't have the driver compiled in, fw_devlink
should have stopped blocking the probe after deferred_probe_timeout
expires and that value is defaulted to 10s (it extends on every
successful driver registration). Can you help figure out if that's
happening and if not, why not?
Thanks,
Saravana
>
> I guess the adi,adv7513 and ti,pcm3168a drivers consider regulators
> optional, so when they probe, they succeed?
>
> With devlinks enabled, they are never probed, as their dependencies
> didn't probe due to the missing regulator driver.
>
> Gr{oetje,eeting}s,
>
> Geert
>
> --
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
>
> In personal conversations with technical people, I call myself a hacker. But
> when I'm talking to journalists I just say "programmer" or something like that.
> -- Linus Torvalds
