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 X-Spam-Level: X-Spam-Status: No, score=-2.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id F05DEC43215 for ; Mon, 25 Nov 2019 07:53:33 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id D00AA2071A for ; Mon, 25 Nov 2019 07:53:33 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725875AbfKYHxd (ORCPT ); Mon, 25 Nov 2019 02:53:33 -0500 Received: from mail-sh.amlogic.com ([58.32.228.43]:22090 "EHLO mail-sh.amlogic.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725535AbfKYHxc (ORCPT ); Mon, 25 Nov 2019 02:53:32 -0500 Received: from [10.18.38.198] (10.18.38.198) by mail-sh.amlogic.com (10.18.11.5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.1591.10; Mon, 25 Nov 2019 15:53:58 +0800 Subject: Re: [PATCH 0/6] arm64: meson: Add support for USB on Amlogic A1 To: Martin Blumenstingl CC: Jerome Brunet , Neil Armstrong , Rob Herring , Greg Kroah-Hartman , Felipe Balbi , Kevin Hilman , , , , , Carlo Caione , Michael Turquette , Stephen Boyd , Liang Yang , Jianxin Pan , Qiufang Dai , Jian Hu , Victor Wan , Yue Wang , Xingyu Chen References: <1574405757-76184-1-git-send-email-hanjie.lin@amlogic.com> From: Hanjie Lin Message-ID: <5c0029ec-7377-4c1a-0062-3b59a87f8dea@amlogic.com> Date: Mon, 25 Nov 2019 15:53:57 +0800 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.9.1 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset="utf-8" Content-Language: en-US Content-Transfer-Encoding: 7bit X-Originating-IP: [10.18.38.198] X-ClientProxiedBy: mail-sh.amlogic.com (10.18.11.5) To mail-sh.amlogic.com (10.18.11.5) Sender: linux-usb-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-usb@vger.kernel.org On 2019/11/22 15:52, Martin Blumenstingl wrote: > Hello Hanjie, > > On Fri, Nov 22, 2019 at 7:55 AM Hanjie Lin wrote: > [...] >> dt-bindings: phy: Add Amlogic G12A USB2 PHY Bindings >> dt-bindings: usb: dwc3: Add the Amlogic A1 Family DWC3 Glue Bindings >> phy: amlogic: Add Amlogic A1 USB2 PHY Driver > drivers/phy/amlogic/phy-meson-g12a-usb2.c seems very similar to the A1 > USB2 PHY you are introducing here. > >> usb: dwc3: Add Amlogic A1 DWC3 glue > drivers/usb/dwc3/dwc3-meson-g12a.c is also very similar to the dwc3 glue. > > I have two questions: > - how is the PHY and the dwc3 glue different from G12A (or SM1)? > - why do we need a separate set of new drivers (instead of updating > the existing drivers)? > > We try to use one driver for the same IP block, even if there are > several revisions with small differences (for example the SAR ADC > driver supports all SoC generations from Meson8 to G12A/G12B/SM1, > because 80-90% of the code is shared across all revisions). > > > Martin > > . > Hi Martin, thanks for the comment. 1, G12A have usb2-phy0/usb2-phy1/usb3-phy0 three phys and an interrupt to support host/peripheral/otg modes. A1 has one usb2-phy0 phy and only support host mode. 2, G12A glue/phy drivers are for G12A SoCs, there are some diffrences to A1. G12A glue driver have dr_mode and interrupts two attributes to support otg mode while A1 hasn't this requirement. G12A glue driver has a hard coding vbus regulator code to support otg mode while A1 hasn't this requirement. G12A glue driver has a hard coding support phys while A1 only supports host mode. enum { USB2_HOST_PHY = 0, USB2_OTG_PHY, USB3_HOST_PHY, PHY_COUNT, }; G12A glue driver only supports one clock while A1 needs four clocks. G12A and A1 phy drivers have different register configurations since hardware differences. 3, We have estimated these differences and we thought it's more clear and readable to have a dedicated glue/phy driver for A1 SoCs, so also dedicated dt-bindings.