diff mbox

[v2,15/20] arm: kirkwood: topkick: the Topkick platform uses a 88F6282 SoC

Message ID 1356101359-6054-16-git-send-email-thomas.petazzoni@free-electrons.com (mailing list archive)
State New, archived
Headers show

Commit Message

Thomas Petazzoni Dec. 21, 2012, 2:49 p.m. UTC
The Topkick board uses a 88F6282 SoC, so the Device Tree Source
describing the Topkick should inherit from kirkwood-6282.dtsi, which
allows to take advantage of pin muxing capabilities.

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
---
Changes since v1:
 * This patch is new in v2
---
 arch/arm/boot/dts/kirkwood-topkick.dts |    1 +
 1 file changed, 1 insertion(+)

Comments

Jason Cooper Jan. 10, 2013, 2:29 a.m. UTC | #1
On Fri, Dec 21, 2012 at 03:49:14PM +0100, Thomas Petazzoni wrote:
> The Topkick board uses a 88F6282 SoC, so the Device Tree Source
> describing the Topkick should inherit from kirkwood-6282.dtsi, which
> allows to take advantage of pin muxing capabilities.
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> ---
> Changes since v1:
>  * This patch is new in v2
> ---
>  arch/arm/boot/dts/kirkwood-topkick.dts |    1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/arch/arm/boot/dts/kirkwood-topkick.dts b/arch/arm/boot/dts/kirkwood-topkick.dts
> index c0de5a7..8f4a5b8 100644
> --- a/arch/arm/boot/dts/kirkwood-topkick.dts
> +++ b/arch/arm/boot/dts/kirkwood-topkick.dts
> @@ -1,6 +1,7 @@
>  /dts-v1/;
>  
>  /include/ "kirkwood.dtsi"
> +/include/ "kirkwood-6282.dtsi"

Hmmm, shouldn't the cooresponding MPP.._GPIO and kirkwood_mpp_conf() be
removed as well?

thx,

Jason.

>  
>  / {
>  	model = "Univeral Scientific Industrial Co. Topkick-1281P2";
> -- 
> 1.7.9.5
> 
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Andrew Lunn Jan. 10, 2013, 6:13 a.m. UTC | #2
On Wed, Jan 09, 2013 at 09:29:26PM -0500, Jason Cooper wrote:
> On Fri, Dec 21, 2012 at 03:49:14PM +0100, Thomas Petazzoni wrote:
> > The Topkick board uses a 88F6282 SoC, so the Device Tree Source
> > describing the Topkick should inherit from kirkwood-6282.dtsi, which
> > allows to take advantage of pin muxing capabilities.
> > 
> > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> > ---
> > Changes since v1:
> >  * This patch is new in v2
> > ---
> >  arch/arm/boot/dts/kirkwood-topkick.dts |    1 +
> >  1 file changed, 1 insertion(+)
> > 
> > diff --git a/arch/arm/boot/dts/kirkwood-topkick.dts b/arch/arm/boot/dts/kirkwood-topkick.dts
> > index c0de5a7..8f4a5b8 100644
> > --- a/arch/arm/boot/dts/kirkwood-topkick.dts
> > +++ b/arch/arm/boot/dts/kirkwood-topkick.dts
> > @@ -1,6 +1,7 @@
> >  /dts-v1/;
> >  
> >  /include/ "kirkwood.dtsi"
> > +/include/ "kirkwood-6282.dtsi"
> 
> Hmmm, shouldn't the cooresponding MPP.._GPIO and kirkwood_mpp_conf() be
> removed as well?

Hi Jason

I have some patches for that. I will dig them out over the weekend.

I did have an issue with one of the LEDs. If i remember correctly, i
sent you an email about it. Could you look back in your archive? 

Thanks
  Andrew
Jason Cooper Jan. 10, 2013, 11:50 a.m. UTC | #3
On Thu, Jan 10, 2013 at 07:13:09AM +0100, Andrew Lunn wrote:
> On Wed, Jan 09, 2013 at 09:29:26PM -0500, Jason Cooper wrote:
> > On Fri, Dec 21, 2012 at 03:49:14PM +0100, Thomas Petazzoni wrote:
> > > The Topkick board uses a 88F6282 SoC, so the Device Tree Source
> > > describing the Topkick should inherit from kirkwood-6282.dtsi, which
> > > allows to take advantage of pin muxing capabilities.
> > > 
> > > Signed-off-by: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
> > > ---
> > > Changes since v1:
> > >  * This patch is new in v2
> > > ---
> > >  arch/arm/boot/dts/kirkwood-topkick.dts |    1 +
> > >  1 file changed, 1 insertion(+)
> > > 
> > > diff --git a/arch/arm/boot/dts/kirkwood-topkick.dts b/arch/arm/boot/dts/kirkwood-topkick.dts
> > > index c0de5a7..8f4a5b8 100644
> > > --- a/arch/arm/boot/dts/kirkwood-topkick.dts
> > > +++ b/arch/arm/boot/dts/kirkwood-topkick.dts
> > > @@ -1,6 +1,7 @@
> > >  /dts-v1/;
> > >  
> > >  /include/ "kirkwood.dtsi"
> > > +/include/ "kirkwood-6282.dtsi"
> > 
> > Hmmm, shouldn't the cooresponding MPP.._GPIO and kirkwood_mpp_conf() be
> > removed as well?
> 
> Hi Jason
> 
> I have some patches for that. I will dig them out over the weekend.
> 
> I did have an issue with one of the LEDs. If i remember correctly, i
> sent you an email about it. Could you look back in your archive? 

Yes, I have it on my todo list.  Unfortunately, I haven't unpacked my
dev plats after my office relocation.  :(  Hopefully, I'll get that set
up this weekend.

thx,

Jason.
diff mbox

Patch

diff --git a/arch/arm/boot/dts/kirkwood-topkick.dts b/arch/arm/boot/dts/kirkwood-topkick.dts
index c0de5a7..8f4a5b8 100644
--- a/arch/arm/boot/dts/kirkwood-topkick.dts
+++ b/arch/arm/boot/dts/kirkwood-topkick.dts
@@ -1,6 +1,7 @@ 
 /dts-v1/;
 
 /include/ "kirkwood.dtsi"
+/include/ "kirkwood-6282.dtsi"
 
 / {
 	model = "Univeral Scientific Industrial Co. Topkick-1281P2";