Re: [PATCH] ARM: OMAP1: ams-delta: Fix MODEM initialization failure

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,

* Janusz Krzysztofik <jmkrzyszt@xxxxxxxxx> [231008 00:47]:
> [    6.823917][    T1] serial8250 serial8250.1: incomplete constraints, dummy supplies not allowed
> [    6.874117][    T1] ------------[ cut here ]------------
> [    6.893918][    T1] WARNING: CPU: 0 PID: 1 at drivers/base/core.c:2486 device_release+0x98/0xa8
> [    6.930626][    T1] Device 'serial8250.1' does not have a release() function, it is broken and must be fixed. See Documentation/core-api/kobject.rst.

The patch looks OK to me, but can you please trim down the warning for
prettier git log output? At least the timestamps can go, and since
you fixed it, you could trim the message down to something like this:

serial8250 serial8250.1: incomplete constraints, dummy supplies not allowed
WARNING: CPU: 0 PID: 1 at drivers/base/core.c:2486 device_release+0x98/0xa8
Device 'serial8250.1' does not have a release() function, it is broken and
must be fixed. See Documentation/core-api/kobject.rst.
...
put_device from platform_device_put+0x1c/0x24
platform_device_put from ams_delta_init_late+0x4c/0x68
ams_delta_init_late from init_machine_late+0x1c/0x94
init_machine_late from do_one_initcall+0x60/0x1d4

This still allows folks seeing similar errors to find them when searching.
Also the ASoC warnings can be trimmed down a bit, formatting up to you
how you prefer them naturally :)

Thanks,

Tony



[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux