Re: [PATCH 2/8] i2c: i2c-qcom-geni: Signify successful driver probe

From: Lee Jones
Date: Wed Jun 05 2019 - 14:53:50 EST


On Wed, 05 Jun 2019, Wolfram Sang wrote:

>
> > To enable dynamic debugging in driver core you could use something like
> >
> > CONFIG_CMDLINE="dyndbg=\"func really_probe =p\""
> >
> > That gives you two printouts per successful probe, for example:
> >
> > bus: 'usb-serial': really_probe: probing driver edgeport_ti_1 with device ttyUSB0
> > bus: 'usb-serial': really_probe: bound device ttyUSB0 to driver edgeport_ti_1
>
> I agree that this scales much better than adding strings to every
> driver. Also, the driver core will report failed probes other than
> -ENODEV, or?
>
> Regarding this patch, however, I don't care much. I'll let the driver
> maintainers decide.

I've downgraded this to dev_dbg() in v2.

--
Lee Jones [æçæ]
Linaro Services Technical Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog