Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
From:   LI Qingwu <qing-wu.li@leica-geosystems.com.cn>
To:     Rob Herring <robh@kernel.org>
CC:     "jic23@kernel.org" <jic23@kernel.org>,
        "lars@metafoo.de" <lars@metafoo.de>,
        "mchehab+huawei@kernel.org" <mchehab+huawei@kernel.org>,
        "ardeleanalex@gmail.com" <ardeleanalex@gmail.com>,
        "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "mike.looijmans@topic.nl" <mike.looijmans@topic.nl>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        HAEMMERLE Thomas <thomas.haemmerle@leica-geosystems.com>
Subject: RE: [PATCH V5 6/6] dt-bindings: iio: accel: Add bmi085 and bmi090l
 bindings
Thread-Topic: [PATCH V5 6/6] dt-bindings: iio: accel: Add bmi085 and bmi090l
 bindings
Thread-Index: AQHYcQVhb1m+58wAMkWozJbGrJeE9608L2AAgAETZMA=
Date:   Fri, 3 Jun 2022 07:32:17 +0000
Message-ID: <AM9PR06MB78445C0FE66A61F6BAC4CE35D7A19@AM9PR06MB7844.eurprd06.prod.outlook.com>
References: <20220526133359.2261928-1-Qing-wu.Li@leica-geosystems.com.cn>
 <20220526133359.2261928-7-Qing-wu.Li@leica-geosystems.com.cn>
 <20220602135734.GA2198822-robh@kernel.org>
In-Reply-To: <20220602135734.GA2198822-robh@kernel.org>
Accept-Language: zh-CN, en-US
Content-Language: zh-CN
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?o8gi1UVkX4OeQ840cdIbq5xE5ByNW4qIr0DJW1w42qFXcJlNAHY3p9NHT6xi?=
 =?us-ascii?Q?97jqIxoLIoaqku9OqiHnhFBNTiKf8Y0R/Rn6srKx6jM+Xh4k+3ZXPTGjm4Oc?=
 =?us-ascii?Q?8re2LLJKI0ARgBK+2tk4XsV+qmsBSRAF5LcDWXJXr+c8EuxIbbKvM7TAMU10?=
 =?us-ascii?Q?j5Wr9YppSal2G6XpMAMbFbMinvUeVc6jK+c+YTDOX2+mXwouvQ/IJgqu+SJw?=
 =?us-ascii?Q?rme61bPrCVsC2oM4OMYxfHHx2jO6NfXyI5s1xBj4wcyzSvocfLEdNhji2sUs?=
 =?us-ascii?Q?GvY0pOYLCmxW4Z0Fke+9EkfHEg5/DrsGTCbnhcvo0x9yVkt7Ii0N2rQWXlSa?=
 =?us-ascii?Q?f7uevbrhKuDSK73H6SywyL29kCAmuaw08Y7jQfU+5NvEo5Jd2HUy/Mf9t6nS?=
 =?us-ascii?Q?GHOWFk19RCuUo65PVZbwA21XFdDnAZvOZoG9hskNV3IZ9Pc771nKYjDZXv4q?=
 =?us-ascii?Q?txx3dQxt4dikdZJvpBS+D4OL4/Y+ynzi6C/qzBbFojugLcqXRZlaKrdNXKcL?=
 =?us-ascii?Q?yn+MsMM3jDcJecoHLpOmlV8SYExwIXrSYyRzsfUmXnKkfJFQJ0+Fhm9UuFdy?=
 =?us-ascii?Q?r7NolmkKxPV4KnrDKJTY8UMfN/joHKr08Am7VcdCpph9IxBoxMxm129eZUTC?=
 =?us-ascii?Q?c00yvtnvgYJX/4yOQcygU9fctoEfBnJqtqP4/5bHVTFGaEq28P4hgM+kBVmv?=
 =?us-ascii?Q?mh6OqD43x6O8JVYIabhoEVZkzpI32oF1b23L0EIcGWAeaCmpjAGtyI8aIlo/?=
 =?us-ascii?Q?9BpyQ2VFefB02xqTahJf6SwRrg9b3OC9Ht5zS2HCujW9YN8qqIFoPy0wHmWb?=
 =?us-ascii?Q?xwWWh2UO28C4Qm3N/EHOTo0tq+Bs+o8HOU2petoZ8lBrnUsGaVQPwtItKkXQ?=
 =?us-ascii?Q?m2lO1KL48zsDeHLw6dyGrXfIczdcODNtglV9iMHah5N7N+UOfcb0u26/OHiU?=
 =?us-ascii?Q?5AZxmJCNWrDaIgrfF1Vmo72fq8KBt5FKIdIxZ8Gk/hTRND8H9ZWwH5XMbHxJ?=
 =?us-ascii?Q?Qra7Q+41yvDuj/xbkzzCo3LOEXSJOPzYvNFJ+jVZz4yrKRjLN9wKGXBX/2+R?=
 =?us-ascii?Q?nCBX1uc7FXglh4CoswQie+S7y1AlsRx8/Zr2RxCCZmlhPffEKz/1SOo28yX2?=
 =?us-ascii?Q?fj7voAKz5dKyiWbu6boZRtqvv+p5z4Qw/5r74jJahU7Ardr/4r+boYBpWj91?=
 =?us-ascii?Q?G6tNmeRvuMkmJBY3G3ScDPSPFmJaIvaA2LidGFICw/Z9Ybde37qjQqgHNyZw?=
 =?us-ascii?Q?4NOryoxtuj326UAzhBG3eJiYO5kjMMhn1W91eGPtacH80XxE+Jk2El667+ft?=
 =?us-ascii?Q?n7ZHkX8XPCTG6FEDK/aSZWRl1VvaUpdxRwBmbIW4g1Q0JcZSIKYylz+eyUKS?=
 =?us-ascii?Q?KaH9U/p1WITylLtoBIS7NtJY6uBcRXh6F64j5kR8+PhfoDstP9ZLiz57o2jQ?=
 =?us-ascii?Q?RqZKLGe/LS9HItA0Z0l324EC5WmUODp/IgvsLPAxjKspby8Ij+ydWGOpBH5h?=
 =?us-ascii?Q?YYLefF/2JkoDgBs9hZIV2t5/j2FITKUWIVF+AmJ1xVy7dTfES3BIMmnmCRd2?=
 =?us-ascii?Q?Z+I4F88VDiXRSD6WJ619wIBNWTqCiR/krM7gBtNa4Kz7iMxnw+3+q66G+9ur?=
 =?us-ascii?Q?AmfxqUlxaET2XOw2b0Bbz//RUiXpWMVtNgz08Y4ykUQrd4bYiIlIw8vSrWxT?=
 =?us-ascii?Q?hFQOLNROlnS+dsSimMf5fDNSqjY0FAQrSutrNoukGtOrUfJCywPdqrijOnB/?=
 =?us-ascii?Q?URO2hhHxwGtGM3wAIvYZe+okjPY5a/w=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: leica-geosystems.com.cn
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: AM9PR06MB7844.eurprd06.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 933d6c4e-2bd7-4cb0-ade4-08da4533303e
X-MS-Exchange-CrossTenant-originalarrivaltime: 03 Jun 2022 07:32:18.0784
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 1b16ab3e-b8f6-4fe3-9f3e-2db7fe549f6a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: +UlwQFS46ZbaMpgzwW5RyZKKDW+3fq5zWWq7ukPSAWm2qbVt7otXRA9CLVituc27+KbcfRCwKROcXa2gW7VpuDxgUuohyK7nivCfQo5tTW9YwmTZNXFRyY2yEr7ltzfG
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR0602MB3620
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RDNS_NONE,SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE
	autolearn=no autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
	lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


> -----Original Message-----
> From: Rob Herring <robh@kernel.org>
> Sent: Thursday, June 2, 2022 9:58 PM
> To: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn>
> Cc: jic23@kernel.org; lars@metafoo.de; mchehab+huawei@kernel.org;
> ardeleanalex@gmail.com; linux-iio@vger.kernel.org;
> linux-kernel@vger.kernel.org; mike.looijmans@topic.nl;
> devicetree@vger.kernel.org; HAEMMERLE Thomas
> <thomas.haemmerle@leica-geosystems.com>
> Subject: Re: [PATCH V5 6/6] dt-bindings: iio: accel: Add bmi085 and bmi09=
0l
> bindings
>=20
> This email is not from Hexagon's Office 365 instance. Please be careful w=
hile
> clicking links, opening attachments, or replying to this email.
>=20
>=20
> On Thu, May 26, 2022 at 01:33:59PM +0000, LI Qingwu wrote:
> > Adds the device-tree bindings for the Bosch
> > BMI085 and BMI090L IMU, the accelerometer part.
> >
> > Datasheet:
> https://eur02.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fwww.b=
os
> ch-sensortec.com%2Fmedia%2Fboschsensortec%2Fdownloads%2Fdatasheets%
> 2Fbst-bmi085-ds001.pdf&amp;data=3D05%7C01%7C%7C6bb7d63d627c49b946c4
> 08da449fd9bf%7C1b16ab3eb8f64fe39f3e2db7fe549f6a%7C0%7C0%7C6378977
> 51065729986%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjo
> iV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&amp;sdat
> a=3DK3PYyQAGsySCIjKuo1QRVm1HE0cuC3BVXbjuAwwhMjM%3D&amp;reserved=3D
> 0
> > Datasheet:
> https://eur02.safelinks.protection.outlook.com/?url=3Dhttps%3A%2F%2Fmedia=
.di
> gikey.com%2Fpdf%2FData%2520Sheets%2FBosch%2FBST-BMI090L-DS000-00.p
> df&amp;data=3D05%7C01%7C%7C6bb7d63d627c49b946c408da449fd9bf%7C1b1
> 6ab3eb8f64fe39f3e2db7fe549f6a%7C0%7C0%7C637897751065729986%7CUnk
> nown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1h
> aWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&amp;sdata=3D8GfqaDLkn5whi%2F
> bsEH9UATPNkJVgsy859sIifJGv%2BHg%3D&amp;reserved=3D0
>=20
> blank line here. These aren't part of the tags.
Thank you, Rob, I did check, the most datasheets are part of the tags,
and few of them has a blank line, do you agree to keep it?
>=20
> With that,
>=20
> Acked-by: Rob Herring <robh@kernel.org>
>=20
> > Signed-off-by: LI Qingwu <Qing-wu.Li@leica-geosystems.com.cn>
> > ---
> >  Documentation/devicetree/bindings/iio/accel/bosch,bmi088.yaml | 2 ++
> >  1 file changed, 2 insertions(+)
> >
> > diff --git a/Documentation/devicetree/bindings/iio/accel/bosch,bmi088.y=
aml
> b/Documentation/devicetree/bindings/iio/accel/bosch,bmi088.yaml
> > index 911a1ae9c83f..272eb48eef5a 100644
> > --- a/Documentation/devicetree/bindings/iio/accel/bosch,bmi088.yaml
> > +++ b/Documentation/devicetree/bindings/iio/accel/bosch,bmi088.yaml
> > @@ -17,7 +17,9 @@ description: |
> >  properties:
> >    compatible:
> >      enum:
> > +      - bosch,bmi085-accel
> >        - bosch,bmi088-accel
> > +      - bosch,bmi090l-accel
> >
> >    reg:
> >      maxItems: 1
> > --
> > 2.25.1
> >
> >