Re: Introducing a kernel driver for the DS28E17 Onewire to I2C master bridge; Feature request: introduce I2C_FUNC_STOP

[Jan Kandziora <jjj@gmx.de>]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Am 15.07.2016 um 13:26 schrieb Wolfram Sang:
> 
> I would say no, since it is not generic.
> 
> You should also populate an i2c_adapter_quirks structure and limit
> reads to 256 (and maybe other stuff), if not done already.
> 
Done.


Next quirk with the DS28E17: It can do an arbitrary number of write
messages, followed by ONE read message in one transfer. As there is no
"Read Without Stop Condition".

My driver code just ignores the fact there always is a Stop Condition
after a read. As the DS28E17 is hardly ever used in a multi-master
setup, that should be okay.

For the combined transfer the DS28E17 also supports, my driver code
checks if all of the conditions are met inside its
i2c_master_transfer() function and uses that function automatically
when applicable.


The current quirks flags only allows to limit the number of messages
to two in general. How to announce the behaviour correctly?


Kind regards

	Jan
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v2

iEYEARECAAYFAleJHPIACgkQzGZqmZvWQdkQFACfUYomuw8JYMdNqouP8sjpd6Nm
U5EAoJ+U6G2gDoZbJ+lGpIAGOxiTTJ52
=SCdT
-----END PGP SIGNATURE-----