From: "Shen Jianping (ME-SE/EAD2)" <Jianping.Shen@de.bosch.com>
To: Krzysztof Kozlowski <krzk@kernel.org>,
"jic23@kernel.org" <jic23@kernel.org>,
"lars@metafoo.de" <lars@metafoo.de>,
"robh@kernel.org" <robh@kernel.org>,
"krzk+dt@kernel.org" <krzk+dt@kernel.org>,
"conor+dt@kernel.org" <conor+dt@kernel.org>,
"dima.fedrau@gmail.com" <dima.fedrau@gmail.com>,
"marcelo.schmitt1@gmail.com" <marcelo.schmitt1@gmail.com>,
"linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
"Lorenz Christian (ME-SE/EAD2)" <Christian.Lorenz3@de.bosch.com>,
"Frauendorf Ulrike (ME/PJ-SW3)" <Ulrike.Frauendorf@de.bosch.com>,
"Dolde Kai (ME-SE/PAE-A3)" <Kai.Dolde@de.bosch.com>
Subject: RE: [PATCH v7 0/2] iio: imu: smi240: add bosch smi240 driver
Date: Wed, 18 Sep 2024 08:14:20 +0000 [thread overview]
Message-ID: <AM8PR10MB4721D6C1ADE1D0ACF14BF701CD622@AM8PR10MB4721.EURPRD10.PROD.OUTLOOK.COM> (raw)
In-Reply-To: <cf53cbb5-104b-4f60-8890-98a53c27d176@kernel.org>
>> Add the iio driver for bosch imu smi240. The smi240 is a combined
>> three axis angular rate and three axis acceleration sensor module with
>> a measurement range of +/-300°/s and up to 16g. This driver provides
>> raw data access for each axis through sysfs, and tiggered buffer for
>> continuous sampling. A synchronous acc and gyro sampling can be
>> triggered by setting the capture bit in spi read command.
>>
>> dt-bindings:
>> v1 -> v2
>> - Add more detail in description
>> - Add maintainer
>> - Add vdd and vddio power supply
>> - Use generic node name
>> - Order the properties according to DTS coding style
>>
>> v2 -> v3
>> - Improve description
>> - Improve supply definition
>> - Make supply definition as required
>> - Add supply definition in example
>>
>> v3 -> v4
>> - No changes
>>
>> v4 -> v5
>> - No changes
>>
>> v5 -> v6
>> - Fix checkpatch findings
>>
>> v6 -> v7
>> - No changes
>>
>> imu driver:
>> v1 -> v2
>> - Use regmap for register access
>> - Redefine channel for each singel axis
>> - Provide triggered buffer
>> - Fix findings in Kconfig
>> - Remove unimportant functions
>>
>> v2 -> v3
>> - Use enum für capture mode
>> - Using spi default init value instead manual init
>> - remove duplicated module declaration
>> - Fix code to avoid warning
>>
>> v3 -> v4
>> - Use DMA safe buffer
>> - Use channel info instead of custom ABI
>> - Fix other findings
>>
>> v4 -> v5
>> - Merge the implementation in one simple file
>> - Add channel info for acc/gyro data channel
>> - Fix other findings
>
>?
>
>>
>> v5 -> v6
>> - Fix checkpatch findings
>> - Fix review findings
>
>? What exactly happened? Your changelog is way too vague.
>
>What happened with our reviews? Why did you get multiple of them?
>
Sorry for the confusing. let me make it clear
1. On 15.08 Conor reviewed the V3 dt-binding
2. On 26.08 we submit V4 (same as v3) without adding review tag. Krzysztof reviewed again and told us, that we need add review tag in patch.
3. On 05.09 we submit V5 which contains 2 tailing space in the description. This leads to checkpatch error. Krzysztof told us to get rid of those checkpatch errors.
4. On 10.09 we removed the tailing space and submit V6 (same as v3 and v4) without adding review tag. Rog gave the last review.
5. On 13.09 we submit V7 (same as v3, v4, and v6) with review tag of Conor, Krzysztof, and Rob.
That is where the 3 reviews come from.
In the dt-binding changelog we have "v5 -> v6 Fix checkpatch findings"
Best regards
Jianping
prev parent reply other threads:[~2024-09-18 8:14 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-09-13 10:00 Jianping.Shen
2024-09-13 10:00 ` [PATCH v7 1/2] dt-bindings: iio: imu: smi240: add Bosch smi240 Jianping.Shen
2024-09-13 17:54 ` Conor Dooley
2024-09-17 16:58 ` Krzysztof Kozlowski
2024-09-17 20:42 ` Conor Dooley
2024-09-13 10:00 ` [PATCH v7 2/2] iio: imu: smi240: add driver Jianping.Shen
2024-09-14 16:32 ` Jonathan Cameron
2024-09-16 20:32 ` Shen Jianping (ME-SE/EAD2)
2024-09-17 11:13 ` Jonathan Cameron
2024-09-17 13:13 ` Shen Jianping (ME-SE/EAD2)
2024-09-28 14:36 ` Jonathan Cameron
2024-09-17 16:59 ` [PATCH v7 0/2] iio: imu: smi240: add bosch smi240 driver Krzysztof Kozlowski
2024-09-18 8:14 ` Shen Jianping (ME-SE/EAD2) [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=AM8PR10MB4721D6C1ADE1D0ACF14BF701CD622@AM8PR10MB4721.EURPRD10.PROD.OUTLOOK.COM \
--to=jianping.shen@de.bosch.com \
--cc=Christian.Lorenz3@de.bosch.com \
--cc=Kai.Dolde@de.bosch.com \
--cc=Ulrike.Frauendorf@de.bosch.com \
--cc=conor+dt@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=dima.fedrau@gmail.com \
--cc=jic23@kernel.org \
--cc=krzk+dt@kernel.org \
--cc=krzk@kernel.org \
--cc=lars@metafoo.de \
--cc=linux-iio@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=marcelo.schmitt1@gmail.com \
--cc=robh@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®