On Tue, 16 Feb 2021 at 11:19, Arnd Bergmann <arnd@xxxxxxxxxx> wrote: > > > + return samsung_early_console_setup(device, opt); > > > > Don't you need to handle the error code - set PROT_DEFAULT() or whatever > > was there before? > > __set_fixmap() has no return value, it just writes a page table entry and > does not fail. I meant, handle samsung_early_console_setup() error code (NULL). Best regards, Krzysztof