Received: by 2002:a05:6358:9144:b0:117:f937:c515 with SMTP id r4csp6633005rwr; Tue, 2 May 2023 03:21:01 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ7HDfYxvwZc3PuMdrZ+EwYTh7MlVr3MPBn4cokImV8vnbJCZYGM7jMBjcn5eGkWtAn0BIn4 X-Received: by 2002:a17:902:f610:b0:1a9:7c6d:abb with SMTP id n16-20020a170902f61000b001a97c6d0abbmr18103929plg.29.1683022861213; Tue, 02 May 2023 03:21:01 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1683022861; cv=none; d=google.com; s=arc-20160816; b=vnRO9k/cUJ4m03lOJ6J6//jbUhiX90PWCTOLBoJFS3RtzpOE/KJ+GuKqse+tYyKRA2 BGnPDcOpK5565pd6G5UjZAbFomYHbQ+/IEqqvfrfGtp6bZCr2nHI4f4W/YT46MSYtTDv TO7gFZyIrQNzhowcXaLtnbf9phdLyOeEBFpW1U849Vm8xwUdKat9n/oWxywoLKpyGxBB fQ72GdtjNM1G3I6u486gR/LnaZNaMFgFEJUi/Ftw2R12nK72XHIbiNUKHjBxbi/W0AXM P8Ce0K6bBq7jSDFuUV+URmFwmwl8vzkrF5raXmdtgWaO+C/wzEcXi0l/GaNZDUBRQNS3 8rYw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to:from :references:cc:to:content-language:subject:user-agent:mime-version :date:message-id:dkim-signature; bh=PSOQWv5cIhWugBKgY0Tf8haO/+bAiIBjy0ljcFCS7pE=; b=bP4ZW11vpIFVhkqWR36IWfiG6ukcu48FVut40OUxU0BDpM2UnlgwUJLA9N7owgVKuZ zUOlvk/eF/21C4mSIAlTqptifC7c409e6c6mkpa0FpyLT3yOnAugpWpR8s0sYH5LW8Bq R0yYfbwMnK0krqqnLdOc93K3TOhxxZqyFCKfsj6ALM0nDs4kAgIlKvbHPFrdLjBavWlu dwjm+0LoGR+VD6cERmlV4gNo7vJnSyzLSnhdt7K98SWQm06Y95AXQs3VIxEsXWLCVTJe BlSAat3jAqG95/vW4ELKGPuE39PgwcnSZI3q1eN1C4NuikKqisqiGUoDJ5Hs66mm+Ihq y4sw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=sU4YH9jO; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id jd15-20020a170903260f00b001aafbb3e06dsi3552175plb.484.2023.05.02.03.20.46; Tue, 02 May 2023 03:21:01 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org header.s=google header.b=sU4YH9jO; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=linaro.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229703AbjEBKTf (ORCPT + 99 others); Tue, 2 May 2023 06:19:35 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57732 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233657AbjEBKTM (ORCPT ); Tue, 2 May 2023 06:19:12 -0400 Received: from mail-ed1-x52d.google.com (mail-ed1-x52d.google.com [IPv6:2a00:1450:4864:20::52d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id EC70E5597 for ; Tue, 2 May 2023 03:19:10 -0700 (PDT) Received: by mail-ed1-x52d.google.com with SMTP id 4fb4d7f45d1cf-50bcc565280so1901834a12.2 for ; Tue, 02 May 2023 03:19:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1683022749; x=1685614749; 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=PSOQWv5cIhWugBKgY0Tf8haO/+bAiIBjy0ljcFCS7pE=; b=sU4YH9jOZeGVjyZ1m4F/RektzfkMf2vg9+4cIdsuJaEOX/tAdSZjMllV324DlTO4is 3BVDhUi3OnaY5mVCoDnuClb/yZ+UXhfMtl8Hw0Hw9LRhwxvcVWx/Jgu+DHUxr/rxFIo6 j38ucgy3ttImNwyrLZzyaSQv2VPW7ugbSoaw+zAleDh1lepsNQXyENSYJXEvDzQ9r2xi Ra7ptpkuNvOZjqkh7EH5yboRJ/5veLuJr2nOB98dSWOpxlWjrPctyxwgoBj3l+ytl4jn HvAOE5WdLxZECWEO4aoKxzZivNqaP+6P3way5gkg5/pL9AfdZz1nRqNLYwpKrZbDtbbh BnQg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1683022749; x=1685614749; 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=PSOQWv5cIhWugBKgY0Tf8haO/+bAiIBjy0ljcFCS7pE=; b=Cwqga4KvgRcUDRVI/yFvuNJ+cST7ZR/e4KcokOFuRUiSfljuypabjtSUTiAB0NGxPY SkQRTSZdXYxAB7626u00aDl8QaYy+zBVjwNMZzSOBeGGA541eEo3mDu8DMOXDHOyl1iF 1bp4Ingi4dOk8jccuEVr1719NRLC91px9EL+VhPIhNMXIEK2benNR2DTRWi8fxOcCL2m IlE3eGyJb84KP3KU8/3geI0wQ7EUQOM3TpMbJ+jwjS0t33S0/62ruRzC0ZBnyGRdfg0W XG40dcc6xbsKHY8Q+o21CT4VUbGMciGxe5nleGeQXenaKE0lkGDFVwffpMoO9WQhIAdg nYDg== X-Gm-Message-State: AC+VfDzMo56g3FR0Nc35o2xZInfCdUy/bVFQiXwo00+m1wcHFMnKlcQK LFkMhVx4CP/LTRqJc55/N1eGbA== X-Received: by 2002:a05:6402:8:b0:504:8c1a:70db with SMTP id d8-20020a056402000800b005048c1a70dbmr7705427edu.32.1683022749367; Tue, 02 May 2023 03:19:09 -0700 (PDT) Received: from ?IPV6:2a02:810d:15c0:828:bafd:1283:b136:5f6a? ([2a02:810d:15c0:828:bafd:1283:b136:5f6a]) by smtp.gmail.com with ESMTPSA id u10-20020a056402064a00b00506addaaab0sm13225333edx.32.2023.05.02.03.19.08 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 02 May 2023 03:19:08 -0700 (PDT) Message-ID: <59b9214a-19e6-fd4e-6982-6e8c28c7e4a8@linaro.org> Date: Tue, 2 May 2023 12:19:07 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.10.1 Subject: Re: [PATCH 1/2] dt-bindings: Add optional gpio property to uartps node to support rs485 Content-Language: en-US To: "Guntupalli, Manikanta" , Rob Herring Cc: "gregkh@linuxfoundation.org" , "krzysztof.kozlowski+dt@linaro.org" , "michal.simek@xilinx.com" , "linux-serial@vger.kernel.org" , "devicetree@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "jirislaby@kernel.org" , "linux-arm-kernel@lists.infradead.org" , "Simek, Michal" , "git (AMD-Xilinx)" , "Pandey, Radhey Shyam" , "Datta, Shubhrajyoti" , "Goud, Srinivas" , "manion05gk@gmail.com" References: <1682512187-8828-1-git-send-email-manikanta.guntupalli@amd.com> <1682512187-8828-2-git-send-email-manikanta.guntupalli@amd.com> <20230427164351.GA3146210-robh@kernel.org> From: Krzysztof Kozlowski In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-3.5 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE, SPF_HELO_NONE,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/05/2023 12:14, Guntupalli, Manikanta wrote: > Hi Rob Herring, > >> -----Original Message----- >> From: Rob Herring >> Sent: Thursday, April 27, 2023 10:14 PM >> To: Guntupalli, Manikanta >> Cc: gregkh@linuxfoundation.org; krzysztof.kozlowski+dt@linaro.org; >> michal.simek@xilinx.com; linux-serial@vger.kernel.org; >> devicetree@vger.kernel.org; linux-kernel@vger.kernel.org; >> jirislaby@kernel.org; linux-arm-kernel@lists.infradead.org; Simek, Michal >> ; git (AMD-Xilinx) ; Pandey, >> Radhey Shyam ; Datta, Shubhrajyoti >> ; Goud, Srinivas ; >> manion05gk@gmail.com >> Subject: Re: [PATCH 1/2] dt-bindings: Add optional gpio property to uartps >> node to support rs485 >> >> On Wed, Apr 26, 2023 at 05:59:46PM +0530, Manikanta Guntupalli wrote: >>> Add optional gpio property to uartps node and add reference to rs485.yaml >> >> The diff tells me that already. Why? > Please check Figure 11-3 (Page number 37) in the below mentioned data sheet, It shows PMOD (ISOW14x2) devices used in half duplex configuration. Driver outputs Y and Z are shorted to A and B respectively. This reduces overall cabling requirements. Also DE/RE are shorted to each other, and at a time, any node acts as either a driver or a receiver. > Using above optional GPIO controlling driver or a receiver. This GPIO is optional because it is not required for uart console node. > > Here, > DE - Driver enable. If pin is floating, driver is disabled (internal pull-down resistor) > RE - Receiver enable. If pin is floating, receiver buffer is disabled (internal pull-up resistor) > > Datasheet: > https://www.ti.com/lit/ds/symlink/isow1432.pdf?ts=1682607122706&ref_url=https%253A%252F%252Fwww.ti.com%252Fproduct%252FISOW1432%252Fpart-details%252FISOW1432DFMR%253FkeyMatch%253DISOW1432DFMR%2526tisearch%253Dsearch-everything%2526usecase%253DOPN Your request for us to get into some datasheet somewhere, to get the answer to the basic question, is not reasonable. Commit must justify its existence, not some external source. > >> >>> >>> Signed-off-by: Manikanta Guntupalli >>> --- >>> Documentation/devicetree/bindings/serial/cdns,uart.yaml | 5 +++++ >>> 1 file changed, 5 insertions(+) >>> >>> diff --git a/Documentation/devicetree/bindings/serial/cdns,uart.yaml >> b/Documentation/devicetree/bindings/serial/cdns,uart.yaml >>> index a8b323d7bf94..34ed0d80b923 100644 >>> --- a/Documentation/devicetree/bindings/serial/cdns,uart.yaml >>> +++ b/Documentation/devicetree/bindings/serial/cdns,uart.yaml >>> @@ -46,6 +46,10 @@ properties: >>> power-domains: >>> maxItems: 1 >>> >>> + txrx-gpios: >>> + description: Optional GPIO to switch transmit and receive modes via >> pmod. >> >> What's pmod? > Pmod is galvanically-isolated RS-485/RS-422 transceivers with a built-in isolated > DC-DC converter, that eliminates the need for a separate isolated power supply in > space constrained isolated designs. It's not about explaining it here, but making your changes justified and obvious once implemented. Best regards, Krzysztof