]> git.baikalelectronics.ru Git - kernel.git/commit
tty: remove platform_sysrq_reset_seq
authorArnd Bergmann <arnd@arndb.de>
Tue, 26 May 2015 21:45:29 +0000 (14:45 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Tue, 2 Jun 2015 17:43:47 +0000 (10:43 -0700)
commit0f2df2a8fb2e676725c489fba427ba36ba3fe55a
tree046abcbc147037360407f0119300032d60f3681a
parent2ce7f704eb03d27c49436d2c4f250723f5e1d348
tty: remove platform_sysrq_reset_seq

The platform_sysrq_reset_seq code was intended as a way for an embedded
platform to provide its own sysrq sequence at compile time. After over two
years, nobody has started using it in an upstream kernel, and the platforms
that were interested in it have moved on to devicetree, which can be used
to configure the sequence without requiring kernel changes. The method is
also incompatible with the way that most architectures build support for
multiple platforms into a single kernel.

Now the code is producing warnings when built with gcc-5.1:

drivers/tty/sysrq.c: In function 'sysrq_init':
drivers/tty/sysrq.c:959:33: warning: array subscript is above array bounds [-Warray-bounds]
   key = platform_sysrq_reset_seq[i];

We could fix this, but it seems unlikely that it will ever be used, so
let's just remove the code instead. We still have the option to pass the
sequence either in DT, using the kernel command line, or using the
/sys/module/sysrq/parameters/reset_seq file.

Fixes: 6565fe4922 ("Input: sysrq - allow specifying alternate reset sequence")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/tty/sysrq.c