Re: [PATCH v2] iio: core: move 'indio_dev->info' null check first

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v2] iio: core: move 'indio_dev->info' null check first
From: Markus Elfring <Markus.Elfring@xxxxxx>
Date: Tue, 7 Apr 2020 10:20:31 +0200
Cc: Jonathan Cameron <jic23@xxxxxxxxxx>, linux-kernel@xxxxxxxxxxxxxxx
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.6.0

> Doesn't fix anything.

Would the change description be nicer without this sentence?


> Just moves this …

I suggest to choose an imperative wording.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?id=7e63420847ae5f1036e4f7c42f0b3282e73efbc2#n151

Regards,
Markus

Follow-Ups:
- Re: [PATCH v2] iio: core: move 'indio_dev->info' null check first
  - From: Ardelean, Alexandru

Prev by Date: Re: [RFC PATCH 1/3] iio: core: rename 'indio_dev->buffer_list' -> 'indio_dev->active_buffers'
Next by Date: [PATCH] iio: imu: st_lsm6dsx: limit variales scope reading hw FIFO
Previous by thread: [PATCH v3] iio: imu: st_lsm6dsx: drop huge include in sensor-hub driver
Next by thread: Re: [PATCH v2] iio: core: move 'indio_dev->info' null check first
Index(es):
- Date
- Thread

[Index of Archives] [Linux USB Devel] [Video for Linux] [Linux Audio Users] [Yosemite News] [Linux Input] [Linux Kernel] [Linux SCSI] [X.org]

Powered by Linux