From patchwork Thu Mar 23 10:31:50 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: Herve Codina X-Patchwork-Id: 13185469 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 69CBCC6FD1C for ; Thu, 23 Mar 2023 10:36:32 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231512AbjCWKga (ORCPT ); Thu, 23 Mar 2023 06:36:30 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:53230 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229917AbjCWKgD (ORCPT ); Thu, 23 Mar 2023 06:36:03 -0400 Received: from relay7-d.mail.gandi.net (relay7-d.mail.gandi.net [IPv6:2001:4b98:dc4:8::227]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 41A9F2CC53; Thu, 23 Mar 2023 03:32:07 -0700 (PDT) Received: (Authenticated sender: herve.codina@bootlin.com) by mail.gandi.net (Postfix) with ESMTPA id 3705E20011; Thu, 23 Mar 2023 10:32:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=bootlin.com; s=gm1; t=1679567526; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=En1dwiDW/nnta459jp55SHmclo/QOWmQxsSNcw/FvxU=; b=dhqqOVVb41Em6WHFrbOqQ4Eqg9sBnbqbDXsnbyYjQehXUbpP9A769e6ODNsqGKwO8tAMPQ mY26w6DciCiCiN0xJsRdirMMjc/zTluHn2SiXP/6RGqGWGl1IFEmh+uZ2KqLcSGG31Stm1 Gbc5XUaTBHF3w1vY3AqZUDMiaAWXb6NkjuMteblY/5XIBGxtGXr0WDGWBki50P7LPtFB/n xfGAfdU46HACaBDFRuO0PUCoJwX7SVwcGE7kI0y7I2QuP+zPXUDjg+pUP+kMzOIB8gOqjg Hd/tm4SBMXIT4p3QKydSVaX4zEZiDjfwXfCu2WWNsj2EoMzzUiytDD6HvWUevA== From: Herve Codina To: Herve Codina , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Vinod Koul , Kishon Vijay Abraham I Cc: linux-kernel@vger.kernel.org, netdev@vger.kernel.org, linux-phy@lists.infradead.org, Christophe Leroy , Thomas Petazzoni Subject: [RFC PATCH 0/4] Add support for QMC HDLC and PHY Date: Thu, 23 Mar 2023 11:31:50 +0100 Message-Id: <20230323103154.264546-1-herve.codina@bootlin.com> X-Mailer: git-send-email 2.39.2 MIME-Version: 1.0 Precedence: bulk List-ID: X-Mailing-List: netdev@vger.kernel.org X-Patchwork-State: RFC Hi, I have a system where I need to handle an HDLC interface. The HDLC data are transferred using a TDM bus on which a PEF2256 is present. The PEF2256 transfers data from/to the TDM bus to/from E1 line. This PEF2256 is also connected to a PowerQUICC SoC for the control path and the TDM is connected to the SoC (QMC component) for the data path. From the HDLC driver, I need to handle data using the QMC and carrier detection using the PEF2256 (E1 line carrier). The HDLC driver consider the PEF2256 as a generic PHY. So, the design is the following: +----------+ +-------------+ +---------+ | HDLC drv | <-data-> | QMC channel | <-- TDM --> | PEF2256 | +----------+ +-------------+ | | <--> E1 ^ +---------+ +---------+ | | +-> | Gen PHY | <-> | PEF2256 | <- local bus -> | | +---------+ | PHY drv | +---------+ +---------+ In order to implement this, I had to: 1 - Extend the generic PHY API to support get_status() and notification on status change. 2 - Introduce a new kind of generic PHY named "basic phy". This PHY familly can provide a link status in the get_status() data. 3 - Support the PEF2256 PHY as a "basic phy" The purpose of this RFC series is to discuss this design. The QMC driver code is available on linux-next. In this series: - patch 1: driver HDLC using the QMC channel - patch 2: Extend the generic PHY API - patch 3: Use the "basic phy" in the HDLC driver - patch 4: Implement the PEF2256 PHY driver I did 2 patches for the HDLC driver in order to point the new PHY family usage in the HDLC driver. In the end, these two patches will be squashed and the bindings will be added. Hope to have some feedback on this proposal. Best regards, Hervé Herve Codina (4): net: wan: Add support for QMC HDLC phy: Extend API to support 'status' get and notification net: wan: fsl_qmc_hdlc: Add PHY support phy: lantiq: Add PEF2256 PHY support drivers/net/wan/Kconfig | 12 + drivers/net/wan/Makefile | 1 + drivers/net/wan/fsl_qmc_hdlc.c | 558 ++++++++++++++++++++++++ drivers/phy/lantiq/Kconfig | 15 + drivers/phy/lantiq/Makefile | 1 + drivers/phy/lantiq/phy-lantiq-pef2256.c | 131 ++++++ drivers/phy/phy-core.c | 88 ++++ include/linux/phy/phy-basic.h | 27 ++ include/linux/phy/phy.h | 89 +++- 9 files changed, 921 insertions(+), 1 deletion(-) create mode 100644 drivers/net/wan/fsl_qmc_hdlc.c create mode 100644 drivers/phy/lantiq/phy-lantiq-pef2256.c create mode 100644 include/linux/phy/phy-basic.h