On 5/2/2024 3:46 AM, Thomas Zimmermann wrote:

Am 30.04.24 um 19:38 schrieb Easwar Hariharan:

quoted

I2C v7, SMBus 3.2, and I3C 1.1.1 specifications have replaced "master/slave"
with more appropriate terms. Inspired by and following on to Wolfram's
series to fix drivers/i2c/[1], fix the terminology for users of
I2C_ALGOBIT bitbanging interface, now that the approved verbiage exists
in the specification.

Compile tested, no functionality changes intended

[1]: https://lore.kernel.org/all/20240322132619.6389-1-wsa+renesas@sang-engineering.com/ (local)

Signed-off-by: Easwar Hariharan <redacted>

Acked-by: Thomas Zimmermann <tzimmermann@suse.de>

Thanks for the ack! I had been addressing feedback as I got it on the v0 series, and it seems
I missed out on updating viafb and smscufx to spec-compliant controller/target terminology like
the v0->v1 changelog calls out before posting v1.

For smscufx, I feel phrasing the following line (as an example)

-/* sets up I2C Controller for 100 Kbps, std. speed, 7-bit addr, host, 
+/* sets up I2C Controller for 100 Kbps, std. speed, 7-bit addr, *controller*, 

would actually impact readability negatively, so I propose to leave smscufx as is.

For viafb, I propose making it compliant with the spec using the controller/target terminology and
posting a v2 respin (which I can send out as soon as you say) and ask you to review again.

What do you think?

Thanks,
Easwar

quoted

---
  drivers/video/fbdev/via/chip.h    |  8 ++++----
  drivers/video/fbdev/via/dvi.c     | 24 ++++++++++++------------
  drivers/video/fbdev/via/lcd.c     |  6 +++---
  drivers/video/fbdev/via/via_aux.h |  2 +-
  drivers/video/fbdev/via/via_i2c.c | 12 ++++++------
  drivers/video/fbdev/via/vt1636.c  |  6 +++---
  6 files changed, 29 insertions(+), 29 deletions(-)

<snip>