Received: by 2002:a05:7412:b130:b0:e2:908c:2ebd with SMTP id az48csp2266185rdb; Mon, 20 Nov 2023 06:43:17 -0800 (PST) X-Google-Smtp-Source: AGHT+IHbnD/KQx33SB2BVNHeVqVOCSw1yihMyRKFc39KdGzUAJc3KmQcD+gcE8vmaZJjd4bhWhJN X-Received: by 2002:a05:6a20:4289:b0:189:fa8a:bd2 with SMTP id o9-20020a056a20428900b00189fa8a0bd2mr5039833pzj.12.1700491397513; Mon, 20 Nov 2023 06:43:17 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1700491397; cv=none; d=google.com; s=arc-20160816; b=VeqAtt1PUjlKvO6fnyn8vg2Z2lZmIHzIqi9C/1zKFcJ6t5kqG+ohbIDgBqc44PcARB 7bL74O1aQz7Y9xgdFlfFYSxxVsly/Kld7lYzaqVWlypjEgWVYVwXVw2O8KQ8Z1OtZTlC /rg7nPrBWMq16Eq+pJqIP4pzGklFM7ipgcQAW0KCXqACxN56q9zNC9Hq87OKdubGlDrc 4nduyeQdEAc5W8MfDFinpKUt3vnU0MRcM2Sc1guHLTlya66f+8yYPnPHxrVgTc8X1lQp lBJbOr3XpDYEjYzcCNY/BxhItKHmSj+yPzapw8nu0ZdBLuHY1AFpaT02wMks8/eRpbQC 9OZA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=CB7qKwV2FUo678XoqETFjSae7tQDhDCV9XuIbu+Umgo=; fh=nrXCPfhvtowACcwf5y39+LUpquKJW9AdrOeyRxe5AnA=; b=mj9mScQzaGrWhx0AvRJA8AaKo1ZsVSPS3idjBIaC7SDkHJt5ww928v5DC0qJ/DxcxG oaePDzpUq/rAL0mv+VPAy/vI8U6MQjFJj7si5Sf9yUovlEZYjk7kXA/4tUOn/6MPjBgD zNth/lHmuf/g+EBvJhGuvp7CrYqfOXNeyoMuDI7z0f6PIPfz9zhZJkFeLorED8e4jT3a x6ZItoqUZocBK9TywZbQh+xcRT9PKFqf7q+aSGiYHZlvCpuAldQgzDeFaYLzrIrFfPRt pqh34A+iWmpEqjqEE6e1bu3OeexB4WhxhNc2PyA03LUJuJ2tSvrgZYflJ5MS/Lc6RIqg /4rQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@subdimension.ro header.s=skycaves header.b="OE1vLD/7"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=subdimension.ro Return-Path: Received: from lipwig.vger.email (lipwig.vger.email. [23.128.96.33]) by mx.google.com with ESMTPS id n10-20020a170902d2ca00b001cc407388a8si4280874plc.337.2023.11.20.06.43.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 20 Nov 2023 06:43:17 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33; Authentication-Results: mx.google.com; dkim=pass header.i=@subdimension.ro header.s=skycaves header.b="OE1vLD/7"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=subdimension.ro Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by lipwig.vger.email (Postfix) with ESMTP id 147E88067E12; Mon, 20 Nov 2023 06:41:28 -0800 (PST) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.11 at lipwig.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S233226AbjKTOlB (ORCPT + 99 others); Mon, 20 Nov 2023 09:41:01 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40060 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230129AbjKTOk7 (ORCPT ); Mon, 20 Nov 2023 09:40:59 -0500 Received: from mail.subdimension.ro (skycaves.subdimension.ro [172.104.132.142]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 02A46A4; Mon, 20 Nov 2023 06:40:54 -0800 (PST) Received: from sunspire (unknown [188.24.94.216]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (prime256v1) server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mail.subdimension.ro (Postfix) with ESMTPSA id 4CF5B28EE6F; Mon, 20 Nov 2023 14:40:53 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=subdimension.ro; s=skycaves; t=1700491253; bh=LZcQN0TV9+lj9l4R7rlzQvmMSI32m2CgUuGmtP41lVg=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=OE1vLD/7KzBE8RbeLZZpdiYT7ke6zXjS4+9zU+J7Leev6ut2AAyXMbTRCHEtkTRtv 21ZbzmEGQhJgni3+H3bs1xIL6A8WjRqH4O5n3EYbWu+4KZ2j4838rmccWyNyO6DFQk Pne+WpkNO0jdeaQg62tLstSC/fzMQZaXTx01oelY= Date: Mon, 20 Nov 2023 16:40:51 +0200 From: Petre Rodan To: Krzysztof Kozlowski Cc: linux-kernel@vger.kernel.org, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, Conor Dooley , Lars-Peter Clausen , Rob Herring , Krzysztof Kozlowski , linux-kernel-mentees@lists.linuxfoundation.org, Jonathan Cameron Subject: Re: [PATCH v2 1/2] dt-bindings: iio: pressure: add honeywell,hsc030 Message-ID: References: <20231117164232.8474-1-petre.rodan@subdimension.ro> <20231117192305.17612-1-petre.rodan@subdimension.ro> <5b2e4b05-9408-48ea-92ac-15883e102013@linaro.org> <8aa11758-5eee-427b-adcb-c3578a521daf@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <8aa11758-5eee-427b-adcb-c3578a521daf@linaro.org> X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Mon, 20 Nov 2023 06:41:28 -0800 (PST) Hello! On Mon, Nov 20, 2023 at 03:04:07PM +0100, Krzysztof Kozlowski wrote: > On 20/11/2023 14:42, Petre Rodan wrote: > > >>> +properties: > >>> + compatible: > >>> + enum: > >>> + - honeywell,hsc > >> > >> Way too generic > > > > I'm new to this, please excuse my ignorance. > > my driver covers all Honeywell pressure sensors under the "TruStability board mount HSC/SSC" moniker. > > We talk here about bindings, not driver. For the driver you can use > whatever name is approved by reviewers of your driver. > > > that is why my intention was to provide a rather generic name for the driver itself. > > are you afraid that they will come up with a different device that they will call "hsc" in the future? > > in this case honeywell,trustability-hsc would be fine? > > > > as I see you prefer to target a particular chip, but I am a bit afraid that the end-user will be confused by needing to set up something like > > > > pressure@28 { > > compatible = "honeywell,hsc030pa"; > > The compatible should be specific, thus for example match exact model > number. there are an infinite number of combinations of 4 transfer functions and 118 ranges + one custom range, so providing an array with all specific chips that could end up as compatible is out of the question. I was aiming at providing a generic name for the binding and get the transfer function and the pressure range as required parameters. > If you can guarantee that all devices from given family are the same in > respect of programming model and hardware requirements (e.g. supplies), > then you could go with family name. However such guarantees are rarely > given. I see your point. > Therefore for mprls0025pa I agreed for using one specific model > for entire family. > > https://lore.kernel.org/all/d577bc44-780f-f25d-29c6-ed1d353b540c@linaro.org/ > > > > reg = <0x28>; > > honeywell,transfer-function = <0>; > > honeywell,pressure-range = "250MD"; > > }; > > > > ie. specifying "hsc030pa" as driver while his chip is not in the 030PA range, but 250MD. > > > > so do you prefer > > honeywell,trustability-hsc OR > > honeywell,hsc030pa > > I think the latter, just like we did for mprls0025pa. How many devices > do you have there? both hsc and ssc have 118 ranges, 4 transfer functions and both can be requested from the manufacturer with custom measurement ranges. ok,I will rename hsc->hsc030pa in the code as you requested. > >>> + honeywell,pmin-pascal: > >>> + description: | > >>> + Minimum pressure value the sensor can measure in pascal. > >>> + To be specified only if honeywell,range_str is set to "NA". > >>> + $ref: /schemas/types.yaml#/definitions/int32 > >> > >> That's uint32. Why do you need negative values? > > > > signed int32 is intentional. some chips have two physical input ports and measure a pressure differential in which case pmin is negative. > > see either of the pdfs at page 14, table 8, column 2, row 7+ > > Then the best would be to change the type in other bindings to have > int32 everywhere... but dtschema also requires unt32: > https://github.com/devicetree-org/dt-schema/blob/main/dtschema/schemas/property-units.yaml yeah, and also '-kpascal' might be too coarse of a unit when we talk about sensors that have a -125 .. 125 pascal measurement range as '0.5ND' has. cheers, peter -- petre rodan