summaryrefslogtreecommitdiff
path: root/Documentation/ia64
diff options
context:
space:
mode:
authorSam Protsenko <semen.protsenko@linaro.org>2021-12-04 22:57:57 +0300
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>2021-12-20 18:54:37 +0300
commit8cf8d3c4a634042f37ba34429e6a36412640ede4 (patch)
tree2cfc152b3b22f48d83a03b5554e5a49a702630df /Documentation/ia64
parent0882b473b084df31288003b3bee974aabac9dcf9 (diff)
downloadlinux-8cf8d3c4a634042f37ba34429e6a36412640ede4.tar.xz
tty: serial: samsung: Fix console registration from module
On modern Exynos SoCs (like Exynos850) the UART can be implemented as a part of USI IP-core. In such case, USI driver is used to initialize USI registers, and it also calls of_platform_populate() to instantiate all sub-nodes (e.g. serial node) of USI node. When serial driver is built-in, but USI driver is a module, and CONFIG_SERIAL_SAMSUNG_CONSOLE is enabled, next call chain will happen when loading USI module from user space: usi_init v usi_probe v of_platform_populate v s3c24xx_serial_probe v uart_add_one_port v uart_configure_port v register_console v try_enable_new_console v s3c24xx_serial_console_setup But because the serial driver is built-in, and s3c24xx_serial_console_setup() is marked with __init keyword, that symbol will discarded and long gone by that time already, causing failed paging request. That happens during the next config combination: EXYNOS_USI=m SERIAL_SAMSUNG=y SERIAL_SAMSUNG_CONSOLE=y That config should be completely possible, so rather than limiting SERIAL_SAMSUNG choice to "m" only when USI=m, remove __init keyword for all affected functions. Signed-off-by: Sam Protsenko <semen.protsenko@linaro.org> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com> Link: https://lore.kernel.org/r/20211204195757.8600-6-semen.protsenko@linaro.org Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Diffstat (limited to 'Documentation/ia64')
0 files changed, 0 insertions, 0 deletions