Re: [PATCH v3 4/7] dt-bindings: gpio: Add gpio-repeater bindings

2020-01-15 Thread Harish Jenny K N
Hi Linus, On 07/01/20 2:52 PM, Harish Jenny K N wrote: > On 06/01/20 1:42 PM, Geert Uytterhoeven wrote: >> Hi Rob, >> >> On Fri, Dec 6, 2019 at 4:04 PM Rob Herring wrote: >>> On Fri, Dec 6, 2019 at 3:17 AM Geert Uytterhoeven >>> wrote: >>>>

Re: [PATCH v3 4/7] dt-bindings: gpio: Add gpio-repeater bindings

2020-01-07 Thread Harish Jenny K N
d, Nov 27, 2019 at 09:42:50AM +0100, Geert Uytterhoeven wrote: >>>>> Add Device Tree bindings for a GPIO repeater, with optional translation >>>>> of physical signal properties. This is useful for describing explicitly >>>>> the presence of e.g. an inverter on a G

Re: [PATCH v3 4/7] dt-bindings: gpio: Add gpio-repeater bindings

2019-12-03 Thread Harish Jenny K N
pio-inverter bindings by Harish Jenny K N > [1]. > > Note that this is different from a GPIO Nexus Node[2], which cannot do > physical signal property translation. > > While an inverter can be described implicitly by exchanging the > GPIO_ACTIVE_HIGH and GPIO_ACTIVE_LOW flags,

Re: [PATCH v3 5/7] gpio: Add GPIO Aggregator/Repeater driver

2019-12-03 Thread Harish Jenny K N
> +static int gpio_aggregator_probe(struct platform_device *pdev) > +{ > + struct device *dev = >dev; > + struct gpio_desc **descs; > + struct gpiochip_fwd *fwd; > + int i, n; > + > + n = gpiod_count(dev, NULL); > + if (n < 0) > + return n; > + > + descs =

Re: [PATCH v3 7/7] MAINTAINERS: Add GPIO Aggregator/Repeater section

2019-12-03 Thread Harish Jenny K N
On 27/11/19 2:12 PM, Geert Uytterhoeven wrote: > Add a maintainership section for the GPIO Aggregator/Repeater, covering > documentation, Device Tree bindings, and driver source code. > > Signed-off-by: Geert Uytterhoeven > --- > Harish: Do you want to be listed as maintainer, too? Yes.

Re: [PATCH v3 5/7] gpio: Add GPIO Aggregator/Repeater driver

2019-12-03 Thread Harish Jenny K N
On 03/12/19 1:47 PM, Geert Uytterhoeven wrote: > Hi Harish, > > On Tue, Dec 3, 2019 at 6:42 AM Harish Jenny K N > wrote: >>> +static int gpio_aggregator_probe(struct platform_device *pdev) >>> +{ >>> + struct device *dev = >dev; >>&g