linux-tegra.vger.kernel.org archive mirror
 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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).