From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 6A61AC19F2B for ; Wed, 3 Aug 2022 21:40:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S236186AbiHCVkH (ORCPT ); Wed, 3 Aug 2022 17:40:07 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52920 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S234745AbiHCVkG (ORCPT ); Wed, 3 Aug 2022 17:40:06 -0400 Received: from mail-io1-f50.google.com (mail-io1-f50.google.com [209.85.166.50]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id C8BB340BE2; Wed, 3 Aug 2022 14:40:05 -0700 (PDT) Received: by mail-io1-f50.google.com with SMTP id e69so13901457iof.5; Wed, 03 Aug 2022 14:40:05 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc; bh=PoAbO2TJzV0lAh1cQiBoTvQXqo+RgOhm6j9tuBM+bjI=; b=O40NjtNtw/dd9gbcbHz9QJsero9lCCmpjjlsgLNi46TQ9klvRBJPk4Qnbj6u0lOTVB UYZ6if6R+AyLyTVfYvzhdcHK5zjshLSlUYCql5CE+Zx1fDyRVfuWFQ4Getk7kvgfynYA DsSmQTO/Utfm/SyWRgoGlxjGV4BoXwd7GUr/NJsCsEo9zq7ZvqgKZ3xP4g3v8DUm/RWf 1OV+NGs6jRYlhsNtVJZzow2hd4iTB6DD3mzJfHEHUdtGKnQhkiEDTrVlc9JuBlABH9nU OLvNqeXAQNPGV5oh7ozL776xAnZX7P0SKLlficJA3G8wuuR/5rsl0vtUg3H5t+qmZYui QH4g== X-Gm-Message-State: ACgBeo3lMl5AAayu0tXVl9nGrEBbo/LQCP/C65YSjqw9M2sBFOqI257u l9EbwfiiTINJXdXPvURcKA== X-Google-Smtp-Source: AA6agR74TqLpu6rXFlxDzpXKcwaDch4+hV9VRV/8owxn5cc8tUiYTeoIQKDKrn8CMLWJddeDtM0HWg== X-Received: by 2002:a05:6602:154c:b0:67f:c11a:6d7c with SMTP id h12-20020a056602154c00b0067fc11a6d7cmr2511343iow.176.1659562805005; Wed, 03 Aug 2022 14:40:05 -0700 (PDT) Received: from robh.at.kernel.org ([64.188.179.248]) by smtp.gmail.com with ESMTPSA id m7-20020a92c527000000b002dc3df1b345sm7339451ili.88.2022.08.03.14.40.01 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 03 Aug 2022 14:40:04 -0700 (PDT) Received: (nullmailer pid 2667120 invoked by uid 1000); Wed, 03 Aug 2022 21:40:00 -0000 Date: Wed, 3 Aug 2022 15:40:00 -0600 From: Rob Herring To: Lukas Wunner Cc: Krzysztof Kozlowski , Lars-Peter Clausen , Michael Hennerich , Jonathan Cameron , Krzysztof Kozlowski , Lucas Stankus , Puranjay Mohan , Cosmin Tanislav , Alexandru Tachici , Marcelo Schmitt , Marcus Folkesson , Kent Gustavsson , Tomislav Denis , Oleksij Rempel , kernel@pengutronix.de, Antoniu Miclaus , Ricardo Ribalda , Dragos Bogdan , Jean-Baptiste Maneyrol , Lorenzo Bianconi , Stefan Popa , Linus Walleij , Stephan Gerhold , Mike Looijmans , Joachim Eastwood , Tomas Melin , Sean Nyekjaer , Beniamin Bia , Patrick Vasseur , Charles-Antoine Couret , Vladimir Barinov , Miquel Raynal , Philippe Reynes , Jacopo Mondi , Akinobu Mita , Alexandru Lazar , Oskar Andero , =?UTF-8?Q?M=C3=A5rten_Lindahl?= , Bogdan Pricop , Angelo Compagnucci , Matt Ranostay , Dan Murphy , Gregory Clement , Phil Reid , Michael Welling , Robert Jones , Chris Coffey , Slawomir Stepien , Sankar Velliangiri , linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-fbdev@vger.kernel.org Subject: Re: [PATCH v2 00/10] dt-bindings: iio: use spi-peripheral-props.yaml Message-ID: <20220803214000.GB2639296-robh@kernel.org> References: <20220727164646.387541-1-krzysztof.kozlowski@linaro.org> <20220730224643.GB11662@wunner.de> <20220801160410.GA6059@wunner.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20220801160410.GA6059@wunner.de> Precedence: bulk List-ID: X-Mailing-List: linux-fbdev@vger.kernel.org On Mon, Aug 01, 2022 at 06:04:10PM +0200, Lukas Wunner wrote: > On Mon, Aug 01, 2022 at 05:45:07PM +0200, Krzysztof Kozlowski wrote: > > On 31/07/2022 00:46, Lukas Wunner wrote: > > > On Wed, Jul 27, 2022 at 06:46:36PM +0200, Krzysztof Kozlowski wrote: > > >> 78 files changed, 324 insertions(+), 249 deletions(-) > > > > > > Pardon me for being dense, but what is the benefit of this series > > > that justifies inflating the schema definitions by a total of 75 lines? > > > > The commits were explaining rationale, so let me bring it here. The > > benefits are: > > This allows using all properties typical for SPI-connected devices, even > > these which device bindings author did not tried yet. > > How do you know these untested properties work with the devices to which > you're adding them? How do we know anything DT works? We don't without testing on h/w. That's not what the schemas provide. The spi-peripheral-props.yaml reference is needed in order to allow controller specific timing properties and to prevent random other undocumented properties from being present. There is not another way to do both of those. Do I wish we didn't have these controller specific timing parameters, yes! But that ship has sailed. Rob