[v2,dt-schema,0/3] schemas: i2c: fix "smbus_alert" handling

Message

Wolfram Sang Aug. 15, 2024, 9:40 p.m. UTC

"smbus_alert" is currently listed as an interrupt for targets. This is
wrong, it is an interrupt for the controller. The mistake (mea culpa!)
was already in the originating "i2c.txt" file in the Linux Kernel. Time
to fix it. Make small changes to descriptions to use inclusive language
while we are here.

Changes since v1:
* use "contains" instead of "items" in patch 2+3
* drop "irq" binding from patch 3
* checked with dt-doc-validate now

Thanks, Rob, for the support!


Wolfram Sang (3):
  schemas: i2c: reword descriptions for inclusive language
  schemas: i2c: clean up interrupt descriptions for I2C targets
  schemas: i2c: add generic interrupt name for I2C controllers

 dtschema/schemas/i2c/i2c-controller.yaml | 37 +++++++++++++++++-------
 1 file changed, 26 insertions(+), 11 deletions(-)

Comments

Rob Herring Aug. 15, 2024, 10:48 p.m. UTC | #1

On Thu, Aug 15, 2024 at 11:40:10PM +0200, Wolfram Sang wrote:
> "smbus_alert" is currently listed as an interrupt for targets. This is
> wrong, it is an interrupt for the controller. The mistake (mea culpa!)
> was already in the originating "i2c.txt" file in the Linux Kernel. Time
> to fix it. Make small changes to descriptions to use inclusive language
> while we are here.
> 
> Changes since v1:
> * use "contains" instead of "items" in patch 2+3
> * drop "irq" binding from patch 3
> * checked with dt-doc-validate now
> 
> Thanks, Rob, for the support!
> 
> 
> Wolfram Sang (3):
>   schemas: i2c: reword descriptions for inclusive language
>   schemas: i2c: clean up interrupt descriptions for I2C targets
>   schemas: i2c: add generic interrupt name for I2C controllers

Applied, thanks!

Rob