All of lore.kernel.org
 help / color / mirror / Atom feed
From: arno@natisbad.org (Arnaud Ebalard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv0 0/2] arm: mvebu: fixes for previous DS213j/DS414 series
Date: Mon, 24 Nov 2014 20:04:45 +0100	[thread overview]
Message-ID: <87oarwmnci.fsf@natisbad.org> (raw)
In-Reply-To: <cover.1416670812.git.arno@natisbad.org> (Arnaud Ebalard's message of "Sat, 22 Nov 2014 17:19:21 +0100")

Hi Jason,

Arnaud Ebalard <arno@natisbad.org> writes:

> This series contains two patches. I spent some time reading the patches
> of my previous series you had to manually apply. I only noticed the error
> Ben reported on v6 (i.e. some pinctrl entries still using pmx_ instead
> of _pins) which patch 1 corrects.
>
> Regarding the second patch, having forgotten to enable MACH_ARMADA_XP in
> my kernel config, I missed a renaming of ge0/1 pinctrl entries in
> armada-xp.dtsi, which resulted in a non-compiling .dts file for DS414.
> To fix my mistake, you did a s/ge[01]_rgmii_pins/pmx_ge[01]_rgmii/,
> reintroducing some of the pmx_ naming that should have been removed in
> the process. So, the second patch in that series finishes the work to
> normalize pinctrl entries in Armada SoCs DT files.
>
> The patches do apply on your mvebu/dt branch. They do pass a whole
> compilation (including 'make dtbs') and I can boot a resulting kernel on
> my DS213j. I do not have a DS414 at hand but will test on this device
> early next week.
>
> I hope eveyhting is now back in order w/ those. Feedback appreciated.

Just tested the series on a DS414, booting a debian on a 4-disk RAID5
array. I do get an IP via DHCP from both ethernet ports. Plugging an
USB  device on rear ports (USB 3.0) or front port (USB 2.0) also works.

Cheers,

a+

WARNING: multiple messages have this Message-ID (diff)
From: arno-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org (Arnaud Ebalard)
To: Jason Cooper <jason-NLaQJdtUoK4Be96aLqz0jA@public.gmane.org>
Cc: Andrew Lunn <andrew-g2DYL2Zd6BY@public.gmane.org>,
	Gregory Clement
	<gregory.clement-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org>,
	Sebastian Hesselbarth
	<sebastian.hesselbarth-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>,
	Ben Peddell <klightspeed-aslSrjg9ejhWX4hkXwHRhw@public.gmane.org>,
	Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
	devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	Russell King <linux-lFZ/pmaqli7XmaaqVzeoHQ@public.gmane.org>,
	Pawel Moll <pawel.moll-5wv7dgnIgG8@public.gmane.org>,
	Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>,
	Ian Campbell
	<ijc+devicetree-KcIKpvwj1kUDXYZnReoRVg@public.gmane.org>,
	Rob Herring <rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org>,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCHv0 0/2] arm: mvebu: fixes for previous DS213j/DS414 series
Date: Mon, 24 Nov 2014 20:04:45 +0100	[thread overview]
Message-ID: <87oarwmnci.fsf@natisbad.org> (raw)
In-Reply-To: <cover.1416670812.git.arno-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org> (Arnaud Ebalard's message of "Sat, 22 Nov 2014 17:19:21 +0100")

Hi Jason,

Arnaud Ebalard <arno-LkuqDEemtHBg9hUCZPvPmw@public.gmane.org> writes:

> This series contains two patches. I spent some time reading the patches
> of my previous series you had to manually apply. I only noticed the error
> Ben reported on v6 (i.e. some pinctrl entries still using pmx_ instead
> of _pins) which patch 1 corrects.
>
> Regarding the second patch, having forgotten to enable MACH_ARMADA_XP in
> my kernel config, I missed a renaming of ge0/1 pinctrl entries in
> armada-xp.dtsi, which resulted in a non-compiling .dts file for DS414.
> To fix my mistake, you did a s/ge[01]_rgmii_pins/pmx_ge[01]_rgmii/,
> reintroducing some of the pmx_ naming that should have been removed in
> the process. So, the second patch in that series finishes the work to
> normalize pinctrl entries in Armada SoCs DT files.
>
> The patches do apply on your mvebu/dt branch. They do pass a whole
> compilation (including 'make dtbs') and I can boot a resulting kernel on
> my DS213j. I do not have a DS414 at hand but will test on this device
> early next week.
>
> I hope eveyhting is now back in order w/ those. Feedback appreciated.

Just tested the series on a DS414, booting a debian on a 4-disk RAID5
array. I do get an IP via DHCP from both ethernet ports. Plugging an
USB  device on rear ports (USB 3.0) or front port (USB 2.0) also works.

Cheers,

a+
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

  parent reply	other threads:[~2014-11-24 19:04 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-11-22 16:19 [PATCHv0 0/2] arm: mvebu: fixes for previous DS213j/DS414 series Arnaud Ebalard
2014-11-22 16:19 ` Arnaud Ebalard
2014-11-22 16:19 ` [PATCHv0 1/2] arm: mvebu: fix wrongly named DS414 pinctrl entries Arnaud Ebalard
2014-11-22 16:19   ` Arnaud Ebalard
2014-11-22 16:23 ` [PATCHv0 2/2] arm: mvebu: normalize pinctrl entries for Armada SoCs Arnaud Ebalard
2014-11-22 16:23   ` Arnaud Ebalard
2014-11-24 19:04 ` Arnaud Ebalard [this message]
2014-11-24 19:04   ` [PATCHv0 0/2] arm: mvebu: fixes for previous DS213j/DS414 series Arnaud Ebalard
2014-11-26  4:12 ` Jason Cooper
2014-11-26  4:12   ` Jason Cooper
2014-11-26 18:10   ` Arnaud Ebalard
2014-11-26 18:10     ` Arnaud Ebalard

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=87oarwmnci.fsf@natisbad.org \
    --to=arno@natisbad.org \
    --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.