Date:   Thu, 23 Dec 2021 16:58:24 +0530
From:   Vinod Koul <vkoul@kernel.org>
To:     Aswath Govindraju <a-govindraju@ti.com>
Cc:     Wolfgang Grandegger <wg@grandegger.com>,
        Marc Kleine-Budde <mkl@pengutronix.de>,
        Kishon Vijay Abraham I <kishon@ti.com>,
        Rob Herring <robh+dt@kernel.org>, linux-can@vger.kernel.org,
        linux-phy@lists.infradead.org, devicetree@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH v3 0/2] CAN: Add support for setting mux
Message-ID: <YcRdWJtREHFWXBKM@matsya>
References: <20211216041012.16892-1-a-govindraju@ti.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20211216041012.16892-1-a-govindraju@ti.com>
Precedence: bulk

On 16-12-21, 09:40, Aswath Govindraju wrote:
> The following series of patches add support for setting
> muxes to route signals from CAN controller to transceiver
> by reading the property mux-states from the device tree
> node

These look okay to me
> 
> The following series of patches are dependent on,
> - https://lkml.org/lkml/2021/12/2/423

Pls post when these are merged..

> 
> Changes since v2:
> - Fixed the bindings removing the description about
>   the arguments in mux-states property
> 
> Changes since v1:
> - Fixed the description for mux-states property in bindings
>   file
> - appended declaration of variable ret in the end
> 
> Aswath Govindraju (2):
>   dt-bindings: phy: ti,tcan104x-can: Document mux-states property
>   phy: phy-can-transceiver: Add support for setting mux
> 
>  .../bindings/phy/ti,tcan104x-can.yaml         |  7 ++++++
>  drivers/phy/Kconfig                           |  1 +
>  drivers/phy/phy-can-transceiver.c             | 24 ++++++++++++++++++-
>  3 files changed, 31 insertions(+), 1 deletion(-)
> 
> -- 
> 2.17.1

-- 
~Vinod