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 99845EE49A4 for ; Sun, 20 Aug 2023 06:56:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229789AbjHTG4T (ORCPT ); Sun, 20 Aug 2023 02:56:19 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:49646 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230136AbjHTG4E (ORCPT ); Sun, 20 Aug 2023 02:56:04 -0400 Received: from mail-lf1-x136.google.com (mail-lf1-x136.google.com [IPv6:2a00:1450:4864:20::136]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 1172C359A for ; Sat, 19 Aug 2023 23:29:12 -0700 (PDT) Received: by mail-lf1-x136.google.com with SMTP id 2adb3069b0e04-4ff8f2630e3so3545001e87.1 for ; Sat, 19 Aug 2023 23:29:11 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1692512950; x=1693117750; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=uc6r5jrTl5KRxyULV2rwJI3udN950yCQjtWBO1UHlTE=; b=A188BHQPbmwpEsQTM+O3IWYN/sJ+IGMzg0pNy7H09QX3+hs1UJ03dS701+yBeYbWsz vQCr34Ui9zvNsIc+BMqDA5ppu5HVfTaUCNmYJsfkMqxUkFI2CFdRhsFAVNm68Eq4fdew rivzw9rDqunEQt/FH7Lr6TS/mEEbFNpI69sBt4+zqQgcB1IqE5TPVQMbOJ51e5UTDhTV bdNfuylUfspFU6T8I23s9pppWFgfp9Hu8yACDrXA/QJ3xNhTggfM8KdBxavQKf/vaGn2 f1AUrpufiIJDoCnG8htpXOwOhjhW5cVqzDZAT5PNN6E7PKtUIai9MK4xtGUzhtfsK3Br STsw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1692512950; x=1693117750; h=content-transfer-encoding:in-reply-to:from:references:cc:to :content-language:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=uc6r5jrTl5KRxyULV2rwJI3udN950yCQjtWBO1UHlTE=; b=Fdi3Wj7GR+OZppYHn5pcVU8Bjtbnb+JBXwLs2AcFTyqH+0DWCM134Y//FLRB0IVHlj AMMlViHhe2j0L9HzHKCA2KQTxKbkkWc5MeW3MaZjfHDyf3gORUQDINCRsuMrblop6SHK 5zCBRZis4VA6/nOuIZmAlQLP2Qnd7QJHI8wZUAnLR0YKdUxEO+q2+raRvEJrcpqaONfz 4TQCZwTQh7h/NSHq9iEko5kh2ZGfT8EPGXDBN3lOtGBciImu2QuBoR8dnRWy2KEZ4iw3 1xYJcicMClI9BdE4LmzrAJBlz7QbocstUraESO3D9KV3ElL89ZCgWc5JTrNPErJdCCPk XEEg== X-Gm-Message-State: AOJu0YyrZXYDFrYFDRS8kdWwy15qvm0vVbSC1S6qK313xQpy/j/C6SvC H3J2lF3iGgUJNajXtiyKGRKu4g== X-Google-Smtp-Source: AGHT+IHGiLfTKXd3POBlHix4H5KyMKF6QEFcujN4xuJyCP7MzrXaQscwFy+zspuseAs8dz2GHwW3BA== X-Received: by 2002:a05:6512:39ca:b0:4fe:82a7:814d with SMTP id k10-20020a05651239ca00b004fe82a7814dmr2334975lfu.48.1692512949514; Sat, 19 Aug 2023 23:29:09 -0700 (PDT) Received: from [192.168.0.22] ([77.252.47.198]) by smtp.gmail.com with ESMTPSA id sb25-20020a170906edd900b0098ec690e6d7sm4029867ejb.73.2023.08.19.23.29.08 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Sat, 19 Aug 2023 23:29:09 -0700 (PDT) Message-ID: Date: Sun, 20 Aug 2023 08:29:08 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.14.0 Subject: Re: [PATCH] dt-bindings: iio: adc: Add TI TWL603X GPADC Content-Language: en-US To: Andreas Kemnade Cc: jic23@kernel.org, lars@metafoo.de, robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org, conor+dt@kernel.org, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-omap@vger.kernel.org References: <20230816202614.324457-1-andreas@kemnade.info> <426d78d6-9fa6-bfeb-b36a-fba264097a27@linaro.org> <20230819221903.726a1c39@aktux> From: Krzysztof Kozlowski In-Reply-To: <20230819221903.726a1c39@aktux> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On 19/08/2023 22:19, Andreas Kemnade wrote: >>> +--- >>> +$id: http://devicetree.org/schemas/iio/adc/ti,twl6030-gpadc.yaml# >>> +$schema: http://devicetree.org/meta-schemas/core.yaml# >>> + >>> +title: GPADC subsystem in the TWL6030 power module >>> + >>> +maintainers: >>> + - Jonathan Cameron >> >> This should be rather someone knowing or having or caring about this >> particular hardware, not subsystem maintainer. >> > Hmm, I have the twl6032, but not the twl6030. So probably > Tony (OMAP-Maintainer) or me? Yes. If you have a device, it's even better, but "caring about" or having datasheet is enough. > >>> + >>> +description: >>> + The GPADC subsystem in the TWL6030 consists of a 10-bit ADC >>> + combined with a 15-input analog multiplexer. >>> + >>> +properties: >>> + compatible: >>> + const: ti,twl6030-gpadc >> >> Devices look fairly similar. Same properties. Why aren't they in one >> binding (enum here instead)? >> > I hope it can be done. See commit message. Maybe my reasoning is wrong. The parent device binding can expect the compatible for the child and it will have the same effect in total as $ref to this binding. The only difference would be that running dtbs_check on parent binding would not spot all the issues in the child node. One need to run dtbs_check with both bindings. For an example: Documentation/devicetree/bindings/display/msm/qcom,sm8450-mdss.yaml Best regards, Krzysztof