Re: [PATCH 1/3] i2c: xlr: add support for Sigma Designs controller variant

From: Måns Rullgård
Date: Sat Nov 21 2015 - 07:03:35 EST


Mans Rullgard <mans@xxxxxxxxx> writes:

> Sigma Designs chips use a variant of this controller with the following
> differences:
>
> - The BUSY bit in the STATUS register is inverted
> - Bit 8 of the CONFIG register must be set
> - The controller can generate interrupts
>
> This patch adds support for the first two of these. It also calculates
> and sets the correct clock divisor if a clk is provided. The bus
> frequency is optionally speficied in the device tree node.
>
> Signed-off-by: Mans Rullgard <mans@xxxxxxxxx>
> ---
> drivers/i2c/busses/Kconfig | 6 ++--
> drivers/i2c/busses/i2c-xlr.c | 81 +++++++++++++++++++++++++++++++++++++++++---
> 2 files changed, 80 insertions(+), 7 deletions(-)

Any comments on these patches?

--
Måns Rullgård
mans@xxxxxxxxx
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/