Hi, On 1/24/23 13:52, Andy Shevchenko wrote: > On Tue, Jan 24, 2023 at 12:31:14PM +0100, Hans de Goede wrote: >> On 1/24/23 11:51, Andy Shevchenko wrote: >>> On Mon, Jan 23, 2023 at 01:51:54PM +0100, Hans de Goede wrote: > > ... > >>>> + if (sensor->is_streaming == enable) { >>>> + dev_warn(&client->dev, "stream already %sed\n", enable ? "start" : "stopp"); >>> >>> stopP ?! >> >> Yes the format string is "%sed" so "stopp" gives us "stopped". > > Can we, please, use full words, so it will be easier to find a candidate for > including into string_choices.h or so? Sure I'll update this before adding it to the PR. Regards, Hans