All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ben Dooks <ben.dooks-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
To: Alexandre Courbot
	<gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Thierry Reding
	<thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: "linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	Linus Walleij
	<linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
	ct173-kernel-internal-81qHHgoATdFT9dQujB1mzip2UmYkHbXO@public.gmane.org,
	Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
Subject: Re: [Ct173-kernel-internal] [PATCH 2/4] ARM: gpio: tegra: Maintain endianness
Date: Wed, 11 Mar 2015 10:03:46 +0000	[thread overview]
Message-ID: <55001302.2020501@codethink.co.uk> (raw)
In-Reply-To: <CAAVeFuLenb8V3rk6bbc2uDWo65sWZ2YLtFfuRLt2yEZ-3KwFwQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>

On 11/03/15 09:58, Alexandre Courbot wrote:
> On Wed, Mar 11, 2015 at 6:55 PM, Thierry Reding
> <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>> On Wed, Mar 11, 2015 at 06:46:10PM +0900, Alexandre Courbot wrote:
>>> On Wed, Mar 11, 2015 at 6:43 PM, Thierry Reding
>>> <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>>>> On Fri, Jan 09, 2015 at 03:27:09PM +0000, Bob Mottram wrote:
>>>>> Replace the _raw-writel with writel_relaxed to fix issue with
>>>>> running tegra in big-endian. Tested on Jetson TK1.
>>>>>
>>>>> Signed-off-by: Bob Mottram <bob.mottram-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
>>>>> ---
>>>>>  drivers/gpio/gpio-tegra.c | 4 ++--
>>>>>  1 file changed, 2 insertions(+), 2 deletions(-)
>>>>
>>>> Linus, Alex,
>>>>
>>>> would you prefer to take this through the GPIO tree or shall I take it
>>>> into the Tegra tree along with the rest of this series. In case of the
>>>> former:
>>>>
>>>> Acked-by: Thierry Reding <treding-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>>>
>>>> I see that the patch wasn't Cc'ed to linux-gpio or Linus so, for
>>>> reference, here's a link to it in patchwork:
>>>>
>>>>         http://patchwork.ozlabs.org/patch/427142/
>>>
>>> Please take it in the Tegra tree, that way the patches of this series
>>> will appear sequentially in the git log instead of being spread across
>>> various trees without any hint about their relation.
>>
>> Does that imply an Acked-by from you as GPIO maintainer?
> 
> My bad:
> 
> Acked-by: Alexandre Courbot <gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>

Thanks for sorting this out.

-- 
Ben Dooks				http://www.codethink.co.uk/
Senior Engineer				Codethink - Providing Genius

  parent reply	other threads:[~2015-03-11 10:03 UTC|newest]

Thread overview: 27+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-01-09 15:27 [PATCH 0/4] ARM: tegra: Support for big-endian (version 3) Bob Mottram
     [not found] ` <1420817231-8346-1-git-send-email-bob.mottram-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-09 15:27   ` [PATCH 1/4] ARM: tegra: Add a separate big-endian configuration Bob Mottram
2015-01-09 15:27   ` [PATCH 2/4] ARM: gpio: tegra: Maintain endianness Bob Mottram
     [not found]     ` <1420817231-8346-3-git-send-email-bob.mottram-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-27 18:41       ` Dmitry Osipenko
2015-03-11  9:43       ` Thierry Reding
     [not found]         ` <20150311094324.GH19577-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2015-03-11  9:46           ` Alexandre Courbot
     [not found]             ` <CAAVeFuLBz0UJsVEi=EYPbGd2x3Q6_JOELb4SHspR+w=tBS8mZw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-03-11  9:55               ` Thierry Reding
     [not found]                 ` <20150311095543.GJ19577-AwZRO8vwLAwmlAP/+Wk3EA@public.gmane.org>
2015-03-11  9:58                   ` Alexandre Courbot
     [not found]                     ` <CAAVeFuLenb8V3rk6bbc2uDWo65sWZ2YLtFfuRLt2yEZ-3KwFwQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-03-11 10:03                       ` Ben Dooks [this message]
2015-03-18  1:36           ` Linus Walleij
2015-01-09 15:27   ` [PATCH 3/4] ARM: tegra: enable big-endian support Bob Mottram
     [not found]     ` <1420817231-8346-4-git-send-email-bob.mottram-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-09 15:46       ` Dmitry Osipenko
     [not found]         ` <54AFF7E6.9030308-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-09 16:27           ` Bob Mottram
     [not found]             ` <54B0017E.2060509-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-09 16:32               ` Dmitry Osipenko
     [not found]                 ` <54B0028C.8050802-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-09 17:14                   ` [Linux-kernel] " Ben Dooks
     [not found]                     ` <54B00C58.3070303-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-12 18:04                       ` Dmitry Osipenko
     [not found]                         ` <54B40C99.4020907-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-13  8:47                           ` Ben Dooks
2015-01-13  9:05                           ` Bob Mottram
2015-01-19 14:26                           ` Bob Mottram
     [not found]                             ` <54BD141F.7070803-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-19 15:08                               ` Dmitry Osipenko
2015-01-09 15:27   ` [PATCH 4/4] ARM: tegra: fix big-endian issue with irq code Bob Mottram
     [not found]     ` <1420817231-8346-5-git-send-email-bob.mottram-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-27 18:42       ` Dmitry Osipenko
2015-03-11  9:48       ` Thierry Reding
2015-01-20 10:31   ` [Ct173-kernel-internal] [PATCH 0/4] ARM: tegra: Support for big-endian (version 3) Ben Dooks
     [not found]     ` <54BE2E77.9050909-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-20 11:36       ` Dmitry Osipenko
     [not found]         ` <54BE3DD2.1060204-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-01-20 11:45           ` Ben Dooks
     [not found]             ` <54BE3FC1.5010107-4yDnlxn2s6sWdaTGBSpHTA@public.gmane.org>
2015-01-20 13:18               ` Dmitry Osipenko

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=55001302.2020501@codethink.co.uk \
    --to=ben.dooks-4ydnlxn2s6swdatgbsphta@public.gmane.org \
    --cc=ct173-kernel-internal-81qHHgoATdFT9dQujB1mzip2UmYkHbXO@public.gmane.org \
    --cc=gnurou-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    --cc=linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org \
    --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \
    --cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org \
    /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.