* [PATCH] input - mk712: fix the kernel crash while mk712 is selected
@ 2021-01-08 0:35 Barry Song
2021-01-12 5:28 ` Dmitry Torokhov
0 siblings, 1 reply; 3+ messages in thread
From: Barry Song @ 2021-01-08 0:35 UTC (permalink / raw
To: dmitry.torokhov, linux-input; +Cc: linuxarm, Barry Song
While debugging other issues, I enabled mk712 then I got the below crash
during arm64 linux boot:
[ 1.150412] pstate: 80000005 (Nzcv daif -PAN -UAO -TCO BTYPE=--)
[ 1.150541] pc : logic_outb+0x3c/0xa8
[ 1.150657] lr : mk712_init+0x74/0x1ec
[ 1.150745] sp : ffff800011efbd40
[ 1.150950] x29: ffff800011efbd40 x28: 0000000000000000
[ 1.151098] x27: ffff800011681078 x26: ffff8000115d04b8
[ 1.151192] x25: ffff8000116ff6b4 x24: ffff8000115bac10
[ 1.151287] x23: ffff800011dfa000 x22: ffff800011b8dbe0
[ 1.151421] x21: ffff800011eac000 x20: 0000000000000000
[ 1.151527] x19: fffffbfffe800266 x18: 0000000000000000
[ 1.151674] x17: ffff800010a7bc38 x16: ffff800010a7b0e0
[ 1.151944] x15: 0000a0d665e7ee58 x14: 000000000000011a
[ 1.152060] x13: 0000000000000001 x12: 0000000000000000
[ 1.152149] x11: 0000000000000000 x10: 0000000000000a60
[ 1.152291] x9 : 0000000000000000 x8 : ffff0000c0a2a480
[ 1.152429] x7 : 0000000000000000 x6 : ffff0000c0b0c9b8
[ 1.152522] x5 : 0000000000001000 x4 : ffff0000c03613b0
[ 1.152614] x3 : 0000000000000267 x2 : 0000000000000260
[ 1.152731] x1 : fffffbfffe800000 x0 : 0000000000ffbffe
[ 1.153135] Call trace:
[ 1.153342] logic_outb+0x3c/0xa8
[ 1.153434] mk712_init+0x74/0x1ec
[ 1.153571] do_one_initcall+0x54/0x1b8
[ 1.153705] kernel_init_freeable+0x1d0/0x238
[ 1.153860] kernel_init+0x14/0x118
[ 1.153960] ret_from_fork+0x10/0x30
[ 1.154304] Code: d2bfd001 f2df7fe1 f2ffffe1 8b010273 (39000274)
[ 1.155013] ---[ end trace f9b157c311d20edd ]---
[ 1.155854] Kernel panic - not syncing: Attempted to kill init! exitcode=0x0000000b
[ 1.156236] SMP: stopping secondary CPUs
The driver is very old and resources are totally hardcoded, no bus model.
It seems the simplest way to fix this issue is disabling it by parameter.
If someone really needs the old driver, it should enable the parameter
by cmdline.
Signed-off-by: Barry Song <song.bao.hua@hisilicon.com>
---
drivers/input/touchscreen/mk712.c | 8 ++++++++
1 file changed, 8 insertions(+)
diff --git a/drivers/input/touchscreen/mk712.c b/drivers/input/touchscreen/mk712.c
index 753d9cc1de1f..8a3110f27ce7 100644
--- a/drivers/input/touchscreen/mk712.c
+++ b/drivers/input/touchscreen/mk712.c
@@ -53,6 +53,9 @@ static unsigned int mk712_irq = 10; /* Also 12, 14, 15 */
module_param_hw_named(irq, mk712_irq, uint, irq, 0);
MODULE_PARM_DESC(irq, "IRQ of MK712 touchscreen controller");
+static bool mk712_enabled;
+module_param(mk712_enabled, bool, 0);
+
/* eight 8-bit registers */
#define MK712_STATUS 0
#define MK712_X 2
@@ -149,6 +152,11 @@ static int __init mk712_init(void)
{
int err;
+ if (!mk712_enabled) {
+ printk(KERN_WARNING "mk712: device is not enabled\n");
+ return -ENODEV;
+ }
+
if (!request_region(mk712_io, 8, "mk712")) {
printk(KERN_WARNING "mk712: unable to get IO region\n");
return -ENODEV;
--
2.25.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] input - mk712: fix the kernel crash while mk712 is selected
2021-01-08 0:35 [PATCH] input - mk712: fix the kernel crash while mk712 is selected Barry Song
@ 2021-01-12 5:28 ` Dmitry Torokhov
2021-01-12 5:32 ` [Linuxarm] " Song Bao Hua (Barry Song)
0 siblings, 1 reply; 3+ messages in thread
From: Dmitry Torokhov @ 2021-01-12 5:28 UTC (permalink / raw
To: Barry Song; +Cc: linux-input, linuxarm
On Fri, Jan 08, 2021 at 01:35:27PM +1300, Barry Song wrote:
> While debugging other issues, I enabled mk712 then I got the below crash
> during arm64 linux boot:
> [ 1.150412] pstate: 80000005 (Nzcv daif -PAN -UAO -TCO BTYPE=--)
> [ 1.150541] pc : logic_outb+0x3c/0xa8
> [ 1.150657] lr : mk712_init+0x74/0x1ec
> [ 1.150745] sp : ffff800011efbd40
> [ 1.150950] x29: ffff800011efbd40 x28: 0000000000000000
> [ 1.151098] x27: ffff800011681078 x26: ffff8000115d04b8
> [ 1.151192] x25: ffff8000116ff6b4 x24: ffff8000115bac10
> [ 1.151287] x23: ffff800011dfa000 x22: ffff800011b8dbe0
> [ 1.151421] x21: ffff800011eac000 x20: 0000000000000000
> [ 1.151527] x19: fffffbfffe800266 x18: 0000000000000000
> [ 1.151674] x17: ffff800010a7bc38 x16: ffff800010a7b0e0
> [ 1.151944] x15: 0000a0d665e7ee58 x14: 000000000000011a
> [ 1.152060] x13: 0000000000000001 x12: 0000000000000000
> [ 1.152149] x11: 0000000000000000 x10: 0000000000000a60
> [ 1.152291] x9 : 0000000000000000 x8 : ffff0000c0a2a480
> [ 1.152429] x7 : 0000000000000000 x6 : ffff0000c0b0c9b8
> [ 1.152522] x5 : 0000000000001000 x4 : ffff0000c03613b0
> [ 1.152614] x3 : 0000000000000267 x2 : 0000000000000260
> [ 1.152731] x1 : fffffbfffe800000 x0 : 0000000000ffbffe
> [ 1.153135] Call trace:
> [ 1.153342] logic_outb+0x3c/0xa8
> [ 1.153434] mk712_init+0x74/0x1ec
> [ 1.153571] do_one_initcall+0x54/0x1b8
> [ 1.153705] kernel_init_freeable+0x1d0/0x238
> [ 1.153860] kernel_init+0x14/0x118
> [ 1.153960] ret_from_fork+0x10/0x30
> [ 1.154304] Code: d2bfd001 f2df7fe1 f2ffffe1 8b010273 (39000274)
> [ 1.155013] ---[ end trace f9b157c311d20edd ]---
> [ 1.155854] Kernel panic - not syncing: Attempted to kill init! exitcode=0x0000000b
> [ 1.156236] SMP: stopping secondary CPUs
>
> The driver is very old and resources are totally hardcoded, no bus model.
> It seems the simplest way to fix this issue is disabling it by parameter.
> If someone really needs the old driver, it should enable the parameter
> by cmdline.
I wonder if we are not better off simply removing the driver. I doubt
there are any "Gateway AOL Connected Touchpad computers left, especially
ones that run 5.x kernels.
Thanks.
--
Dmitry
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [Linuxarm] Re: [PATCH] input - mk712: fix the kernel crash while mk712 is selected
2021-01-12 5:28 ` Dmitry Torokhov
@ 2021-01-12 5:32 ` Song Bao Hua (Barry Song)
0 siblings, 0 replies; 3+ messages in thread
From: Song Bao Hua (Barry Song) @ 2021-01-12 5:32 UTC (permalink / raw
To: Dmitry Torokhov; +Cc: linux-input@vger.kernel.org, linuxarm@openeuler.org
> -----Original Message-----
> From: Dmitry Torokhov [mailto:dmitry.torokhov@gmail.com]
> Sent: Tuesday, January 12, 2021 6:29 PM
> To: Song Bao Hua (Barry Song) <song.bao.hua@hisilicon.com>
> Cc: linux-input@vger.kernel.org; linuxarm@openeuler.org
> Subject: [Linuxarm] Re: [PATCH] input - mk712: fix the kernel crash while mk712
> is selected
>
> On Fri, Jan 08, 2021 at 01:35:27PM +1300, Barry Song wrote:
> > While debugging other issues, I enabled mk712 then I got the below crash
> > during arm64 linux boot:
> > [ 1.150412] pstate: 80000005 (Nzcv daif -PAN -UAO -TCO BTYPE=--)
> > [ 1.150541] pc : logic_outb+0x3c/0xa8
> > [ 1.150657] lr : mk712_init+0x74/0x1ec
> > [ 1.150745] sp : ffff800011efbd40
> > [ 1.150950] x29: ffff800011efbd40 x28: 0000000000000000
> > [ 1.151098] x27: ffff800011681078 x26: ffff8000115d04b8
> > [ 1.151192] x25: ffff8000116ff6b4 x24: ffff8000115bac10
> > [ 1.151287] x23: ffff800011dfa000 x22: ffff800011b8dbe0
> > [ 1.151421] x21: ffff800011eac000 x20: 0000000000000000
> > [ 1.151527] x19: fffffbfffe800266 x18: 0000000000000000
> > [ 1.151674] x17: ffff800010a7bc38 x16: ffff800010a7b0e0
> > [ 1.151944] x15: 0000a0d665e7ee58 x14: 000000000000011a
> > [ 1.152060] x13: 0000000000000001 x12: 0000000000000000
> > [ 1.152149] x11: 0000000000000000 x10: 0000000000000a60
> > [ 1.152291] x9 : 0000000000000000 x8 : ffff0000c0a2a480
> > [ 1.152429] x7 : 0000000000000000 x6 : ffff0000c0b0c9b8
> > [ 1.152522] x5 : 0000000000001000 x4 : ffff0000c03613b0
> > [ 1.152614] x3 : 0000000000000267 x2 : 0000000000000260
> > [ 1.152731] x1 : fffffbfffe800000 x0 : 0000000000ffbffe
> > [ 1.153135] Call trace:
> > [ 1.153342] logic_outb+0x3c/0xa8
> > [ 1.153434] mk712_init+0x74/0x1ec
> > [ 1.153571] do_one_initcall+0x54/0x1b8
> > [ 1.153705] kernel_init_freeable+0x1d0/0x238
> > [ 1.153860] kernel_init+0x14/0x118
> > [ 1.153960] ret_from_fork+0x10/0x30
> > [ 1.154304] Code: d2bfd001 f2df7fe1 f2ffffe1 8b010273 (39000274)
> > [ 1.155013] ---[ end trace f9b157c311d20edd ]---
> > [ 1.155854] Kernel panic - not syncing: Attempted to kill init!
> exitcode=0x0000000b
> > [ 1.156236] SMP: stopping secondary CPUs
> >
> > The driver is very old and resources are totally hardcoded, no bus model.
> > It seems the simplest way to fix this issue is disabling it by parameter.
> > If someone really needs the old driver, it should enable the parameter
> > by cmdline.
>
> I wonder if we are not better off simply removing the driver. I doubt
> there are any "Gateway AOL Connected Touchpad computers left, especially
> ones that run 5.x kernels.
Same here. Once a platform selects this driver, the platform will crash during
boot. It seems we have no way to stop users from selecting it in Kconfig.
Thanks
Barry
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2021-01-12 5:33 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-01-08 0:35 [PATCH] input - mk712: fix the kernel crash while mk712 is selected Barry Song
2021-01-12 5:28 ` Dmitry Torokhov
2021-01-12 5:32 ` [Linuxarm] " Song Bao Hua (Barry Song)
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.