Re: [PATCH v1 03/12] drm/i915: Make I2C terminology more inclusive

Easwar Hariharan
2024-04-30T21:40:22Z

On 4/30/2024 1:29 PM, Rodrigo Vivi wrote:
> On Tue, Apr 30, 2024 at 05:38:02PM +0000, Easwar Hariharan wrote:
>> 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/
>>
>> Signed-off-by: Easwar Hariharan <eahariha@linux.microsoft.com>
>
> I'm glad to see this change!
>
> Reviewed-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
>
>> ---
>> drivers/gpu/drm/i915/display/dvo_ch7017.c | 14 ++++-----
>> drivers/gpu/drm/i915/display/dvo_ch7xxx.c | 18 +++++------
>> drivers/gpu/drm/i915/display/dvo_ivch.c | 16 +++++-----
>> drivers/gpu/drm/i915/display/dvo_ns2501.c | 18 +++++------
>> drivers/gpu/drm/i915/display/dvo_sil164.c | 18 +++++------
>> drivers/gpu/drm/i915/display/dvo_tfp410.c | 18 +++++------
>> drivers/gpu/drm/i915/display/intel_bios.c | 22 +++++++-------
>> drivers/gpu/drm/i915/display/intel_ddi.c | 2 +-
>> .../gpu/drm/i915/display/intel_display_core.h | 2 +-
>> drivers/gpu/drm/i915/display/intel_dsi.h | 2 +-
>> drivers/gpu/drm/i915/display/intel_dsi_vbt.c | 20 ++++++-------
>> drivers/gpu/drm/i915/display/intel_dvo.c | 14 ++++-----
>> drivers/gpu/drm/i915/display/intel_dvo_dev.h | 2 +-
>> drivers/gpu/drm/i915/display/intel_gmbus.c | 4 +--
>> drivers/gpu/drm/i915/display/intel_sdvo.c | 30 +++++++++----------
>> drivers/gpu/drm/i915/display/intel_vbt_defs.h | 4 +--
>> drivers/gpu/drm/i915/gvt/edid.c | 28 ++++++++---------
>> drivers/gpu/drm/i915/gvt/edid.h | 4 +--
>> drivers/gpu/drm/i915/gvt/opregion.c | 2 +-
>> 19 files changed, 119 insertions(+), 119 deletions(-)
>
> The chances of conflicts are high with this many changes,
> but should be easy enough to deal with later, so feel free
> to move with this i915 patch on any other tree and we catch-up
> later.
>
> Acked-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
>

Thanks for the review and ack! I actually thought that this might end up going in as individual
patches via the various respective trees since it's now completely independent of Wolfram's enabling
series with the drop of the final patch that was treewide.

What do you think?

Thanks,
Easwar