| Message ID | 20221223-hid-dbg-v1-0-f641e80263eb@weissschuh.net (mailing list archive) |
|---|---|
| Headers | show |
| Series | HID: i2c-hid: use standard debugging APIs | expand |
On Fri, 23 Dec 2022, Thomas Weißschuh wrote: > When trying to debug i2c-hid I was not able to use the builtin debug > mechanism because the "debug" module parameter is not writable. > > We can change i2c-hid to use the standard kernel debugging APIs which are > more discoverable and have more features. > > There is a bit of a stilistic conflict between consistently using > i2c_hid_dbg() and consistently using dev_<level>(). > > The second patch aligns debugging on i2c_hid_dbg(). > If this is unneeded, feel free to drop it. > > It would also be easy to just use dev_dbg() everywhere for consistency. > > To: Jiri Kosina <jikos@kernel.org> > To: Benjamin Tissoires <benjamin.tissoires@redhat.com> > Cc: linux-input@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > Signed-off-by: Thomas Weißschuh <linux@weissschuh.net> > > --- > Thomas Weißschuh (2): > HID: i2c-hid: switch to standard debugging APIs > HID: i2c-hid: use uniform debugging APIs > > drivers/hid/i2c-hid/i2c-hid-core.c | 15 +++------------ > 1 file changed, 3 insertions(+), 12 deletions(-) Applied to for-6.3/i2c-hid branch, thanks Thomas.
When trying to debug i2c-hid I was not able to use the builtin debug mechanism because the "debug" module parameter is not writable. We can change i2c-hid to use the standard kernel debugging APIs which are more discoverable and have more features. There is a bit of a stilistic conflict between consistently using i2c_hid_dbg() and consistently using dev_<level>(). The second patch aligns debugging on i2c_hid_dbg(). If this is unneeded, feel free to drop it. It would also be easy to just use dev_dbg() everywhere for consistency. To: Jiri Kosina <jikos@kernel.org> To: Benjamin Tissoires <benjamin.tissoires@redhat.com> Cc: linux-input@vger.kernel.org Cc: linux-kernel@vger.kernel.org Signed-off-by: Thomas Weißschuh <linux@weissschuh.net> --- Thomas Weißschuh (2): HID: i2c-hid: switch to standard debugging APIs HID: i2c-hid: use uniform debugging APIs drivers/hid/i2c-hid/i2c-hid-core.c | 15 +++------------ 1 file changed, 3 insertions(+), 12 deletions(-) --- base-commit: 8395ae05cb5a2e31d36106e8c85efa11cda849be change-id: 20221223-hid-dbg-cdc1d56d8cee Best regards,