Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1505571 > unrolled thread

Re: [PATCH 05/10] gpio: i2cs: Juniper I2CS to GPIO pin mapping driver

Started byLinus Walleij <linus.walleij@linaro.org>
First post2016-10-21 10:50 +0200
Last post2016-10-21 10:50 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [PATCH 05/10] gpio: i2cs: Juniper I2CS to GPIO pin mapping driver Linus Walleij <linus.walleij@linaro.org> - 2016-10-21 10:50 +0200

#1505571 — Re: [PATCH 05/10] gpio: i2cs: Juniper I2CS to GPIO pin mapping driver

FromLinus Walleij <linus.walleij@linaro.org>
Date2016-10-21 10:50 +0200
SubjectRe: [PATCH 05/10] gpio: i2cs: Juniper I2CS to GPIO pin mapping driver
Message-ID<suDGx-37e-3@gated-at.bofh.it>
On Fri, Oct 7, 2016 at 5:21 PM, Pantelis Antoniou
<pantelis.antoniou@konsulko.com> wrote:

> From: Guenter Roeck <groeck@juniper.net>
>
> This driver maps I2C slave register bits to GPIO pins. Registers
> are supposed to be 8 bit wide. Interrupt support is optional.
>
> The driver is implemented as client of the I2CS MFD driver.
>
> Signed-off-by: Georgi Vlaev <gvlaev@juniper.net>
> Signed-off-by: Guenter Roeck <groeck@juniper.net>
> Signed-off-by: JawaharBalaji Thirumalaisamy <jawaharb@juniper.net>
> [Ported from Juniper kernel]
> Signed-off-by: Pantelis Antoniou <pantelis.antoniou@konsulko.com>

This driver again has the same review comments as the two first
ones, GPIOLIB_IRQCHIP etc. Revise accordingly and repost
and I will look at it again.

(Interestingly this one uses BIT() properly.)

Yours,
Linus Walleij

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web