All of lore.kernel.org
 help / color / mirror / Atom feed
From: Lee Jones <lee.jones@linaro.org>
To: Dan Carpenter <dan.carpenter@oracle.com>
Cc: Robin Murphy <robin.murphy@arm.com>,
	linux-kernel@vger.kernel.org,
	linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 1/1] mfd: tps65911-comparator: Fix an off by one bug
Date: Tue, 1 May 2018 14:24:12 +0100	[thread overview]
Message-ID: <20180501132412.GO5147@dell> (raw)
In-Reply-To: <20180501131036.bm7rcujohxi5devs@mwanda>

On Tue, 01 May 2018, Dan Carpenter wrote:

> The background is that no one noticed that this driver stopped being
> buildable in 2012 until I fixed it last week.  This is the third
> proposed patch to fix the off by one.  All three patches fix the bug,
> but Lee refused to accept my style and I refused to use Lee's style...
> Absolutely *no way*!  LOL.  But really neither of us take it personally
> and we've met in person before and collaborate over email and get along.
> 
> This thread has been pretty amusing for me, but let's move on.  :P

+999

Plan is to let it sit for a week or so.  Give some more idle
passers-by to have their say.  Then I'll merge it regardless (/me
jokes).

-- 
Lee Jones [李琼斯]
Linaro Services Technical Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

WARNING: multiple messages have this Message-ID (diff)
From: lee.jones@linaro.org (Lee Jones)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 1/1] mfd: tps65911-comparator: Fix an off by one bug
Date: Tue, 1 May 2018 14:24:12 +0100	[thread overview]
Message-ID: <20180501132412.GO5147@dell> (raw)
In-Reply-To: <20180501131036.bm7rcujohxi5devs@mwanda>

On Tue, 01 May 2018, Dan Carpenter wrote:

> The background is that no one noticed that this driver stopped being
> buildable in 2012 until I fixed it last week.  This is the third
> proposed patch to fix the off by one.  All three patches fix the bug,
> but Lee refused to accept my style and I refused to use Lee's style...
> Absolutely *no way*!  LOL.  But really neither of us take it personally
> and we've met in person before and collaborate over email and get along.
> 
> This thread has been pretty amusing for me, but let's move on.  :P

+999

Plan is to let it sit for a week or so.  Give some more idle
passers-by to have their say.  Then I'll merge it regardless (/me
jokes).

-- 
Lee Jones [???]
Linaro Services Technical Lead
Linaro.org ? Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

  reply	other threads:[~2018-05-01 13:24 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-05-01  9:45 [PATCH 1/1] mfd: tps65911-comparator: Fix an off by one bug Lee Jones
2018-05-01  9:45 ` Lee Jones
2018-05-01 12:41 ` Robin Murphy
2018-05-01 12:41   ` Robin Murphy
2018-05-01 13:10   ` Dan Carpenter
2018-05-01 13:10     ` Dan Carpenter
2018-05-01 13:24     ` Lee Jones [this message]
2018-05-01 13:24       ` Lee Jones
2018-05-01 13:22   ` Lee Jones
2018-05-01 13:22     ` Lee Jones

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=20180501132412.GO5147@dell \
    --to=lee.jones@linaro.org \
    --cc=dan.carpenter@oracle.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=robin.murphy@arm.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.