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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 560E2C77B6F for ; Tue, 11 Apr 2023 17:58:09 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=paYwQgguYg/p6zZwUl5DL6R131wKVgjWiVdOJVt4p5g=; b=Cyf65MLMgu9ZmA MEqZphvZ8K6JZtY7165PVCFdZRI91cnLAtM2XF+RFlK8nVrMTppq8hgae5EW44Vy43mHJXDJB5Dhl 72bzdyfbzfuRRKRYefBL9gc+JSoMSxYvbRCMpqf2GEgFVcnrxlKR/KiX5KvGEL1nvmgitlMRkd3+j C3rxII3pc6jaX11JzVk8cLz+yxNTU8jrgqA7BjmEObUWHXN2zolWmO7WTMfxSAteMe4FgYQnmg7PB FBz64hx0w8SM891QeHNTbTEgGwNqQ7FJ2Q1qISP6OksUYzpJZ4JQRGBNoBdVL11WSqOK/wTG2is4/ XEXQdy3HKHUqJ0H1q2Hw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1pmIF2-000lXt-2N; Tue, 11 Apr 2023 17:57:16 +0000 Received: from mail-pl1-x62f.google.com ([2607:f8b0:4864:20::62f]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1pmIEy-000lWo-2A for linux-arm-kernel@lists.infradead.org; Tue, 11 Apr 2023 17:57:14 +0000 Received: by mail-pl1-x62f.google.com with SMTP id n14so33741536plc.8 for ; Tue, 11 Apr 2023 10:57:10 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1681235830; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=ZXbDM1ImhdsbAlX5z/W3cgh/RDeLJTPqUHcpprXXqrI=; b=Pe50bdhimru+N/2HOljtZmyOoxclKuA8dwD8vHMj/wTGm2q+w6oqit4YahOasjbSha AMbONG0/q2aUcdmelgB9qgZ1iwUAmwsiiXC1x7I3jdHtRlYE6cz+vvfq2yB/6ZUVp8Gl MEysICPLawzNdU6WcAZ2PpjMCi7/uwmAdt8PZHuW6NgeGBpRfeqBnQ/swpi+zcnx0iC+ 24wHa5d31Kn6QfGldVxQrQQ20PZsRNVJj4yfBGLeNwmK3NOaSxs1NnlD0lYVKhJC1s+h hNYmqVJU0gg01GaWFc1JaNe+gOleUte9vINUNhzY6WhovauG7H+/+BoF3wWE6NVBZxM9 AhnQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; t=1681235830; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=ZXbDM1ImhdsbAlX5z/W3cgh/RDeLJTPqUHcpprXXqrI=; b=7lQyKAsMEv98MET/FPD+IHooEhGzq1Qo6jg4+pVcrx8kDrocYjRxTVWo8tpOfVfNIC zJWn/tZcdqkE3sRzJS66jbvFVllXPoE+TBuniVQF6Te35UQLjKN5lbaxj9OrwTINTcDU /v5Mr6Uj7j+s5ZO8D2CVRF7OIauKuoAHRwzeIHsXbZ8NrjeikPykXmOHETvgRQGLYlpr ATG6K3c9fgfsPhyl1IJT7wYF0droEb7xUbrF0aum5fg3QveSzTCBQKeNjdrkG3n5boPA qR7wuQwDOWo/duZSnmKLydCPLG4OpF79mg34Mk1HXd1shP5YXmK/tPIrxaMq1AgCBwgg AQLg== X-Gm-Message-State: AAQBX9cPF7Tbds64o3ni92lXhyl8qkjZR7LtwShhmMYxJFRW59IKfY8R URv4Xnt73H8IkMpz2oteG+tbyQ== X-Google-Smtp-Source: AKy350ZN9Il+nr1BgDR94DewMoio2CPd0j/HHP3oUqhd0/9J/DasM1Br0S+P8KV0fQblKMu/Th/CVw== X-Received: by 2002:a17:902:788f:b0:1a6:6afc:2d67 with SMTP id q15-20020a170902788f00b001a66afc2d67mr37276pll.61.1681235830196; Tue, 11 Apr 2023 10:57:10 -0700 (PDT) Received: from p14s ([2604:3d09:148c:c800:f795:eecb:467b:d183]) by smtp.gmail.com with ESMTPSA id d3-20020a170902854300b001a1a9a639c2sm5501289plo.134.2023.04.11.10.57.08 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 11 Apr 2023 10:57:09 -0700 (PDT) Date: Tue, 11 Apr 2023 11:57:07 -0600 From: Mathieu Poirier To: MD Danish Anwar Subject: Re: [PATCH v7 4/4] soc: ti: pruss: Add helper functions to set GPI mode, MII_RT_event and XFR Message-ID: <20230411175707.GE38361@p14s> References: <20230404115336.599430-1-danishanwar@ti.com> <20230404115336.599430-5-danishanwar@ti.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230404115336.599430-5-danishanwar@ti.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230411_105712_710327_580C05CA X-CRM114-Status: GOOD ( 27.84 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Nishanth Menon , srk@ti.com, linux-omap@vger.kernel.org, Vignesh Raghavendra , Tero Kristo , devicetree@vger.kernel.org, netdev@vger.kernel.org, Bjorn Andersson , linux-remoteproc@vger.kernel.org, linux-kernel@vger.kernel.org, "Andrew F. Davis" , Roger Quadros , Santosh Shilimkar , linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Apr 04, 2023 at 05:23:36PM +0530, MD Danish Anwar wrote: > From: Suman Anna > > The PRUSS CFG module is represented as a syscon node and is currently > managed by the PRUSS platform driver. Add easy accessor functions to set > GPI mode, MII_RT event enable/disable and XFR (XIN XOUT) enable/disable > to enable the PRUSS Ethernet usecase. These functions reuse the generic > pruss_cfg_update() API function. > > Signed-off-by: Suman Anna > Co-developed-by: Grzegorz Jaszczyk > Signed-off-by: Grzegorz Jaszczyk > Signed-off-by: Puranjay Mohan > Reviewed-by: Roger Quadros > Reviewed-by: Tony Lindgren > Signed-off-by: MD Danish Anwar > --- > drivers/remoteproc/pru_rproc.c | 15 ------- > drivers/soc/ti/pruss.c | 74 ++++++++++++++++++++++++++++++++ > include/linux/remoteproc/pruss.h | 51 ++++++++++++++++++++++ > 3 files changed, 125 insertions(+), 15 deletions(-) > > diff --git a/drivers/remoteproc/pru_rproc.c b/drivers/remoteproc/pru_rproc.c > index 4ddd5854d56e..a88861737dec 100644 > --- a/drivers/remoteproc/pru_rproc.c > +++ b/drivers/remoteproc/pru_rproc.c > @@ -81,21 +81,6 @@ enum pru_iomem { > PRU_IOMEM_MAX, > }; > > -/** > - * enum pru_type - PRU core type identifier > - * > - * @PRU_TYPE_PRU: Programmable Real-time Unit > - * @PRU_TYPE_RTU: Auxiliary Programmable Real-Time Unit > - * @PRU_TYPE_TX_PRU: Transmit Programmable Real-Time Unit > - * @PRU_TYPE_MAX: just keep this one at the end > - */ > -enum pru_type { > - PRU_TYPE_PRU = 0, > - PRU_TYPE_RTU, > - PRU_TYPE_TX_PRU, > - PRU_TYPE_MAX, > -}; > - > /** > * struct pru_private_data - device data for a PRU core > * @type: type of the PRU core (PRU, RTU, Tx_PRU) > diff --git a/drivers/soc/ti/pruss.c b/drivers/soc/ti/pruss.c > index 0e37fe142615..64a1880ba4ee 100644 > --- a/drivers/soc/ti/pruss.c > +++ b/drivers/soc/ti/pruss.c > @@ -213,6 +213,80 @@ int pruss_cfg_set_gpmux(struct pruss *pruss, enum pruss_pru_id pru_id, u8 mux) > } > EXPORT_SYMBOL_GPL(pruss_cfg_set_gpmux); > > +/** > + * pruss_cfg_gpimode() - set the GPI mode of the PRU > + * @pruss: the pruss instance handle > + * @pru_id: id of the PRU core within the PRUSS > + * @mode: GPI mode to set > + * > + * Sets the GPI mode for a given PRU by programming the > + * corresponding PRUSS_CFG_GPCFGx register > + * > + * Return: 0 on success, or an error code otherwise > + */ > +int pruss_cfg_gpimode(struct pruss *pruss, enum pruss_pru_id pru_id, > + enum pruss_gpi_mode mode) > +{ > + if (pru_id < 0 || pru_id >= PRUSS_NUM_PRUS) > + return -EINVAL; > + > + if (mode < 0 || mode > PRUSS_GPI_MODE_MAX) > + return -EINVAL; > + > + return pruss_cfg_update(pruss, PRUSS_CFG_GPCFG(pru_id), > + PRUSS_GPCFG_PRU_GPI_MODE_MASK, > + mode << PRUSS_GPCFG_PRU_GPI_MODE_SHIFT); > +} > +EXPORT_SYMBOL_GPL(pruss_cfg_gpimode); > + > +/** > + * pruss_cfg_miirt_enable() - Enable/disable MII RT Events > + * @pruss: the pruss instance > + * @enable: enable/disable > + * > + * Enable/disable the MII RT Events for the PRUSS. > + * > + * Return: 0 on success, or an error code otherwise > + */ > +int pruss_cfg_miirt_enable(struct pruss *pruss, bool enable) > +{ > + u32 set = enable ? PRUSS_MII_RT_EVENT_EN : 0; > + > + return pruss_cfg_update(pruss, PRUSS_CFG_MII_RT, > + PRUSS_MII_RT_EVENT_EN, set); > +} > +EXPORT_SYMBOL_GPL(pruss_cfg_miirt_enable); > + > +/** > + * pruss_cfg_xfr_enable() - Enable/disable XIN XOUT shift functionality > + * @pruss: the pruss instance > + * @pru_type: PRU core type identifier > + * @enable: enable/disable > + * > + * Return: 0 on success, or an error code otherwise > + */ > +int pruss_cfg_xfr_enable(struct pruss *pruss, enum pru_type pru_type, > + bool enable) > +{ > + u32 mask, set; > + > + switch (pru_type) { > + case PRU_TYPE_PRU: > + mask = PRUSS_SPP_XFER_SHIFT_EN; > + break; > + case PRU_TYPE_RTU: > + mask = PRUSS_SPP_RTU_XFR_SHIFT_EN; > + break; > + default: > + return -EINVAL; > + } > + > + set = enable ? mask : 0; > + > + return pruss_cfg_update(pruss, PRUSS_CFG_SPP, mask, set); > +} > +EXPORT_SYMBOL_GPL(pruss_cfg_xfr_enable); > + > static void pruss_of_free_clk_provider(void *data) > { > struct device_node *clk_mux_np = data; > diff --git a/include/linux/remoteproc/pruss.h b/include/linux/remoteproc/pruss.h > index 5641153459a7..b68ab8735247 100644 > --- a/include/linux/remoteproc/pruss.h > +++ b/include/linux/remoteproc/pruss.h > @@ -34,6 +34,33 @@ enum pruss_gp_mux_sel { > PRUSS_GP_MUX_SEL_MAX, > }; > > +/* > + * enum pruss_gpi_mode - PRUSS GPI configuration modes, used > + * to program the PRUSS_GPCFG0/1 registers > + */ > +enum pruss_gpi_mode { > + PRUSS_GPI_MODE_DIRECT = 0, > + PRUSS_GPI_MODE_PARALLEL, > + PRUSS_GPI_MODE_28BIT_SHIFT, > + PRUSS_GPI_MODE_MII, > + PRUSS_GPI_MODE_MAX, > +}; > + > +/** > + * enum pru_type - PRU core type identifier > + * > + * @PRU_TYPE_PRU: Programmable Real-time Unit > + * @PRU_TYPE_RTU: Auxiliary Programmable Real-Time Unit > + * @PRU_TYPE_TX_PRU: Transmit Programmable Real-Time Unit > + * @PRU_TYPE_MAX: just keep this one at the end > + */ > +enum pru_type { > + PRU_TYPE_PRU = 0, > + PRU_TYPE_RTU, > + PRU_TYPE_TX_PRU, > + PRU_TYPE_MAX, > +}; > + These go in pruss_driver.h > /** > * enum pruss_pru_id - PRU core identifiers > * @PRUSS_PRU0: PRU Core 0. > @@ -98,6 +125,11 @@ int pruss_release_mem_region(struct pruss *pruss, > struct pruss_mem_region *region); > int pruss_cfg_get_gpmux(struct pruss *pruss, enum pruss_pru_id pru_id, u8 *mux); > int pruss_cfg_set_gpmux(struct pruss *pruss, enum pruss_pru_id pru_id, u8 mux); > +int pruss_cfg_gpimode(struct pruss *pruss, enum pruss_pru_id pru_id, > + enum pruss_gpi_mode mode); > +int pruss_cfg_miirt_enable(struct pruss *pruss, bool enable); > +int pruss_cfg_xfr_enable(struct pruss *pruss, enum pru_type pru_type, > + bool enable); > > #else > > @@ -133,6 +165,25 @@ static inline int pruss_cfg_set_gpmux(struct pruss *pruss, > return ERR_PTR(-EOPNOTSUPP); > } > > +static inline int pruss_cfg_gpimode(struct pruss *pruss, > + enum pruss_pru_id pru_id, > + enum pruss_gpi_mode mode) > +{ > + return ERR_PTR(-EOPNOTSUPP); > +} > + > +static inline int pruss_cfg_miirt_enable(struct pruss *pruss, bool enable) > +{ > + return ERR_PTR(-EOPNOTSUPP); > +} > + > +static inline int pruss_cfg_xfr_enable(struct pruss *pruss, > + enum pru_type pru_type, > + bool enable); > +{ > + return ERR_PTR(-EOPNOTSUPP); > +} > + So do these. Thanks, Mathieu > #endif /* CONFIG_TI_PRUSS */ > > #if IS_ENABLED(CONFIG_PRU_REMOTEPROC) > -- > 2.25.1 > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel