Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   "Miclaus, Antoniu" <Antoniu.Miclaus@analog.com>
To:     Rob Herring <robh@kernel.org>
CC:     "jic23@kernel.org" <jic23@kernel.org>,
        "linux-iio@vger.kernel.org" <linux-iio@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>
Subject: RE: [PATCH 3/4] dt-bindings:iio:filter: add admv8818 doc
Thread-Topic: [PATCH 3/4] dt-bindings:iio:filter: add admv8818 doc
Thread-Index: AQHX1WXROa8gBfces0OdmTnnZfx+/6v788IAgACOyBA=
Date:   Wed, 10 Nov 2021 09:39:38 +0000
Message-ID: <CY4PR03MB3399F82177CECAA5EB53F3699B939@CY4PR03MB3399.namprd03.prod.outlook.com>
References: <20211109123127.96399-1-antoniu.miclaus@analog.com>
 <20211109123127.96399-4-antoniu.miclaus@analog.com>
 <1636506426.248164.3016194.nullmailer@robh.at.kernel.org>
In-Reply-To: <1636506426.248164.3016194.nullmailer@robh.at.kernel.org>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?iso-8859-2?Q?0SKVLERFFzr2P/5Rk/8Zr9wdwb8jFHgapsK1lKDsqXEMmrzkvF7eXkR9yA?=
 =?iso-8859-2?Q?vIa4tVUCSGkg2hvJWD32FBVMwFhcKyyNMwretVyr2FW/Q5dkyajNhEGMQf?=
 =?iso-8859-2?Q?d2tqM42WXP9wB2lHpLH4/cWIfFItaxzC0+NU1wi5ldFhYOWrMAU4grKNlk?=
 =?iso-8859-2?Q?X8iRw4tjepNtQ0KXRS8ouwIdS8QmW34IFoaNjgEjY+bSCLdWpVRXt1TEyH?=
 =?iso-8859-2?Q?ikwZMZ3FKb6idLjzF1Qq+5hAd2H4VVlsoq5ibFZhtb/kG4X/wkvfFZjBcQ?=
 =?iso-8859-2?Q?rOOGo62FPJ7S6CJ6pq6385CcMxv3aM9CWkSgA3yIChFW3T8ff62KI0fmgP?=
 =?iso-8859-2?Q?+D/iupefYTiq5OgHFYm5yYzjLu2TciySTa3ONLZ4fqy+mrSu6mBsTU6xrW?=
 =?iso-8859-2?Q?urfn6O7Bw0rgsUzrWo+S7ZGDh6KPKozV85aHDK5hSgHTd7W96dbWDzNcle?=
 =?iso-8859-2?Q?UU1gRQEJPgpgM2mITJQQmEGIlgbX3g/xmON/WMIpsixj97B2hEScXBMfkW?=
 =?iso-8859-2?Q?87VzPRvPROTZBMtq9cumTFE4CCY6ZI7VnRiEoq0FkrhfWtSKvcGmE6eXfe?=
 =?iso-8859-2?Q?0ln6jrbWzolmqn6FnPcINOHn24jfQ0QGkhiDgfkbuZufmR2Io4k6LEDGo7?=
 =?iso-8859-2?Q?ObYMtqOX1lmcQn3bei07G7BAsPyuuSv3+UN/d6vFymYTE/f5O1zSx8wALd?=
 =?iso-8859-2?Q?y51gQmFbAe6NEV0nWss0OTnfZLb7PnGr8LNU1dTjqMs5uxLEzEUHNQDVb2?=
 =?iso-8859-2?Q?dzj7vN6ebTd0EfYic4vPiL7pZC+y3ubuiXTWHVHEkPRD6QI3qgW00/slGv?=
 =?iso-8859-2?Q?JfCTVsSEpYkgzBGYE48LP3u2lDXu5r1N1R8TFX6vSuwGpiDMT8IZ6M8fJv?=
 =?iso-8859-2?Q?G42dPbYzLq6MNDqNFsCvMZwLZb+n1BjIGt/hxQCVFXquPiSuBZ23YHgECe?=
 =?iso-8859-2?Q?kqXFgn97KFZlKmzs5agmzgm0xg1bwhSEHhBv9ON5bG4Kn6P1aCofrXLCVI?=
 =?iso-8859-2?Q?JQYbr49zyfk9Z13PmFOoXr+zVhesw4DGpci5NBx+C5L3Intaj6ca1X5ZwV?=
 =?iso-8859-2?Q?5zre3DxVkqzWox2iFeBWA2b3Ex1hDSS80Q62Ih3fuTYOMXD4LDn6G6F2Wn?=
 =?iso-8859-2?Q?xT79t0myjgtf2P1+lmW1zIhpBuokMBf4aF/U/dILNIDsT2GPzCxMyRFYTU?=
 =?iso-8859-2?Q?slq0Z4/Gvvd6wJjqOXYOiujrbtaqseSnVuICIaq1UXFtCsPii7PIoA8loo?=
 =?iso-8859-2?Q?blN1rjmxzkanzHBOhsgRMY0uZC/OYqt3vA3qmA6pQv3ZC0zjc0h3E9Jl2X?=
 =?iso-8859-2?Q?RWt6VHQ2HYvrKtJ2SvUYO0raH4bS2qOKtExOneFIdsOQxF3xwwNhenRKCz?=
 =?iso-8859-2?Q?SPCxVJFI8DUUQYT74m6mgQ5sEdgxBkXfXhYkr4y8w9H6LoM/7jkEKtDuYg?=
 =?iso-8859-2?Q?lw+Baqdjfhjl79uJodqc3u6yaOs2Xh5VPhcBH/FdDt5KnTC+UsrNeF/uRh?=
 =?iso-8859-2?Q?CRA4cnkPhVfs2UQqcj70pjNhUDv4fcdHoEQgF/v7VdTl9LMwnLn0NHKx3S?=
 =?iso-8859-2?Q?EFfkKqqbKbCXngRySyZVu4VRwif8IvgI+MowI1lfUlr+y/87VIRuyfVvcE?=
 =?iso-8859-2?Q?pSUU4wdijB64a9rmG13dwB/fi5jV7yFZ55eiXIM1FYKlYAxs9kZtZnlSzI?=
 =?iso-8859-2?Q?vYEIYFSzWGNGmKgZ7nX4exMGjWg/gKr4r0tGnynhHRN9DRzjTh8rXKuRkT?=
 =?iso-8859-2?Q?3qCg=3D=3D?=
Content-Type: text/plain; charset="iso-8859-2"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: analog.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: CY4PR03MB3399.namprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ae54c5d8-b7e3-424f-c9a3-08d9a42e03d7
X-MS-Exchange-CrossTenant-originalarrivaltime: 10 Nov 2021 09:39:38.7881
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: eaa689b4-8f87-40e0-9c6f-7228de4d754a
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: KNDo2GlnwOPxRSEM/Dk6MAi6xtnUwfwKG8owXKyKiZEMpR4jsUnXDXId8TXzc/gPMB0mLTffjkO6W/FYdH3+IMN8sS9af54c4s8/KuO4nmM=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR03MB2550
X-Proofpoint-ORIG-GUID: AbMfExOVfJt-MYRy84opTgIMHKD9kQEH
X-Proofpoint-GUID: AbMfExOVfJt-MYRy84opTgIMHKD9kQEH
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.205,Aquarius:18.0.790,Hydra:6.0.425,FMLib:17.0.607.475
 definitions=2021-11-10_03,2021-11-08_02,2020-04-07_01
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 impostorscore=0 clxscore=1015
 lowpriorityscore=0 priorityscore=1501 mlxlogscore=917 phishscore=0
 bulkscore=0 spamscore=0 malwarescore=0 adultscore=0 suspectscore=0
 mlxscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2110150000 definitions=main-2111100052
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


--
Antoniu Micl=E3u=BA

> -----Original Message-----
> From: Rob Herring <robh@kernel.org>
> Sent: Wednesday, November 10, 2021 3:07 AM
> To: Miclaus, Antoniu <Antoniu.Miclaus@analog.com>
> Cc: jic23@kernel.org; linux-iio@vger.kernel.org; linux-
> kernel@vger.kernel.org; devicetree@vger.kernel.org; robh+dt@kernel.org
> Subject: Re: [PATCH 3/4] dt-bindings:iio:filter: add admv8818 doc
>=20
> [External]
>=20
> On Tue, 09 Nov 2021 14:31:26 +0200, Antoniu Miclaus wrote:
> > Add device tree bindings for the ADMV8818 Filter.
> >
> > Signed-off-by: Antoniu Miclaus <antoniu.miclaus@analog.com>
> > ---
> >  .../bindings/iio/filter/adi,admv8818.yaml     | 78 +++++++++++++++++++
> >  1 file changed, 78 insertions(+)
> >  create mode 100644
> Documentation/devicetree/bindings/iio/filter/adi,admv8818.yaml
> >
>=20
> My bot found errors running 'make DT_CHECKER_FLAGS=3D-m
> dt_binding_check'
> on your patch (DT_CHECKER_FLAGS is new in v5.13):
>=20
> yamllint warnings/errors:
>=20
> dtschema/dtc warnings/errors:
> /builds/robherring/linux-dt-
> review/Documentation/devicetree/bindings/iio/filter/adi,admv8818.yaml:
> properties:adi,bw-hz: '$ref' should not be valid under {'const': '$ref'}
> 	hint: Standard unit suffix properties don't need a type $ref
> 	from schema $id:
> https://urldefense.com/v3/__http://devicetree.org/meta-
> schemas/core.yaml*__;Iw!!A3Ni8CS0y2Y!tVywX0fQFR2vYZq-
> YTIJiJ0AXF2WapK4hXuLoFCYYlg19vxsZrLtIe7gWW7NfqCnlwuk$
> /builds/robherring/linux-dt-
> review/Documentation/devicetree/bindings/iio/filter/adi,admv8818.yaml:
> ignoring, error in schema: properties: adi,bw-hz
> warning: no schema found in file:
> ./Documentation/devicetree/bindings/iio/filter/adi,admv8818.yaml
> Documentation/devicetree/bindings/iio/filter/adi,admv8818.example.dt.ya
> ml:0:0: /example-0/spi/admv8818@0: failed to match any schema with
> compatible: ['adi,admv8818']
>=20
> doc reference errors (make refcheckdocs):
>=20
> See
> https://urldefense.com/v3/__https://patchwork.ozlabs.org/patch/1552959_
> _;!!A3Ni8CS0y2Y!tVywX0fQFR2vYZq-
> YTIJiJ0AXF2WapK4hXuLoFCYYlg19vxsZrLtIe7gWW7NfqkOdOfT$
>=20
> This check can fail if there are any dependencies. The base for a patch
> series is generally the most recent rc1.
>=20
> If you already ran 'make dt_binding_check' and didn't see the above
> error(s), then make sure 'yamllint' is installed and dt-schema is up to
> date:
>=20
> pip3 install dtschema --upgrade
>=20
> Please check and re-submit.

Thanks!
I will resubmit this patch in v2 after the driver is reviewed.

Regards,
Antoniu
=20