From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ryder Lee Date: Thu, 25 Oct 2018 14:13:37 +0800 Subject: [U-Boot] [PATCH v2 11/18] pinctrl: MediaTek: add pinctrl driver for MT7623 SoC In-Reply-To: References: Message-ID: <1540448017.32603.20.camel@mtkswgap22> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de On Wed, 2018-10-24 at 21:29 -0600, Simon Glass wrote: > Hi Ryder, > > On 12 October 2018 at 01:01, Ryder Lee wrote: > > This patch adds pinctrl support for MT7623 SoC. And most of the > > structures are used to hold the hardware configuration for each > > pin. > > > > Signed-off-by: Ryder Lee > > Tested-by: Matthias Brugger > > --- > > drivers/pinctrl/mediatek/Kconfig | 4 + > > drivers/pinctrl/mediatek/Makefile | 1 + > > drivers/pinctrl/mediatek/pinctrl-mt7623.c | 1284 +++++++++++++++++++++++++ > > drivers/pinctrl/mediatek/pinctrl-mtk-common.h | 1 + > > 4 files changed, 1290 insertions(+) > > create mode 100644 drivers/pinctrl/mediatek/pinctrl-mt7623.c > > [..] > > > +void mtk_rmw(struct udevice *dev, u32 reg, u32 mask, u32 set); > > What is this for? It has no comment and is an exported function. We > should not export things from drivers unless there is a good reason. This is a common 'read-modify-write' helper. I think we could export it in this shared header? > Regards, > Simon