Re: [PATCH 46/57] media: atomisp: ov2680: Delay power-on till streaming is started

Hans de Goede
2023-01-24T13:36:21Z

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