| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <nbi3fngfcipt35gzguk2mh4zzh3vy5a5gsk7dti5smm2iimytl@drm7p2iqsinp> Date: Thu, 11 Jul 2024 15:16:40 +0200 From: Andi Shyti <andi.shyti@...nel.org> To: Wolfram Sang <wsa+renesas@...g-engineering.com> Cc: linux-i2c@...r.kernel.org, Ajay Gupta <ajayg@...dia.com>, Alain Volmat <alain.volmat@...s.st.com>, Alexandre Torgue <alexandre.torgue@...s.st.com>, Andreas Färber <afaerber@...e.de>, Andrew Lunn <andrew@...n.ch>, AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>, asahi@...ts.linux.dev, Baolin Wang <baolin.wang@...ux.alibaba.com>, Bartosz Golaszewski <brgl@...ev.pl>, Baruch Siach <baruch@...s.co.il>, Bence Csókás <bence98@....bme.hu>, Benson Leung <bleung@...omium.org>, Binbin Zhou <zhoubinbin@...ngson.cn>, Chen-Yu Tsai <wens@...e.org>, Chris Brandt <chris.brandt@...esas.com>, Chris Packham <chris.packham@...iedtelesis.co.nz>, chrome-platform@...ts.linux.dev, Chunyan Zhang <zhang.lyra@...il.com>, Conghui Chen <conghui.chen@...el.com>, Eddie James <eajames@...ux.ibm.com>, Fabrizio Castro <fabrizio.castro.jz@...esas.com>, Florian Fainelli <florian.fainelli@...adcom.com>, Gregory CLEMENT <gregory.clement@...tlin.com>, Guenter Roeck <linux@...ck-us.net>, Hans de Goede <hdegoede@...hat.com>, Hans Hu <hanshu@...oxin.com>, Hector Martin <marcan@...can.st>, Heiko Stuebner <heiko@...ech.de>, Jean Delvare <jdelvare@...e.com>, Jernej Skrabec <jernej.skrabec@...il.com>, Jochen Friedrich <jochen@...am.de>, Jonathan Hunter <jonathanh@...dia.com>, Kamal Dasu <kamal.dasu@...adcom.com>, Kunihiko Hayashi <hayashi.kunihiko@...ionext.com>, linux-actions@...ts.infradead.org, linux-kernel@...r.kernel.org, linux-mediatek@...ts.infradead.org, linux-mips@...r.kernel.org, linuxppc-dev@...ts.ozlabs.org, linux-renesas-soc@...r.kernel.org, linux-rockchip@...ts.infradead.org, linux-rpi-kernel@...ts.infradead.org, linux-stm32@...md-mailman.stormreply.com, linux-sunxi@...ts.linux.dev, linux-tegra@...r.kernel.org, Manivannan Sadhasivam <manivannan.sadhasivam@...aro.org>, Masami Hiramatsu <mhiramat@...nel.org>, Matthias Brugger <matthias.bgg@...il.com>, Maxime Coquelin <mcoquelin.stm32@...il.com>, Michael Ellerman <mpe@...erman.id.au>, Michael Shych <michaelsh@...dia.com>, openbmc@...ts.ozlabs.org, Orson Zhai <orsonzhai@...il.com>, Patrice Chotard <patrice.chotard@...s.st.com>, Paul Cercueil <paul@...pouillou.net>, Peter Korsgaard <peter@...sgaard.com>, Pierre-Yves MORDRET <pierre-yves.mordret@...s.st.com>, Ray Jui <rjui@...adcom.com>, Robert Richter <rric@...nel.org>, Samuel Holland <samuel@...lland.org>, Scott Branden <sbranden@...adcom.com>, Stefan Roese <sr@...x.de>, Sven Peter <sven@...npeter.dev>, Thierry Reding <thierry.reding@...il.com>, Thor Thayer <thor.thayer@...ux.intel.com>, Till Harbaum <till@...baum.org>, Vadim Pasternak <vadimp@...dia.com>, Viresh Kumar <viresh.kumar@...aro.org>, virtualization@...ts.linux.dev, Vladimir Zapolskiy <vz@...ia.com>, Yicong Yang <yangyicong@...ilicon.com> Subject: Re: [PATCH v2 00/60] i2c: reword first drivers according to newest specification Hi Wolfram, pushed in i2c/i2c-host. Thanks for this big work, at the end it turned out quite nice and I'm happy of the outcome! Thanks Andi On Sat, Jul 06, 2024 at 01:20:00PM GMT, Wolfram Sang wrote: > Start changing the wording of the I2C main header wrt. the newest I2C > v7 and SMBus 3.2 specifications and replace "master/slave" with more > appropriate terms. This first step renames the members of struct > i2c_algorithm. Once all in-tree users are converted, the anonymous union > will go away again. All this work will also pave the way for finally > seperating the monolithic header into more fine-grained headers like > "i2c/clients.h" etc. So, this is not a simple renaming-excercise but > also a chance to update the I2C core to recent Linux standards. > > Changes since v1: > > * changed wording according to the terminology we agreed on and defined > upstream. That means consistent use of "controller/target", and no > more "host/client". I added "local/remote target" where necessary. > * added tags which I kept despite some changes in wording. The approach > and code changes (if necessary) did not change. > * rebased to Andi's for-next branch > * this series only contains patches which convert the drivers fully. If > all goes well, no more updates for them are needed. The previous > series converted all users of "master_xfer". But to avoid tons of > incremental patches to one driver, I will incrementally improve i2c.h > and see which drivers can be fully converted step-by-step. > * do not mention I3C specs in commit messages, not really relevant here > > Please note that I am not super strict with the 80 char limit. And, as > agreed, I did not convert occasions where old terminology is used in > register names or bits etc. or in function names outside of the I2C > realm. > > The outcome is that before this series 115 drivers use old terminology, > after this only 54. Hooray. > > And a comment to all janitors: Do not convert I2C drivers outside of > drivers/i2c yet. Let us first gain experience here and present the > well-tested results of what we figured out to other maintainers then. > This ensures they have to deal with way less patch revisions. > > Thanks and happy hacking! > > > Wolfram Sang (60): > i2c: reword i2c_algorithm according to newest specification > i2c: ali15x3: reword according to newest specification > i2c: altera: reword according to newest specification > i2c: au1550: reword according to newest specification > i2c: bcm-kona: reword according to newest specification > i2c: bcm2835: reword according to newest specification > i2c: brcmstb: reword according to newest specification > i2c: cht-wc: reword according to newest specification > i2c: cp2615: reword according to newest specification > i2c: cros-ec-tunnel: reword according to newest specification > i2c: davinci: reword according to newest specification > i2c: digicolor: reword according to newest specification > i2c: diolan-u2c: reword according to newest specification > i2c: dln2: reword according to newest specification > i2c: fsi: reword according to newest specification > i2c: gpio: reword according to newest specification > i2c: highlander: reword according to newest specification > i2c: hisi: reword according to newest specification > i2c: hix5hd2: reword according to newest specification > i2c: i801: reword according to newest specification > i2c: ibm_iic: reword according to newest specification > i2c: iop3xx: reword according to newest specification > i2c: isch: reword according to newest specification > i2c: jz4780: reword according to newest specification > i2c: kempld: reword according to newest specification > i2c: ljca: reword according to newest specification > i2c: lpc2k: reword according to newest specification > i2c: ls2x: reword according to newest specification > i2c: mlxcpld: reword according to newest specification > i2c: mpc: reword according to newest specification > i2c: mt7621: reword according to newest specification > i2c: mv64xxx: reword according to newest specification > i2c: ocores: reword according to newest specification > i2c: octeon: reword according to newest specification > i2c: opal: reword according to newest specification > i2c: owl: reword according to newest specification > i2c: pasemi: reword according to newest specification > i2c: piix4: reword according to newest specification > i2c: powermac: reword according to newest specification > i2c: pxa-pci: reword according to newest specification > i2c: riic: reword according to newest specification > i2c: rk3x: reword according to newest specification > i2c: robotfuzz-osif: reword according to newest specification > i2c: rzv2m: reword according to newest specification > i2c: sis5595: reword according to newest specification > i2c: sprd: reword according to newest specification > i2c: stm32f4: reword according to newest specification > i2c: sun6i-p2wi: reword according to newest specification > i2c: taos-evm: reword according to newest specification > i2c: tegra-bpmp: reword according to newest specification > i2c: thunderx-pcidrv: reword according to newest specification > i2c: tiny-usb: reword according to newest specification > i2c: uniphier-f: reword according to newest specification > i2c: uniphier: reword according to newest specification > i2c: viperboard: reword according to newest specification > i2c: viai2c: reword according to newest specification > i2c: nvidia-gpu: reword according to newest specification > i2c: virtio: reword according to newest specification > i2c: cpm: reword according to newest specification > i2c: st: reword according to newest specification > > drivers/i2c/busses/i2c-ali15x3.c | 2 +- > drivers/i2c/busses/i2c-altera.c | 4 +- > drivers/i2c/busses/i2c-au1550.c | 15 +++---- > drivers/i2c/busses/i2c-bcm-kona.c | 13 +++--- > drivers/i2c/busses/i2c-bcm2835.c | 10 ++--- > drivers/i2c/busses/i2c-brcmstb.c | 11 +++-- > drivers/i2c/busses/i2c-cht-wc.c | 8 ++-- > drivers/i2c/busses/i2c-cp2615.c | 8 ++-- > drivers/i2c/busses/i2c-cpm.c | 4 +- > drivers/i2c/busses/i2c-cros-ec-tunnel.c | 4 +- > drivers/i2c/busses/i2c-davinci.c | 17 ++++--- > drivers/i2c/busses/i2c-digicolor.c | 6 +-- > drivers/i2c/busses/i2c-diolan-u2c.c | 2 +- > drivers/i2c/busses/i2c-dln2.c | 4 +- > drivers/i2c/busses/i2c-fsi.c | 56 ++++++++++++------------ > drivers/i2c/busses/i2c-gpio.c | 8 ++-- > drivers/i2c/busses/i2c-highlander.c | 2 +- > drivers/i2c/busses/i2c-hisi.c | 8 ++-- > drivers/i2c/busses/i2c-hix5hd2.c | 6 +-- > drivers/i2c/busses/i2c-i801.c | 12 ++--- > drivers/i2c/busses/i2c-ibm_iic.c | 27 +++++------- > drivers/i2c/busses/i2c-iop3xx.c | 15 +++---- > drivers/i2c/busses/i2c-isch.c | 2 +- > drivers/i2c/busses/i2c-jz4780.c | 4 +- > drivers/i2c/busses/i2c-kempld.c | 4 +- > drivers/i2c/busses/i2c-ljca.c | 20 ++++----- > drivers/i2c/busses/i2c-lpc2k.c | 10 ++--- > drivers/i2c/busses/i2c-ls2x.c | 11 +++-- > drivers/i2c/busses/i2c-mlxcpld.c | 14 +++--- > drivers/i2c/busses/i2c-mpc.c | 4 +- > drivers/i2c/busses/i2c-mt7621.c | 26 +++++------ > drivers/i2c/busses/i2c-mv64xxx.c | 12 ++--- > drivers/i2c/busses/i2c-nvidia-gpu.c | 7 ++- > drivers/i2c/busses/i2c-ocores.c | 8 ++-- > drivers/i2c/busses/i2c-octeon-core.c | 6 +-- > drivers/i2c/busses/i2c-octeon-core.h | 4 +- > drivers/i2c/busses/i2c-octeon-platdrv.c | 2 +- > drivers/i2c/busses/i2c-opal.c | 10 ++--- > drivers/i2c/busses/i2c-owl.c | 12 ++--- > drivers/i2c/busses/i2c-pasemi-core.c | 6 +-- > drivers/i2c/busses/i2c-piix4.c | 2 +- > drivers/i2c/busses/i2c-powermac.c | 14 +++--- > drivers/i2c/busses/i2c-pxa-pci.c | 2 +- > drivers/i2c/busses/i2c-riic.c | 8 ++-- > drivers/i2c/busses/i2c-rk3x.c | 20 ++++----- > drivers/i2c/busses/i2c-robotfuzz-osif.c | 4 +- > drivers/i2c/busses/i2c-rzv2m.c | 12 ++--- > drivers/i2c/busses/i2c-sis5595.c | 2 +- > drivers/i2c/busses/i2c-sprd.c | 16 +++---- > drivers/i2c/busses/i2c-st.c | 8 ++-- > drivers/i2c/busses/i2c-stm32f4.c | 8 ++-- > drivers/i2c/busses/i2c-sun6i-p2wi.c | 20 ++++----- > drivers/i2c/busses/i2c-taos-evm.c | 2 +- > drivers/i2c/busses/i2c-tegra-bpmp.c | 4 +- > drivers/i2c/busses/i2c-thunderx-pcidrv.c | 2 +- > drivers/i2c/busses/i2c-tiny-usb.c | 6 +-- > drivers/i2c/busses/i2c-uniphier-f.c | 26 +++++------ > drivers/i2c/busses/i2c-uniphier.c | 15 +++---- > drivers/i2c/busses/i2c-viai2c-common.c | 2 +- > drivers/i2c/busses/i2c-viai2c-wmt.c | 8 ++-- > drivers/i2c/busses/i2c-viai2c-zhaoxin.c | 12 ++--- > drivers/i2c/busses/i2c-viperboard.c | 10 ++--- > drivers/i2c/busses/i2c-virtio.c | 2 +- > include/linux/i2c.h | 24 +++++++--- > 64 files changed, 313 insertions(+), 320 deletions(-) > > -- > 2.43.0 >
Powered by blists - more mailing lists