All of lore.kernel.org
 help / color / mirror / Atom feed
From: o.schinagl@ultimaker.com (Olliver Schinagl)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 0/5] Add eeprom to the Olimex series of boards
Date: Wed, 09 Sep 2015 16:26:10 +0200	[thread overview]
Message-ID: <55F04182.2000707@ultimaker.com> (raw)
In-Reply-To: <20150909140559.GD9885@lukather>



On 09-09-15 16:05, Maxime Ripard wrote:
> On Wed, Sep 09, 2015 at 11:25:10AM +0200, Olliver Schinagl wrote:
>> Some of the olimex series of boards feature a permanently connected eeprom on
>> i2c-1. This patch series adds them on all the boards that have them according
>> to the spec-sheets.
>>
>> I explicitly added my personal --Tested-by to boards I actually tested.
> Testing your changes is the strict minimum before testing your
> patches. Just like running checkpatch on them, which you clearly
> failed to do, otherwise you would have seen that this is not the right
> syntax. Drop the tested-by, fix your SoB.
Actually, I did run check-patch and it complained about the format, but 
it was the same as when outputted with a git -s. I obviously missed 
something there! So I double check that. I will resubmit without a 
tested-by for the boards I did test it on.
>
>> There are two boards, that according to the sheets have them as well
>> and physically inspecting the boards shows them clearly, I did not
>> actually test them on those the boards that are missing the
>> --Tested-by.
> Then don't send those patches.
I put them out there so someone could pick up on that with the least 
amount of effort. I will drop them
>
> Maxime
>

-- 
Met vriendelijke groeten, Kind regards, ??????

Olliver Schinagl
Research & Development
Ultimaker B.V.

WARNING: multiple messages have this Message-ID (diff)
From: Olliver Schinagl <o.schinagl@ultimaker.com>
To: Maxime Ripard <maxime.ripard@free-electrons.com>
Cc: Rob Herring <robh+dt@kernel.org>, Pawel Moll <pawel.moll@arm.com>,
	Mark Rutland <mark.rutland@arm.com>,
	Ian Campbell <ijc+devicetree@hellion.org.uk>,
	Kumar Gala <galak@codeaurora.org>,
	Russell King <linux@arm.linux.org.uk>,
	devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
	linux-kernel@vger.kernel.org
Subject: Re: [PATCH 0/5] Add eeprom to the Olimex series of boards
Date: Wed, 09 Sep 2015 16:26:10 +0200	[thread overview]
Message-ID: <55F04182.2000707@ultimaker.com> (raw)
In-Reply-To: <20150909140559.GD9885@lukather>



On 09-09-15 16:05, Maxime Ripard wrote:
> On Wed, Sep 09, 2015 at 11:25:10AM +0200, Olliver Schinagl wrote:
>> Some of the olimex series of boards feature a permanently connected eeprom on
>> i2c-1. This patch series adds them on all the boards that have them according
>> to the spec-sheets.
>>
>> I explicitly added my personal --Tested-by to boards I actually tested.
> Testing your changes is the strict minimum before testing your
> patches. Just like running checkpatch on them, which you clearly
> failed to do, otherwise you would have seen that this is not the right
> syntax. Drop the tested-by, fix your SoB.
Actually, I did run check-patch and it complained about the format, but 
it was the same as when outputted with a git -s. I obviously missed 
something there! So I double check that. I will resubmit without a 
tested-by for the boards I did test it on.
>
>> There are two boards, that according to the sheets have them as well
>> and physically inspecting the boards shows them clearly, I did not
>> actually test them on those the boards that are missing the
>> --Tested-by.
> Then don't send those patches.
I put them out there so someone could pick up on that with the least 
amount of effort. I will drop them
>
> Maxime
>

-- 
Met vriendelijke groeten, Kind regards, 与亲切的问候

Olliver Schinagl
Research & Development
Ultimaker B.V.

  reply	other threads:[~2015-09-09 14:26 UTC|newest]

Thread overview: 20+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-09-09  9:25 [PATCH 0/5] Add eeprom to the Olimex series of boards Olliver Schinagl
2015-09-09  9:25 ` Olliver Schinagl
2015-09-09  9:25 ` [PATCH 1/5] dts: sunxi: add eeprom to A10 OLinuXino Lime board Olliver Schinagl
2015-09-09  9:25   ` Olliver Schinagl
2015-09-09  9:25 ` [PATCH 2/5] dts: sunxi: correct vendor prefix on OLinuXino a10s micro Olliver Schinagl
2015-09-09  9:25   ` Olliver Schinagl
2015-09-09  9:25   ` Olliver Schinagl
2015-09-09  9:25 ` [PATCH 3/5] dts: sunxi: add eeprom to A20 OLinuXino Lime board Olliver Schinagl
2015-09-09  9:25   ` Olliver Schinagl
2015-09-09  9:25   ` Olliver Schinagl
2015-09-09  9:25 ` [PATCH 4/5] dts: sunxi: add eeprom to A20 OLinuXino Lime2 board Olliver Schinagl
2015-09-09  9:25   ` Olliver Schinagl
2015-09-09  9:25   ` Olliver Schinagl
2015-09-09  9:25 ` [PATCH 5/5] dts: sunxi: add eeprom to A20 OLinuXino micro board Olliver Schinagl
2015-09-09  9:25   ` Olliver Schinagl
2015-09-09 14:05 ` [PATCH 0/5] Add eeprom to the Olimex series of boards Maxime Ripard
2015-09-09 14:05   ` Maxime Ripard
2015-09-09 14:05   ` Maxime Ripard
2015-09-09 14:26   ` Olliver Schinagl [this message]
2015-09-09 14:26     ` Olliver Schinagl

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=55F04182.2000707@ultimaker.com \
    --to=o.schinagl@ultimaker.com \
    --cc=linux-arm-kernel@lists.infradead.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.