linux-kernel - Re: [PATCH] i2c: cadence: Added slave support

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]

Message-ID: <6504123809effc310ade02dbb8a63f10db6b6c92.camel@upb.ro>
Date:   Thu, 19 Dec 2019 15:29:30 +0200
From:   Radu Pirea <radu_nicolae.pirea@....ro>
To:     Wolfram Sang <wsa@...-dreams.de>
Cc:     linux-arm-kernel@...ts.infradead.org, linux-i2c@...r.kernel.org,
        linux-kernel@...r.kernel.org,
        Chirag Parekh <chirag.parekh@...inx.com>,
        Michal Simek <michal.simek@...inx.com>
Subject: Re: [PATCH] i2c: cadence: Added slave support

On Thu, 2019-12-19 at 14:05 +0100, Wolfram Sang wrote:
> > +/**
> > + * enum cdns_i2c_mode - I2C Controller current operating mode
> > + *
> > + * @CDNS_I2C_MODE_SLAVE:       I2C controller operating in slave
> > mode
> > + * @CDNS_I2C_MODE_MASTER:      I2C Controller operating in master
> > mode
> > + */
> 
> Can't the hardware operate as master and slave at the same time?
> 

Of course, it can. If the driver has a slave registered wait and
listens and if the subsystem needs to use the controller as master, the
driver changes the state of the controller to master, sends and reads
data from the bus and after this change the state of the controller to
slave. In cdns_i2c_master_xfer is done all the magic.