netdev - Re: [PATCH net-next 2/6] net: txgbe: Implement I2C bus master driver

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 7 Apr 2023 04:03:33 +0200
From:   Andrew Lunn <andrew@...n.ch>
To:     Jiawen Wu <jiawenwu@...stnetic.com>
Cc:     netdev@...r.kernel.org, linux@...linux.org.uk,
        mengyuanlou@...-swift.com
Subject: Re: [PATCH net-next 2/6] net: txgbe: Implement I2C bus master driver

> By consulting with hardware designers, our I2C licensed the IP of
> Synopsys.
> But I can't use the I2C_DESIGNWARE_CORE directly, because IRQ is not
> supported in our I2C.

When you repost this patch and Cc: the i2c list, please point this
out. Please also Cc: the Designware I2C maintainers. They might have
some ideas how the driver can be used in polled mode.

Ideally, you want to use the existing driver, since it has hardware
work around, optimisations, and probably less bugs because it is used
and tested a lot.

And it is not too unusual for interrupt support to be missing. The
ocores driver can be used without interrupts. And there is sometimes a
polled mode supported so you can for example use the I2C bus to talk
to the power controller to turn the power at the end of shutdown when
nearly everything has stop working.

	Andrew