All of lore.kernel.org
 help / color / mirror / Atom feed
From: Wolfram Sang <wsa@the-dreams.de>
To: Johan Hovold <johan@kernel.org>
Cc: Lee Jones <lee.jones@linaro.org>,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	balbi@kernel.org, wsa+renesas@sang-engineering.com,
	gregkh@linuxfoundation.org, linus.walleij@linaro.org,
	linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
	david.brown@linaro.org, alokc@codeaurora.org,
	kramasub@codeaurora.org, linux-i2c@vger.kernel.org,
	linux-gpio@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	andy.gross@linaro.org, jlhugo@gmail.com,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 2/8] i2c: i2c-qcom-geni: Signify successful driver probe
Date: Wed, 5 Jun 2019 16:18:12 +0200	[thread overview]
Message-ID: <20190605141812.GA962@kunai> (raw)
In-Reply-To: <20190605085527.GE29637@localhost>

[-- Attachment #1: Type: text/plain, Size: 624 bytes --]


> 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.


[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: Wolfram Sang <wsa@the-dreams.de>
To: Johan Hovold <johan@kernel.org>
Cc: balbi@kernel.org, david.brown@linaro.org,
	gregkh@linuxfoundation.org, linus.walleij@linaro.org,
	linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org,
	Bjorn Andersson <bjorn.andersson@linaro.org>,
	wsa+renesas@sang-engineering.com, alokc@codeaurora.org,
	kramasub@codeaurora.org, linux-i2c@vger.kernel.org,
	linux-gpio@vger.kernel.org, linux-arm-msm@vger.kernel.org,
	andy.gross@linaro.org, jlhugo@gmail.com,
	Lee Jones <lee.jones@linaro.org>,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 2/8] i2c: i2c-qcom-geni: Signify successful driver probe
Date: Wed, 5 Jun 2019 16:18:12 +0200	[thread overview]
Message-ID: <20190605141812.GA962@kunai> (raw)
In-Reply-To: <20190605085527.GE29637@localhost>


[-- Attachment #1.1: Type: text/plain, Size: 624 bytes --]


> 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.


[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2019-06-05 14:18 UTC|newest]

Thread overview: 72+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-06-04 10:44 [PATCH 1/8] i2c: i2c-qcom-geni: Provide support for ACPI Lee Jones
2019-06-04 10:44 ` Lee Jones
2019-06-04 10:44 ` [PATCH 2/8] i2c: i2c-qcom-geni: Signify successful driver probe Lee Jones
2019-06-04 10:44   ` Lee Jones
2019-06-04 10:44   ` Lee Jones
2019-06-05  6:20   ` Bjorn Andersson
2019-06-05  6:20     ` Bjorn Andersson
2019-06-05  7:16     ` Lee Jones
2019-06-05  7:16       ` Lee Jones
2019-06-05  7:22       ` Ard Biesheuvel
2019-06-05  7:22         ` Ard Biesheuvel
2019-06-05  8:23         ` Lee Jones
2019-06-05  8:23           ` Lee Jones
2019-06-05  7:56       ` Johan Hovold
2019-06-05  7:56         ` Johan Hovold
2019-06-05  8:20         ` Lee Jones
2019-06-05  8:20           ` Lee Jones
2019-06-05  8:33           ` Johan Hovold
2019-06-05  8:33             ` Johan Hovold
2019-06-05  8:49             ` Lee Jones
2019-06-05  8:49               ` Lee Jones
2019-06-05  8:55               ` Johan Hovold
2019-06-05  8:55                 ` Johan Hovold
2019-06-05 14:18                 ` Wolfram Sang [this message]
2019-06-05 14:18                   ` Wolfram Sang
2019-06-05 18:49                   ` Lee Jones
2019-06-05 18:49                     ` Lee Jones
2019-06-12 14:54                   ` Johan Hovold
2019-06-12 14:54                     ` Johan Hovold
2019-06-04 10:44 ` [PATCH 3/8] pinctrl: msm: Add ability for drivers to supply a reserved GPIO list Lee Jones
2019-06-04 10:44   ` Lee Jones
2019-06-04 10:44 ` [PATCH 4/8] pinctrl: qcom: sdm845: Provide ACPI support Lee Jones
2019-06-04 10:44   ` Lee Jones
2019-06-05  6:17   ` Bjorn Andersson
2019-06-05  6:17     ` Bjorn Andersson
2019-06-05  7:31     ` Lee Jones
2019-06-05  7:31       ` Lee Jones
2019-06-05 19:06       ` Bjorn Andersson
2019-06-05 19:06         ` Bjorn Andersson
2019-06-05 19:35         ` Lee Jones
2019-06-05 19:35           ` Lee Jones
2019-06-04 10:44 ` [PATCH 5/8] soc: qcom: geni: Add support for ACPI Lee Jones
2019-06-04 10:44   ` Lee Jones
2019-06-04 10:44 ` [PATCH 6/8] usb: dwc3: qcom: Add support for booting with ACPI Lee Jones
2019-06-04 10:44   ` Lee Jones
2019-06-05  6:35   ` Bjorn Andersson
2019-06-05  6:35     ` Bjorn Andersson
2019-06-05  7:09     ` Lee Jones
2019-06-05  7:09       ` Lee Jones
2019-06-05  9:55       ` Lee Jones
2019-06-05  9:55         ` Lee Jones
2019-06-04 10:44 ` [PATCH 7/8] usb: dwc3: qcom: Start USB in 'host mode' on the SDM845 Lee Jones
2019-06-04 10:44   ` Lee Jones
2019-06-05  7:00   ` Bjorn Andersson
2019-06-05  7:00     ` Bjorn Andersson
2019-06-05  8:34     ` Lee Jones
2019-06-05  8:34       ` Lee Jones
2019-06-05 14:07       ` Jeffrey Hugo
2019-06-05 14:07         ` Jeffrey Hugo
2019-06-05 18:50         ` Lee Jones
2019-06-05 18:50           ` Lee Jones
2019-06-05 19:14       ` Bjorn Andersson
2019-06-05 19:14         ` Bjorn Andersson
2019-06-05 19:29         ` Lee Jones
2019-06-05 19:29           ` Lee Jones
2019-06-04 10:44 ` [PATCH 8/8] usb: dwc3: qcom: Improve error handling Lee Jones
2019-06-04 10:44   ` Lee Jones
2019-06-05  7:03   ` Bjorn Andersson
2019-06-05  7:03     ` Bjorn Andersson
2019-06-05 11:42 [PATCH 1/8] i2c: i2c-qcom-geni: Provide support for ACPI Lee Jones
2019-06-05 11:42 ` [PATCH 2/8] i2c: i2c-qcom-geni: Signify successful driver probe Lee Jones
2019-06-05 11:42   ` Lee Jones
2019-06-05 11:42   ` Lee Jones

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20190605141812.GA962@kunai \
    --to=wsa@the-dreams.de \
    --cc=alokc@codeaurora.org \
    --cc=andy.gross@linaro.org \
    --cc=balbi@kernel.org \
    --cc=bjorn.andersson@linaro.org \
    --cc=david.brown@linaro.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=jlhugo@gmail.com \
    --cc=johan@kernel.org \
    --cc=kramasub@codeaurora.org \
    --cc=lee.jones@linaro.org \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-i2c@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-usb@vger.kernel.org \
    --cc=wsa+renesas@sang-engineering.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.