Re: [PATCH dt-schema] schemas: i2c: add optional GPIO binding for SMBALERT# line

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



> > I had this originally in my RFC[1]. I got convinced by Geert's arguments
> > because the DT snippet in the board DTS looked kinda ugly. The board
> > needs to override the DTSI of the SoC to replace "interrupts" with
> > "interrupts-extended":
> > 
> > ===
> > 
> >  &i2c3	{
> >  	pinctrl-0 = <&i2c3_pins>;
> >  	pinctrl-names = "i2c-pwr";
> > +
> > +	/delete-property/ interrupts;
> > +	interrupts-extended = <&gic GIC_SPI 290 IRQ_TYPE_LEVEL_HIGH>, <&gpio1 26 IRQ_TYPE_EDGE_FALLING>;
> > +	interrupt-names = "main", "smbus_alert";
> > +
> > +	smbus;
> >  };
> > 
> > ===
> 
> I guess my questions here are: is this proper? Is there a better way to
> describe it? Is using interrupts still the way to go?

Hi Rob,

do you still prefer "interrupts" over "smbalert-gpios" given the above
snippet?

Thanks,

   Wolfram

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux