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 089BCEB64D9 for ; Tue, 27 Jun 2023 14:23:07 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231421AbjF0OXG (ORCPT ); Tue, 27 Jun 2023 10:23:06 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:60824 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231208AbjF0OXF (ORCPT ); Tue, 27 Jun 2023 10:23:05 -0400 Received: from mail-ej1-x62c.google.com (mail-ej1-x62c.google.com [IPv6:2a00:1450:4864:20::62c]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 94DD92D67 for ; Tue, 27 Jun 2023 07:23:03 -0700 (PDT) Received: by mail-ej1-x62c.google.com with SMTP id a640c23a62f3a-9923833737eso58863666b.3 for ; Tue, 27 Jun 2023 07:23:03 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20221208.gappssmtp.com; s=20221208; t=1687875782; x=1690467782; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=ukf8kcwkmzk9LYwqXTolLjKgBXXfv/auIr6rnxGGo+0=; b=lDipDssM6fhnzbxP/+otLf3ADAtXGyAWRBBj5ozeFqK8p3nHDp9UbZapzYCqpThk7A V1J32lVWt4v2aH4iJoJdRTJogfKlEk4UUtAz2pYJ9gkn/KsL1MlfUCOlVjqi+Gmc43CR XFwHayAXhaF4YsUIm3g1AHDKCYA8Q0kmdoUy1pRvvObgIV565hclnHeyYr9feSaFlnSd Wnzx5fFl/9mbdLvb6v/D4KxnfDiGfYQZUbadrZfvgtpKfVY+0LhOTGuxZSUWxzZM433i HRx164LS4cDISfoD7dPe6Fc2UfktqMo01/1RtNb/AMUx28tqCsWrljq5OKogYlTi9/SZ qzxg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1687875782; x=1690467782; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=ukf8kcwkmzk9LYwqXTolLjKgBXXfv/auIr6rnxGGo+0=; b=kVJfzg4kyZMGjVeprFiRuK6rG8zk4TjuCljvYfQakoWkcuT1IyxoKMv2g/ghtVAP4o i1n/MJL9H6KXGIXDhAjPomLSG2lLxnMrrJf6/ifzcIhSWdSJ2SRC7bYa8OG2KX5l3wtN h1Lk613yOfY2ue+GcuJJiy4w6Zz6ne9SRUbW3sd4YHpIawXXjCOP1WGMJedxjyBZ+pLF uRZw7/xp6nnlHXVE+NzN3WxQxq0tV5+wN0+NwN8CJvGkZG7lQFowY5xVv62hv4txyZ4O 7TQLQgZGwf0bIlCuHgjsZteD4THXTNKJqf0m4figtjTvd5QanuI29HGLTJv9AqrjBVvg 7KjQ== X-Gm-Message-State: AC+VfDzfU7UgwgAW+ae+dyfjlUkuTvALMKBNaM3O4SIN2YI7KsdUY/he QlWQPXH65v1sFBQgFezp92IqOw== X-Google-Smtp-Source: ACHHUZ6OV19c18Cv0nNU8zXxPholYkVouLpnSEi8QQ79pSZZvEUYZC9ugwMwAT4QmdwLm8ARWM1TxA== X-Received: by 2002:a17:907:d1a:b0:988:e223:9566 with SMTP id gn26-20020a1709070d1a00b00988e2239566mr23513151ejc.62.1687875782075; Tue, 27 Jun 2023 07:23:02 -0700 (PDT) Received: from blmsp ([2001:4091:a247:82fa:b762:4f68:e1ed:5041]) by smtp.gmail.com with ESMTPSA id qh15-20020a170906ecaf00b009885462a644sm4575596ejb.215.2023.06.27.07.23.00 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 27 Jun 2023 07:23:01 -0700 (PDT) Date: Tue, 27 Jun 2023 16:23:00 +0200 From: Markus Schneider-Pargmann To: Krzysztof Kozlowski , Marc Kleine-Budde Cc: Wolfgang Grandegger , Marc Kleine-Budde , Rob Herring , Krzysztof Kozlowski , "David S . Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Conor Dooley , Chandrasekar Ramakrishnan , Michal Kubiak , Vivek Yadav , linux-can@vger.kernel.org, netdev@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Simon Horman Subject: Re: [PATCH v2 5/6] can: tcan4x5x: Add support for tcan4552/4553 Message-ID: <20230627142300.heju4qccian5hsjk@blmsp> References: <20230621093103.3134655-1-msp@baylibre.com> <20230621093103.3134655-6-msp@baylibre.com> <32557326-650c-192d-9a82-ca5451b01f70@linaro.org> <20230621123158.fd3pd6i7aefawobf@blmsp> <21f12495-ffa9-a0bf-190a-11b6ae30ca45@linaro.org> <20230622122339.6tkajdcenj5r3vdm@blmsp> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Hi Krzysztof, On Thu, Jun 22, 2023 at 02:52:48PM +0200, Krzysztof Kozlowski wrote: > On 22/06/2023 14:23, Markus Schneider-Pargmann wrote: > >> > >> Yeah, but your code is different, although maybe we just misunderstood > >> each other. You wrote that you cannot use the GPIOs, so I assumed you > >> need to know the variant before using the GPIOs. Then you need > >> compatibles. It's not the case here. You can read the variant and based > >> on this skip entirely GPIOs as they are entirely missing. > > > > The version information is always readable for that chip, regardless of > > state and wake GPIOs as far as I know. So yes it is possible to setup > > the GPIOs based on the content of the ID register. > > > > I personally would prefer separate compatibles. The binding > > documentation needs to address that wake and state GPIOs are not > > available for tcan4552/4553. I think having compatibles that are for > > these chips would make sense then. However this is my opinion, you are > > the maintainer. > > We do not talk about compatibles in the bindings here. This is > discussion about your driver. The entire logic of validating DTB is > flawed and not needed. Detect the variant and act based on this. I thought it was about the bindings, sorry. So to summarize the compatibles ti,tcan4552 and ti,tcan4553 are fine. But the driver should use the ID register for detection and not compare the detected variant with the given compatible? In my opinion it is useful to have an error messages that says there is something wrong with the devicetree as this can be very helpful for the developers who bringup new devices. This helps to quickly find issues with the devicetree. @Marc: What do you think? Maybe I misinterpreted your mail from last version? Best, Markus