All of lore.kernel.org
 help / color / mirror / Atom feed
From: Nandor Han <nandor.han@vaisala.com>
To: Sebastian Reichel <sebastian.reichel@collabora.com>,
	"Vaittinen, Matti" <Matti.Vaittinen@fi.rohmeurope.com>,
	Ian Ray <ian.ray@ge.com>
Cc: "andriy.shevchenko@linux.intel.com" 
	<andriy.shevchenko@linux.intel.com>,
	"geert+renesas@glider.be" <geert+renesas@glider.be>,
	"linus.walleij@linaro.org" <linus.walleij@linaro.org>,
	"andrew@aj.id.au" <andrew@aj.id.au>,
	"bgolaszewski@baylibre.com" <bgolaszewski@baylibre.com>,
	"linux-renesas-soc@vger.kernel.org" 
	<linux-renesas-soc@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v2 0/2] Add definition for GPIO direction
Date: Fri, 8 Nov 2019 09:46:16 +0200	[thread overview]
Message-ID: <8722c9bd-6fc9-681a-d5e0-b8c7884ea69c@vaisala.com> (raw)
In-Reply-To: <20191107120859.zoml7cmxbtaetjsk@earth.universe>

On 11/7/19 2:08 PM, Sebastian Reichel wrote:
> Hi,
> 
> On Thu, Nov 07, 2019 at 08:52:11AM +0000, Vaittinen, Matti wrote:
>> That reminded me.. I got 'unknown recipient' replies
>> <nandor.han@ge.com>: 550 5.1.1 No such user - pp
>> <semi.malinen@ge.com>: 550 5.1.1 No such user - pp
>>
>> from both Semi and Nandor - who seem to be the only maintainers listed
>> for XRA1403 GPIO EXPANDER in MAINTAINERS.
>>
>> XRA1403 GPIO EXPANDER
>> M:      Nandor Han <nandor.han@ge.com>
>> M:      Semi Malinen <semi.malinen@ge.com>
>> L:      linux-gpio@vger.kernel.org
>> S:      Maintained
>> F:      drivers/gpio/gpio-xra1403.c
>> F:      Documentation/devicetree/bindings/gpio/gpio-xra1403.txt
>>
>> Anyone knows new mail addresses or perhaps the separate entry
>> for XRA1403 should be removed?
> 
> +cc new mail address from Nandor Han
> +cc Ian Ray with valid GE mail address
> 
> -- Sebastian
> 

Thanks Sebastian.

Matti, I did review the gpio-xra1403 changes and LGTM.

According with the datasheet, 1 means direction input:

XRA1403 datasheet (https://www.maxlinear.com/ds/xra1403_100_092011.pdf)
```
  Setting these bits to ’1’ will enable the GPIOs as inputs.
```

Note: I will update also the maintainers list.

Nandor


WARNING: multiple messages have this Message-ID (diff)
From: Nandor Han <nandor.han@vaisala.com>
To: Sebastian Reichel <sebastian.reichel@collabora.com>,
	"Vaittinen, Matti" <Matti.Vaittinen@fi.rohmeurope.com>,
	Ian Ray <ian.ray@ge.com>
Cc: "geert+renesas@glider.be" <geert+renesas@glider.be>,
	"andrew@aj.id.au" <andrew@aj.id.au>,
	"linus.walleij@linaro.org" <linus.walleij@linaro.org>,
	"linux-renesas-soc@vger.kernel.org"
	<linux-renesas-soc@vger.kernel.org>,
	"bgolaszewski@baylibre.com" <bgolaszewski@baylibre.com>,
	"andriy.shevchenko@linux.intel.com"
	<andriy.shevchenko@linux.intel.com>,
	"linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v2 0/2] Add definition for GPIO direction
Date: Fri, 8 Nov 2019 09:46:16 +0200	[thread overview]
Message-ID: <8722c9bd-6fc9-681a-d5e0-b8c7884ea69c@vaisala.com> (raw)
In-Reply-To: <20191107120859.zoml7cmxbtaetjsk@earth.universe>

On 11/7/19 2:08 PM, Sebastian Reichel wrote:
> Hi,
> 
> On Thu, Nov 07, 2019 at 08:52:11AM +0000, Vaittinen, Matti wrote:
>> That reminded me.. I got 'unknown recipient' replies
>> <nandor.han@ge.com>: 550 5.1.1 No such user - pp
>> <semi.malinen@ge.com>: 550 5.1.1 No such user - pp
>>
>> from both Semi and Nandor - who seem to be the only maintainers listed
>> for XRA1403 GPIO EXPANDER in MAINTAINERS.
>>
>> XRA1403 GPIO EXPANDER
>> M:      Nandor Han <nandor.han@ge.com>
>> M:      Semi Malinen <semi.malinen@ge.com>
>> L:      linux-gpio@vger.kernel.org
>> S:      Maintained
>> F:      drivers/gpio/gpio-xra1403.c
>> F:      Documentation/devicetree/bindings/gpio/gpio-xra1403.txt
>>
>> Anyone knows new mail addresses or perhaps the separate entry
>> for XRA1403 should be removed?
> 
> +cc new mail address from Nandor Han
> +cc Ian Ray with valid GE mail address
> 
> -- Sebastian
> 

Thanks Sebastian.

Matti, I did review the gpio-xra1403 changes and LGTM.

According with the datasheet, 1 means direction input:

XRA1403 datasheet (https://www.maxlinear.com/ds/xra1403_100_092011.pdf)
```
  Setting these bits to ’1’ will enable the GPIOs as inputs.
```

Note: I will update also the maintainers list.

Nandor


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2019-11-08  7:53 UTC|newest]

Thread overview: 52+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-06  8:51 [PATCH v2 0/2] Add definition for GPIO direction Matti Vaittinen
2019-11-06  8:51 ` Matti Vaittinen
2019-11-06  8:51 ` [alsa-devel] " Matti Vaittinen
2019-11-06  8:51 ` [PATCH v2 1/2] gpio: " Matti Vaittinen
2019-11-06  8:51   ` Matti Vaittinen
2019-11-06  8:51   ` [alsa-devel] " Matti Vaittinen
2019-11-07  8:20   ` Linus Walleij
2019-11-07  8:20     ` Linus Walleij
2019-11-07  8:20     ` [alsa-devel] " Linus Walleij
2019-11-06  8:54 ` [PATCH v2 2/2] gpio: Use new GPIO_LINE_DIRECTION Matti Vaittinen
2019-11-06  8:54   ` Matti Vaittinen
2019-11-06  8:54   ` [alsa-devel] " Matti Vaittinen
2019-11-06 14:22   ` Andy Shevchenko
2019-11-06 14:22     ` Andy Shevchenko
2019-11-06 14:22     ` [alsa-devel] " Andy Shevchenko
2019-11-06 14:43   ` William Breathitt Gray
2019-11-06 14:43     ` William Breathitt Gray
2019-11-06 14:43     ` [alsa-devel] " William Breathitt Gray
2019-11-06 17:35   ` Kuppuswamy Sathyanarayanan
2019-11-06 17:35     ` Kuppuswamy Sathyanarayanan
2019-11-06 17:35     ` [alsa-devel] " Kuppuswamy Sathyanarayanan
2019-11-07  8:33   ` Linus Walleij
2019-11-06 12:08 ` [PATCH v2 0/2] Add definition for GPIO direction Sebastian Reichel
2019-11-06 12:08   ` Sebastian Reichel
2019-11-06 12:08   ` [alsa-devel] " Sebastian Reichel
2019-11-06 12:25   ` Vaittinen, Matti
2019-11-06 12:25     ` Vaittinen, Matti
2019-11-06 12:25     ` [alsa-devel] " Vaittinen, Matti
2019-11-06 14:24     ` andriy.shevchenko
2019-11-06 14:24       ` andriy.shevchenko
2019-11-06 14:24       ` [alsa-devel] " andriy.shevchenko
2019-11-07  8:26       ` Linus Walleij
2019-11-07  8:26         ` Linus Walleij
2019-11-07  8:32         ` Geert Uytterhoeven
2019-11-07  8:32           ` Geert Uytterhoeven
2019-11-07  8:52         ` Vaittinen, Matti
2019-11-07  8:52           ` Vaittinen, Matti
2019-11-07 12:08           ` Sebastian Reichel
2019-11-07 12:08             ` Sebastian Reichel
2019-11-08  7:46             ` Nandor Han [this message]
2019-11-08  7:46               ` Nandor Han
2019-11-07  8:52         ` andriy.shevchenko
2019-11-07  8:52           ` andriy.shevchenko
2019-11-07  8:22     ` Linus Walleij
2019-11-07  8:22       ` Linus Walleij
2019-11-07  8:22       ` [alsa-devel] " Linus Walleij
2019-11-06 14:39 ` William Breathitt Gray
2019-11-06 14:39   ` William Breathitt Gray
2019-11-06 14:39   ` [alsa-devel] " William Breathitt Gray
2019-11-06 17:28   ` Vaittinen, Matti
2019-11-06 17:28     ` Vaittinen, Matti
2019-11-06 17:28     ` [alsa-devel] " Vaittinen, Matti

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8722c9bd-6fc9-681a-d5e0-b8c7884ea69c@vaisala.com \
    --to=nandor.han@vaisala.com \
    --cc=Matti.Vaittinen@fi.rohmeurope.com \
    --cc=andrew@aj.id.au \
    --cc=andriy.shevchenko@linux.intel.com \
    --cc=bgolaszewski@baylibre.com \
    --cc=geert+renesas@glider.be \
    --cc=ian.ray@ge.com \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=sebastian.reichel@collabora.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.