[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <2ee4d89e-ad8e-49ff-9121-feab57e2acf1@kernel.org>
Date: Tue, 24 Sep 2024 11:08:04 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Marco Felsch <m.felsch@...gutronix.de>
Cc: Michal Vokáč <michal.vokac@...ft.com>,
Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Shawn Guo <shawnguo@...nel.org>, Petr Benes <petr.benes@...ft.com>,
devicetree@...r.kernel.org, linux-usb@...r.kernel.org,
Michael Walle <mwalle@...nel.org>, imx@...ts.linux.dev,
Alexander Stein <alexander.stein@...tq-group.com>,
Sascha Hauer <s.hauer@...gutronix.de>,
Mathieu Othacehe <m.othacehe@...il.com>,
Hugo Villeneuve <hvilleneuve@...onoff.com>, linux-kernel@...r.kernel.org,
Hiago De Franco <hiago.franco@...adex.com>,
Herburger <gregor.herburger@...tq-group.com>,
Pengutronix Kernel Team <kernel@...gutronix.de>,
Joao Paulo Goncalves <joao.goncalves@...adex.com>,
Fabio Estevam <festevam@...il.com>, linux-arm-kernel@...ts.infradead.org
Subject: Re: [PATCH v3 3/4] dt-bindings: usb: Add Diodes Incorporated
PI5USB30213A Type-C Controller
On 24/09/2024 10:21, Krzysztof Kozlowski wrote:
> On Tue, Sep 24, 2024 at 09:24:36AM +0200, Marco Felsch wrote:
>> On 24-09-23, Michal Vokáč wrote:
>>> From: Petr Benes <petr.benes@...ft.com>
>>>
>>> Diodes Incorporated PI5USB30213A Type-C Controller supports host,
>>> device, and dual-role mode based on voltage levels detected on CC
>>> pin. Supports dual differential channel, 2:1 USB 3.0 Mux/Demux,
>>> USB Type-C specification 1.1.
>>>
>>> Reviewed-by: Krzysztof Kozlowski <krzk@...nel.org>
>>> Signed-off-by: Petr Benes <petr.benes@...ft.com>
>>> Signed-off-by: Michal Vokáč <michal.vokac@...ft.com>
>>> ---
>>> v3:
>>> - Collected R-b tag from Krzysztof.
>>> v2:
>>> - Moved maintainers before description: block.
>>> - Used full paths for references.
>>> - Removed unneeded items form connector property.
>>> - Fixed example.
>>>
>>> .../bindings/usb/diodes,pi5usb30213a.yaml | 88 +++++++++++++++++++
>>
>> I suppose the driver is part of an other patchset?
>
> Hm, indeed, where is the driver?
>
> This patch should not be here.
I found remark in the cover letter. It's fine.
Best regards,
Krzysztof
Powered by blists - more mailing lists