From nobody Fri Sep 20 01:29:14 2024 Received: from mail.zeus03.de (zeus03.de [194.117.254.33]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0DCC813D61D for ; Sat, 6 Jul 2024 11:21:48 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=194.117.254.33 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720264911; cv=none; b=siBBEdQKS3cDL5JwP6HC5za7zPrVcSjNaKWZH5JH5X7/gd97rDcLWeUS+gltMhvqrJKMbX9O530UoNcQPtjTG8z52/JH4qtDmBFxGbxWyG2NZRvbjWPHnx0YY8tnjdrNx+mpAI1MAchFLJVBQV3h7bJCgX2RfF4fspMbU6aSP+8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1720264911; c=relaxed/simple; bh=bxJLK/DZfAC2qxewxSmw3TGcGeoQ0tAM4+qHE+O9k7E=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=iW85SP2iPy1CpHk2AJs9RU50XV7Ql0I0qFTZhpR/2+p8wG2OMh+PI/3vP4C2VE1XIp17op5Q7wJO0ORKQZzxwW2dz/XUHoTM8cA9b/NcaM10PMix111Vo1HJ+VjaqdqvPJdqUpm/kJYl+DnZQDaWxztUW/OGKn5jXsFJeVyYVwg= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=sang-engineering.com; spf=pass smtp.mailfrom=sang-engineering.com; dkim=pass (2048-bit key) header.d=sang-engineering.com header.i=@sang-engineering.com header.b=C2dETlZz; arc=none smtp.client-ip=194.117.254.33 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=sang-engineering.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=sang-engineering.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=sang-engineering.com header.i=@sang-engineering.com header.b="C2dETlZz" DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= sang-engineering.com; h=from:to:cc:subject:date:message-id :in-reply-to:references:mime-version:content-transfer-encoding; s=k1; bh=bStUOS8iU5VvMWJqkoZjIhVx9CNCjjqHFUdXvxQ/eQo=; b=C2dETl ZzAeaNzd+UUuqIzPvM8BtCuyS2oOnrC0JwLx21TKUhrVUEOfkADrlopZf6PbALNg /4dGTEnBmgDt/uumEDRijJr7G+MfmwgBYsT9z6dT7HBYg7cVQP40qqWQxXVlDFE9 yM/JPUq8O/vqN3BgYbuqeUvsX+nSTwwz6qY7cIGqL+r87RRRGh+wKu+8jLtpKHBI HBfEIGC76ZimPIzlGKqkqdK1a64eBjnc8otLTikAjg0rdfBl6KwJdQYE7hzZ2r/9 F2lP22jDo5/u9Uh8FPD3orr30hznDEcc8Wx67TUC/JWShNFHsleB7ye1vdHeMj7g 4QowA+0xVQITCKjA== Received: (qmail 3809811 invoked from network); 6 Jul 2024 13:21:33 +0200 Received: by mail.zeus03.de with ESMTPSA (TLS_AES_256_GCM_SHA384 encrypted, authenticated); 6 Jul 2024 13:21:33 +0200 X-UD-Smtp-Session: l3s3148p1@ULXbYZIcqJBQT+F6 From: Wolfram Sang To: linux-i2c@vger.kernel.org Cc: Wolfram Sang , Eddie James , Andi Shyti , openbmc@lists.ozlabs.org, linux-kernel@vger.kernel.org Subject: [PATCH v2 15/60] i2c: fsi: reword according to newest specification Date: Sat, 6 Jul 2024 13:20:15 +0200 Message-ID: <20240706112116.24543-16-wsa+renesas@sang-engineering.com> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20240706112116.24543-1-wsa+renesas@sang-engineering.com> References: <20240706112116.24543-1-wsa+renesas@sang-engineering.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Change the wording of this driver wrt. the newest I2C v7 and SMBus 3.2 specifications and replace "master/slave" with more appropriate terms. Signed-off-by: Wolfram Sang Reviewed-by: Andi Shyti Reviewed-by: Eddie James --- drivers/i2c/busses/i2c-fsi.c | 56 ++++++++++++++++++------------------ 1 file changed, 28 insertions(+), 28 deletions(-) diff --git a/drivers/i2c/busses/i2c-fsi.c b/drivers/i2c/busses/i2c-fsi.c index 10332693edf0..ae016a9431da 100644 --- a/drivers/i2c/busses/i2c-fsi.c +++ b/drivers/i2c/busses/i2c-fsi.c @@ -1,6 +1,6 @@ // SPDX-License-Identifier: GPL-2.0+ /* - * FSI-attached I2C master algorithm + * FSI-attached I2C controller algorithm * * Copyright 2018 IBM Corporation * @@ -145,7 +145,7 @@ /* choose timeout length from legacy driver; it's well tested */ #define I2C_ABORT_TIMEOUT msecs_to_jiffies(100) =20 -struct fsi_i2c_master { +struct fsi_i2c_ctrl { struct fsi_device *fsi; u8 fifo_size; struct list_head ports; @@ -155,7 +155,7 @@ struct fsi_i2c_master { struct fsi_i2c_port { struct list_head list; struct i2c_adapter adapter; - struct fsi_i2c_master *master; + struct fsi_i2c_ctrl *ctrl; u16 port; u16 xfrd; }; @@ -183,7 +183,7 @@ static int fsi_i2c_write_reg(struct fsi_device *fsi, un= signed int reg, return fsi_device_write(fsi, reg, &data_be, sizeof(data_be)); } =20 -static int fsi_i2c_dev_init(struct fsi_i2c_master *i2c) +static int fsi_i2c_dev_init(struct fsi_i2c_ctrl *i2c) { int rc; u32 mode =3D I2C_MODE_ENHANCED, extended_status, watermark; @@ -214,7 +214,7 @@ static int fsi_i2c_dev_init(struct fsi_i2c_master *i2c) static int fsi_i2c_set_port(struct fsi_i2c_port *port) { int rc; - struct fsi_device *fsi =3D port->master->fsi; + struct fsi_device *fsi =3D port->ctrl->fsi; u32 mode, dummy =3D 0; =20 rc =3D fsi_i2c_read_reg(fsi, I2C_FSI_MODE, &mode); @@ -236,7 +236,7 @@ static int fsi_i2c_set_port(struct fsi_i2c_port *port) static int fsi_i2c_start(struct fsi_i2c_port *port, struct i2c_msg *msg, bool stop) { - struct fsi_i2c_master *i2c =3D port->master; + struct fsi_i2c_ctrl *i2c =3D port->ctrl; u32 cmd =3D I2C_CMD_WITH_START | I2C_CMD_WITH_ADDR; =20 port->xfrd =3D 0; @@ -268,7 +268,7 @@ static int fsi_i2c_write_fifo(struct fsi_i2c_port *port= , struct i2c_msg *msg, { int write; int rc; - struct fsi_i2c_master *i2c =3D port->master; + struct fsi_i2c_ctrl *i2c =3D port->ctrl; int bytes_to_write =3D i2c->fifo_size - fifo_count; int bytes_remaining =3D msg->len - port->xfrd; =20 @@ -294,7 +294,7 @@ static int fsi_i2c_read_fifo(struct fsi_i2c_port *port,= struct i2c_msg *msg, { int read; int rc; - struct fsi_i2c_master *i2c =3D port->master; + struct fsi_i2c_ctrl *i2c =3D port->ctrl; int bytes_to_read; int xfr_remaining =3D msg->len - port->xfrd; u32 dummy; @@ -330,7 +330,7 @@ static int fsi_i2c_get_scl(struct i2c_adapter *adap) { u32 stat =3D 0; struct fsi_i2c_port *port =3D adap->algo_data; - struct fsi_i2c_master *i2c =3D port->master; + struct fsi_i2c_ctrl *i2c =3D port->ctrl; =20 fsi_i2c_read_reg(i2c->fsi, I2C_FSI_STAT, &stat); =20 @@ -341,7 +341,7 @@ static void fsi_i2c_set_scl(struct i2c_adapter *adap, i= nt val) { u32 dummy =3D 0; struct fsi_i2c_port *port =3D adap->algo_data; - struct fsi_i2c_master *i2c =3D port->master; + struct fsi_i2c_ctrl *i2c =3D port->ctrl; =20 if (val) fsi_i2c_write_reg(i2c->fsi, I2C_FSI_SET_SCL, &dummy); @@ -353,7 +353,7 @@ static int fsi_i2c_get_sda(struct i2c_adapter *adap) { u32 stat =3D 0; struct fsi_i2c_port *port =3D adap->algo_data; - struct fsi_i2c_master *i2c =3D port->master; + struct fsi_i2c_ctrl *i2c =3D port->ctrl; =20 fsi_i2c_read_reg(i2c->fsi, I2C_FSI_STAT, &stat); =20 @@ -364,7 +364,7 @@ static void fsi_i2c_set_sda(struct i2c_adapter *adap, i= nt val) { u32 dummy =3D 0; struct fsi_i2c_port *port =3D adap->algo_data; - struct fsi_i2c_master *i2c =3D port->master; + struct fsi_i2c_ctrl *i2c =3D port->ctrl; =20 if (val) fsi_i2c_write_reg(i2c->fsi, I2C_FSI_SET_SDA, &dummy); @@ -377,7 +377,7 @@ static void fsi_i2c_prepare_recovery(struct i2c_adapter= *adap) int rc; u32 mode; struct fsi_i2c_port *port =3D adap->algo_data; - struct fsi_i2c_master *i2c =3D port->master; + struct fsi_i2c_ctrl *i2c =3D port->ctrl; =20 rc =3D fsi_i2c_read_reg(i2c->fsi, I2C_FSI_MODE, &mode); if (rc) @@ -392,7 +392,7 @@ static void fsi_i2c_unprepare_recovery(struct i2c_adapt= er *adap) int rc; u32 mode; struct fsi_i2c_port *port =3D adap->algo_data; - struct fsi_i2c_master *i2c =3D port->master; + struct fsi_i2c_ctrl *i2c =3D port->ctrl; =20 rc =3D fsi_i2c_read_reg(i2c->fsi, I2C_FSI_MODE, &mode); if (rc) @@ -402,7 +402,7 @@ static void fsi_i2c_unprepare_recovery(struct i2c_adapt= er *adap) fsi_i2c_write_reg(i2c->fsi, I2C_FSI_MODE, &mode); } =20 -static int fsi_i2c_reset_bus(struct fsi_i2c_master *i2c, +static int fsi_i2c_reset_bus(struct fsi_i2c_ctrl *i2c, struct fsi_i2c_port *port) { int rc; @@ -435,7 +435,7 @@ static int fsi_i2c_reset_bus(struct fsi_i2c_master *i2c, return fsi_i2c_dev_init(i2c); } =20 -static int fsi_i2c_reset_engine(struct fsi_i2c_master *i2c, u16 port) +static int fsi_i2c_reset_engine(struct fsi_i2c_ctrl *i2c, u16 port) { int rc; u32 mode, dummy =3D 0; @@ -478,7 +478,7 @@ static int fsi_i2c_abort(struct fsi_i2c_port *port, u32= status) unsigned long start; u32 cmd =3D I2C_CMD_WITH_STOP; u32 stat; - struct fsi_i2c_master *i2c =3D port->master; + struct fsi_i2c_ctrl *i2c =3D port->ctrl; struct fsi_device *fsi =3D i2c->fsi; =20 rc =3D fsi_i2c_reset_engine(i2c, port->port); @@ -505,7 +505,7 @@ static int fsi_i2c_abort(struct fsi_i2c_port *port, u32= status) if (rc) return rc; =20 - /* wait until we see command complete in the master */ + /* wait until we see command complete in the controller */ start =3D jiffies; =20 do { @@ -579,7 +579,7 @@ static int fsi_i2c_wait(struct fsi_i2c_port *port, stru= ct i2c_msg *msg, unsigned long start =3D jiffies; =20 do { - rc =3D fsi_i2c_read_reg(port->master->fsi, I2C_FSI_STAT, + rc =3D fsi_i2c_read_reg(port->ctrl->fsi, I2C_FSI_STAT, &status); if (rc) return rc; @@ -609,10 +609,10 @@ static int fsi_i2c_xfer(struct i2c_adapter *adap, str= uct i2c_msg *msgs, int i, rc; unsigned long start_time; struct fsi_i2c_port *port =3D adap->algo_data; - struct fsi_i2c_master *master =3D port->master; + struct fsi_i2c_ctrl *ctrl =3D port->ctrl; struct i2c_msg *msg; =20 - mutex_lock(&master->lock); + mutex_lock(&ctrl->lock); =20 rc =3D fsi_i2c_set_port(port); if (rc) @@ -633,7 +633,7 @@ static int fsi_i2c_xfer(struct i2c_adapter *adap, struc= t i2c_msg *msgs, } =20 unlock: - mutex_unlock(&master->lock); + mutex_unlock(&ctrl->lock); return rc ? : num; } =20 @@ -654,7 +654,7 @@ static struct i2c_bus_recovery_info fsi_i2c_bus_recover= y_info =3D { }; =20 static const struct i2c_algorithm fsi_i2c_algorithm =3D { - .master_xfer =3D fsi_i2c_xfer, + .xfer =3D fsi_i2c_xfer, .functionality =3D fsi_i2c_functionality, }; =20 @@ -676,7 +676,7 @@ static struct device_node *fsi_i2c_find_port_of_node(st= ruct device_node *fsi, =20 static int fsi_i2c_probe(struct device *dev) { - struct fsi_i2c_master *i2c; + struct fsi_i2c_ctrl *i2c; struct fsi_i2c_port *port; struct device_node *np; u32 port_no, ports, stat; @@ -699,7 +699,7 @@ static int fsi_i2c_probe(struct device *dev) return rc; =20 ports =3D FIELD_GET(I2C_STAT_MAX_PORT, stat) + 1; - dev_dbg(dev, "I2C master has %d ports\n", ports); + dev_dbg(dev, "I2C controller has %d ports\n", ports); =20 for (port_no =3D 0; port_no < ports; port_no++) { np =3D fsi_i2c_find_port_of_node(dev->of_node, port_no); @@ -712,7 +712,7 @@ static int fsi_i2c_probe(struct device *dev) break; } =20 - port->master =3D i2c; + port->ctrl =3D i2c; port->port =3D port_no; =20 port->adapter.owner =3D THIS_MODULE; @@ -742,7 +742,7 @@ static int fsi_i2c_probe(struct device *dev) =20 static int fsi_i2c_remove(struct device *dev) { - struct fsi_i2c_master *i2c =3D dev_get_drvdata(dev); + struct fsi_i2c_ctrl *i2c =3D dev_get_drvdata(dev); struct fsi_i2c_port *port, *tmp; =20 list_for_each_entry_safe(port, tmp, &i2c->ports, list) { @@ -772,5 +772,5 @@ static struct fsi_driver fsi_i2c_driver =3D { module_fsi_driver(fsi_i2c_driver); =20 MODULE_AUTHOR("Eddie James "); -MODULE_DESCRIPTION("FSI attached I2C master"); +MODULE_DESCRIPTION("FSI attached I2C controller"); MODULE_LICENSE("GPL"); --=20 2.43.0