n_hdlc: put init/exit strings directly to prints
authorJiri Slaby <jslaby@suse.cz>
Wed, 19 Feb 2020 08:40:58 +0000 (09:40 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Fri, 21 Feb 2020 09:24:32 +0000 (10:24 +0100)
commitaebe5fc3b5685e1d9b86cc8314a8e8f3c6f3284e
tree41bb80c7a9e1c3c8bcc491663571f407deefa0bf
parentf3c2e27750ea42b3cec96e5264d95c2a2b793156
n_hdlc: put init/exit strings directly to prints

These strings were put aside from prints to save some bytes after module
load or when built-in -- they were freed after module load (__init ones) or
when the driver is selected as built-in (__exit ones).

The savings are negligible, but the code readability is worse by the
order of magnitude. So put the strings where they belong. Note that it
also used to make little sense putting const data in .data (the __exit
case).

While at it, switch to pr_info, pr_err, not using the KERN_INFO and _ERR
directly.

Signed-off-by: Jiri Slaby <jslaby@suse.cz>
Link: https://lore.kernel.org/r/20200219084118.26491-4-jslaby@suse.cz
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/n_hdlc.c