Re: [PATCH v3 1/4] i2c: designware: Introduce shutdown exported function

[Andy Shevchenko <andriy.shevchenko@linux.intel.com>]

On Thu, May 07, 2026 at 12:56:50AM -0700, William A. Kennington III wrote:
> On 5/5/26 00:29, Andy Shevchenko wrote:
> > On Mon, May 04, 2026 at 08:15:02PM +0000, William A. Kennington III wrote:
> > > Introduce an exported shutdown function to safely shutdown the
> > > DesignWare I2C controller.
> > > 
> > > This shutdown hook gracefully sets the slave disable bit before disabling
> > > the controller. This guarantees that any incoming requests from the master
> > > are immediately NACKed during shutdown, preventing the bus from hanging.
...

> Is this just language correction?

I have asked for replacing 'slave'/'master' terminology to a new,
inclusive one.

...


> > > +	/*
> > > +	 * We only need to handle shutdown for slave mode to ensure
> > > +	 * we NACK any incoming master requests. Master mode cleanup
> > > +	 * is handled after each transfer in i2c_dw_xfer.
> > i2c_dw_xfer()
> > 
> > > +	 */
> > Since it's a newly added comment, can you switch to use inclusive language?
> > Same for the whole series related to the commit messages, comments, and
> > documentation.

Same here and...

> > > +	/*
> > > +	 * To quickly NACK the master during shutdown, we set the slave
> > > +	 * disable bit while the controller is still enabled.
> > > +	 */

...here.

...

> > > +EXPORT_SYMBOL_GPL(i2c_dw_shutdown);
> > Can we use namespace?
> 
> It already does use namespaces with
> 
> `#define DEFAULT_SYMBOL_NAMESPACE    "I2C_DW_COMMON"`
> 
> I assume I should just keep it consistent as-is?

Yep, thanks for pointing that out!

-- 
With Best Regards,
Andy Shevchenko