[PATCH linux dev-5.2 v2 3/4] ARM: dts: Aspeed: ast2600: Add I2C busses

Cédric Le Goater clg at kaod.org
Thu Sep 19 01:57:24 AEST 2019


On 18/09/2019 16:53, Eddie James wrote:
> 
> On 9/18/19 5:18 AM, Cédric Le Goater wrote:
>> On 13/09/2019 18:15, Eddie James wrote:
>>> Add all the I2C busses to the AST2600 dtsi and set their required
>>> properties.
>>>
>> The DT defines an interrupt per I2C bus but this is not how the I2C
>> driver operates. It still uses the old mode from the Aspeed AST2500.
> 
> 
> Oh? The I2C interrupt controller driver is a separate driver, which I didn't include a node for in the AST2600 dts. As far as I can tell the I2C bus driver just asks for it's interrupt and uses it, so by setting each bus to it's GIC interrupt line (instead of the line from the I2C interrupt controller like in the AST2500), the I2C driver receives the correct interrupt.

ok. I am seeing this from the I2C model side and that is where the 
problem must be. I will dig in QEMU.

Thanks,

C. 


More information about the openbmc mailing list