FYI, we noticed the below changes on
https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux i2c/for-next
commit 95026658c46ea2d94498d0dac1282e28cd47c64a ("i2c: do not use internal data from driver core")
+----------------------------------------------------------+------------+------------+
| | 30e31a1fbc | 95026658c4 |
+----------------------------------------------------------+------------+------------+
| boot_successes | 12 | 0 |
| boot_failures | 0 | 8 |
| WARNING:at_drivers/i2c/i2c-core.c:#i2c_register_driver() | 0 | 8 |
| BUG:unable_to_handle_kernel | 0 | 8 |
| Oops | 0 | 8 |
| RIP:notifier_chain_register | 0 | 8 |
| Kernel_panic-not_syncing:Fatal_exception | 0 | 8 |
| backtrace:i2c_register_driver | 0 | 8 |
| backtrace:i2c_init | 0 | 8 |
| backtrace:kernel_init_freeable | 0 | 8 |
| backtrace:max1586_pmic_init | 0 | 8 |
| backtrace:max8649_init | 0 | 8 |
| backtrace:max8660_init | 0 | 8 |
| backtrace:tps_65023_init | 0 | 8 |
| backtrace:wm#_module_init | 0 | 8 |
| backtrace:pcf50633_init | 0 | 8 |
| backtrace:maven_driver_init | 0 | 8 |
| backtrace:lp#_i2c_driver_init | 0 | 8 |
| backtrace:ch#_init | 0 | 8 |
| backtrace:sil164_init | 0 | 8 |
| backtrace:ad_dpot_i2c_driver_init | 0 | 8 |
| backtrace:ics932s401_driver_init | 0 | 8 |
| backtrace:bh1780_driver_init | 0 | 8 |
| backtrace:bh1770_driver_init | 0 | 8 |
| backtrace:apds990x_driver_init | 0 | 8 |
| backtrace:apds9802als_driver_init | 0 | 8 |
| backtrace:isl29003_driver_init | 0 | 8 |
| backtrace:isl29020_driver_init | 0 | 8 |
| backtrace:tsl2550_driver_init | 0 | 8 |
| backtrace:ds1682_driver_init | 0 | 8 |
| backtrace:hmc6352_driver_init | 0 | 8 |
| backtrace:at24_init | 0 | 8 |
| backtrace:eeprom_driver_init | 0 | 8 |
| backtrace:max6875_driver_init | 0 | 8 |
| backtrace:lis3lv02d_i2c_driver_init | 0 | 8 |
| backtrace:adp5588_driver_init | 0 | 8 |
| backtrace:lm#_i2c_driver_init | 0 | 8 |
| backtrace:max7359_i2c_driver_init | 0 | 8 |
| backtrace:qt2160_driver_init | 0 | 8 |
| backtrace:synaptics_i2c_driver_init | 0 | 8 |
| backtrace:ad7879_i2c_driver_init | 0 | 8 |
| backtrace:eeti_ts_driver_init | 0 | 8 |
| backtrace:mcs5000_ts_driver_init | 0 | 8 |
| backtrace:tsc2007_driver_init | 0 | 8 |
| backtrace:bq32k_driver_init | 0 | 8 |
| backtrace:ds1307_driver_init | 0 | 8 |
| backtrace:ds1374_driver_init | 0 | 8 |
| backtrace:ds1672_driver_init | 0 | 8 |
| backtrace:fm3130_driver_init | 0 | 8 |
| backtrace:isl1208_driver_init | 0 | 8 |
| backtrace:m41t80_driver_init | 0 | 8 |
| backtrace:max6900_driver_init | 0 | 8 |
| backtrace:pcf8563_driver_init | 0 | 8 |
| backtrace:pcf8583_driver_init | 0 | 8 |
| backtrace:rs5c372_driver_init | 0 | 8 |
| backtrace:rx#_driver_init | 0 | 8 |
| backtrace:s35390a_driver_init | 0 | 8 |
| backtrace:x1205_driver_init | 0 | 8 |
| backtrace:smbalert_driver_init | 0 | 8 |
| backtrace:bus_register_notifier | 0 | 8 |
| backtrace:i2c_dev_init | 0 | 8 |
+----------------------------------------------------------+------------+------------+
[ 0.667068] atomic64_test: passed for x86-64 platform with CX8 and with SSE
[ 0.669694] NET: Registered protocol family 16
[ 0.673822] ------------[ cut here ]------------
[ 0.674538] WARNING: CPU: 0 PID: 1 at drivers/i2c/i2c-core.c:1930 i2c_register_driver+0x2c/0x79()
[ 0.675999] Modules linked in:
[ 0.676598] CPU: 0 PID: 1 Comm: swapper/0 Not tainted 4.5.0-rc3-00030-g9502665 #1
[ 0.677715] Hardware name: QEMU Standard PC (i440FX + PIIX, 1996), BIOS Debian-1.8.2-1 04/01/2014
[ 0.678968] 0000000000000000 ffff880031927e10 ffffffff81756b9f 0000000000000000
[ 0.680344] 0000000000000009 ffff880031927e48 ffffffff810de3bd ffffffff826544ad
[ 0.681717] 0000000000000000 00000000fffffff5 0000000000000000 ffffffff8402b040
[ 0.683095] Call Trace:
[ 0.683543] [<ffffffff81756b9f>] dump_stack+0x81/0xb6
[ 0.684278] [<ffffffff810de3bd>] warn_slowpath_common+0x99/0xb2
[ 0.685110] [<ffffffff826544ad>] ? i2c_register_driver+0x2c/0x79
[ 0.685952] [<ffffffff810de489>] warn_slowpath_null+0x1a/0x1c
[ 0.686762] [<ffffffff826544ad>] i2c_register_driver+0x2c/0x79
[ 0.687607] [<ffffffff846b3b94>] ? trace_event_define_fields_smbus_write+0xb/0xb
[ 0.688718] [<ffffffff846b3bfe>] i2c_init+0x6a/0xa4
[ 0.689456] [<ffffffff846b3b94>] ? trace_event_define_fields_smbus_write+0xb/0xb
[ 0.690559] [<ffffffff81002174>] do_one_initcall+0xe7/0x173
[ 0.691354] [<ffffffff84629144>] kernel_init_freeable+0x1bf/0x24c
[ 0.692205] [<ffffffff82e98b59>] ? rest_init+0x140/0x140
[ 0.692975] [<ffffffff82e98b67>] kernel_init+0xe/0xde
[ 0.693717] [<ffffffff82ea4e9f>] ret_from_fork+0x3f/0x70
[ 0.694486] [<ffffffff82e98b59>] ? rest_init+0x140/0x140
[ 0.695266] ---[ end trace 56255c48d0658884 ]---
[ 0.696041] cpuidle: using governor ladder
Thanks,
Kernel Test Robot
On Mon, Mar 14, 2016 at 08:47:59AM +0800, kernel test robot wrote:
> FYI, we noticed the below changes on
>
> https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux i2c/for-next
> commit 95026658c46ea2d94498d0dac1282e28cd47c64a ("i2c: do not use internal data from driver core")
Ouch! So nobody of us ever boot-tested this patch? :( Will send a fix
soon.
On Mon, Mar 14, 2016 at 02:14:19AM +0100, Wolfram Sang wrote:
> On Mon, Mar 14, 2016 at 08:47:59AM +0800, kernel test robot wrote:
> > FYI, we noticed the below changes on
> >
> > https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux i2c/for-next
> > commit 95026658c46ea2d94498d0dac1282e28cd47c64a ("i2c: do not use internal data from driver core")
>
> Ouch! So nobody of us ever boot-tested this patch? :(
No. Sorry. It was only compile tested but Fengguang sent me the warning
before you applied the patch but I was travelling and so could not do anything.
regards
sudip