On Sat, Nov 30, 2019 at 10:53 AM Matan Ziv-Av <matan@xxxxxxxxxxx> wrote: > > reader_mode_show() is now called from the driver itself, so only allow Where? > a small buffer for the boolean text. > - return snprintf(buffer, PAGE_SIZE, "%d\n", status); > + return snprintf(buffer, BUFFER_SIZE, "%d\n", status); > } No, this doesn't make sense, Actually sprintf() can be used here. -- With Best Regards, Andy Shevchenko