mx35pdk lan911x support

mx35pdk lan911x support

[Uwe Kleine-König]

Hello Xiao (hoping this is the part of your name corresponding to your
first name),

On Thu, Oct 14, 2010 at 12:06:25AM -0400, xiao jiang wrote:
> I saw the mach-mx35_3ds.c  are come from you, so  I think you are the right
> person to answer my queston, :).
It was me, who renamed the file and touched it a few times when cleaning
up and reorganising things for the i.MX machines.  Initially it was
Fabio Estevam who brought the machine into mainline.

I suggest you ask on the linux-arm-kernel mailing (added to Cc:).

> I tried the newest kernel(2.6.36.rc7) to boot mx35pdk board, and selected
> lan911x  not fec as the ethernet port,
> but I failed boot it because the ethernet problem.
> 
> My question is for mx35pdk the kernel support boot from lan911x?
I lost my crystal ball and will only find it next week.  So I'm unable
to see the exact cause of the error.  You might want to follow up with
the kernel boot log and your adaptions to mach-mx35_3ds.c.  You seem not
to have an original 3ds machine, so I'd suggest you register your own
machine type for it.

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-K?nig            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

mx35pdk lan911x support

[Fabio Estevam]

Hi Xiao Jiang,

2010/10/14 Uwe Kleine-K?nig <u.kleine-koenig@pengutronix.de>:
> Hello Xiao (hoping this is the part of your name corresponding to your
> first name),
>
> On Thu, Oct 14, 2010 at 12:06:25AM -0400, xiao jiang wrote:
>> I saw the mach-mx35_3ds.c ?are come from you, so ?I think you are the right
>> person to answer my queston, :).
> It was me, who renamed the file and touched it a few times when cleaning
> up and reorganising things for the i.MX machines. ?Initially it was
> Fabio Estevam who brought the machine into mainline.

I suggest you to look at MX51 3stack board or MX31 3stack boards, as
they have support for LAN9217 controller.

Regards,

Fabio Estevam

mx35pdk lan911x support

[xiao jiang]

Hi Uwe & Fabio

2010/10/14 Fabio Estevam <festevam@gmail.com>

> Hi Xiao Jiang,
>
> 2010/10/14 Uwe Kleine-K?nig <u.kleine-koenig@pengutronix.de>:
> > Hello Xiao (hoping this is the part of your name corresponding to your
> > first name),
> >
> > On Thu, Oct 14, 2010 at 12:06:25AM -0400, xiao jiang wrote:
> >> I saw the mach-mx35_3ds.c  are come from you, so  I think you are the
> right
> >> person to answer my queston, :).
> > It was me, who renamed the file and touched it a few times when cleaning
> > up and reorganising things for the i.MX machines.  Initially it was
> > Fabio Estevam who brought the machine into mainline.
>
> I suggest you to look at MX51 3stack board or MX31 3stack boards, as
> they have support for LAN9217 controller.
>
> Thanks for your help, I can boot the kernel thru lan9217 now, only a few
lines added.

diff --git a/arch/arm/mach-mx3/mach-mx35_3ds.c
b/arch/arm/mach-mx3/mach-mx35_3ds.c
index 1c30d72..7e30a4e 100644
--- a/arch/arm/mach-mx3/mach-mx35_3ds.c
+++ b/arch/arm/mach-mx3/mach-mx35_3ds.c
@@ -35,6 +35,7 @@
 #include <mach/hardware.h>
 #include <mach/common.h>
 #include <mach/iomux-mx35.h>
+#include <mach/3ds_debugboard.h>

 #include "devices-imx35.h"
 #include "devices.h"
@@ -86,6 +87,7 @@ static struct fsl_usb2_platform_data usb_pdata = {
 /*
  * Board specific initialization.
  */
+#define EXPIO_PARENT_INT 65
 static void __init mxc_board_init(void)
 {
        mxc_iomux_v3_setup_multiple_pads(mx35pdk_pads,
ARRAY_SIZE(mx35pdk_pads));
@@ -95,6 +97,10 @@ static void __init mxc_board_init(void)
        imx35_add_imx_uart0(&uart_pdata);

        mxc_register_device(&mxc_otg_udc_device, &usb_pdata);
+
+       if (!mxc_expio_init(MX35_CS5_BASE_ADDR, EXPIO_PARENT_INT))
+               printk(KERN_WARNING "Init of the debugboard failed, all "
+                       "devices on the board are unusable.\n");
 }

 static void __init mx35pdk_timer_init(void)

BRs,
Xiao Jiang

> Regards,
>
> Fabio Estevam
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20101014/b6f67d02/attachment-0001.html>

mx35pdk lan911x support

[Uwe Kleine-König]

Hello,

On Thu, Oct 14, 2010 at 09:05:39PM +0800, xiao jiang wrote:
> Hi,
> 2010/10/14 Fabio Estevam <festevam@gmail.com>
> 
> > Hi Xiao Jiang,
> >
> > 2010/10/14 Uwe Kleine-K?nig <u.kleine-koenig@pengutronix.de>:
> > > Hello Xiao (hoping this is the part of your name corresponding to your
> > > first name),
> > >
> > > On Thu, Oct 14, 2010 at 12:06:25AM -0400, xiao jiang wrote:
> > >> I saw the mach-mx35_3ds.c  are come from you, so  I think you are the
> > right
> > >> person to answer my queston, :).
> > > It was me, who renamed the file and touched it a few times when cleaning
> > > up and reorganising things for the i.MX machines.  Initially it was
> > > Fabio Estevam who brought the machine into mainline.
> >
> > I suggest you to look at MX51 3stack board or MX31 3stack boards, as
> > they have support for LAN9217 controller.
> >
> >
> Uwe,
> I forgot to subscribe the list, now it is ok after done it.:)
> 
> Fabio,
> Thanks for your hint, I can boot the kernel thru lan9217 now, only a few
> lines added.
> 
> diff --git a/arch/arm/mach-mx3/mach-mx35_3ds.c
> b/arch/arm/mach-mx3/mach-mx35_3ds.c
> index 1c30d72..7e30a4e 100644
> --- a/arch/arm/mach-mx3/mach-mx35_3ds.c
> +++ b/arch/arm/mach-mx3/mach-mx35_3ds.c
> @@ -35,6 +35,7 @@
>  #include <mach/hardware.h>
>  #include <mach/common.h>
>  #include <mach/iomux-mx35.h>
> +#include <mach/3ds_debugboard.h>
> 
>  #include "devices-imx35.h"
>  #include "devices.h"
> @@ -86,6 +87,7 @@ static struct fsl_usb2_platform_data usb_pdata = {
>  /*
>   * Board specific initialization.
>   */
> +#define EXPIO_PARENT_INT 65
>  static void __init mxc_board_init(void)
>  {
>         mxc_iomux_v3_setup_multiple_pads(mx35pdk_pads,
> ARRAY_SIZE(mx35pdk_pads));
> @@ -95,6 +97,10 @@ static void __init mxc_board_init(void)
>         imx35_add_imx_uart0(&uart_pdata);
> 
>         mxc_register_device(&mxc_otg_udc_device, &usb_pdata);
> +
> +       if (!mxc_expio_init(MX35_CS5_BASE_ADDR, EXPIO_PARENT_INT))
> +               printk(KERN_WARNING "Init of the debugboard failed, all "
> +                       "devices on the board are unusable.\n");
If you intend this patch to go mainline please use pr_warn instead of
printk(KERN_WARNING ...).

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-K?nig            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |

mx35pdk lan911x support

[xiao jiang]

Hello Uwe,
2010/10/14 Uwe Kleine-K?nig <u.kleine-koenig@pengutronix.de>

> Hello,
>
> On Thu, Oct 14, 2010 at 09:05:39PM +0800, xiao jiang wrote:
> > Hi,
> > 2010/10/14 Fabio Estevam <festevam@gmail.com>
> >
> > > Hi Xiao Jiang,
> > >
> > > 2010/10/14 Uwe Kleine-K?nig <u.kleine-koenig@pengutronix.de>:
> > > > Hello Xiao (hoping this is the part of your name corresponding to
> your
> > > > first name),
> > > >
> > > > On Thu, Oct 14, 2010 at 12:06:25AM -0400, xiao jiang wrote:
> > > >> I saw the mach-mx35_3ds.c  are come from you, so  I think you are
> the
> > > right
> > > >> person to answer my queston, :).
> > > > It was me, who renamed the file and touched it a few times when
> cleaning
> > > > up and reorganising things for the i.MX machines.  Initially it was
> > > > Fabio Estevam who brought the machine into mainline.
> > >
> > > I suggest you to look at MX51 3stack board or MX31 3stack boards, as
> > > they have support for LAN9217 controller.
> > >
> > >
> > Uwe,
> > I forgot to subscribe the list, now it is ok after done it.:)
> >
> > Fabio,
> > Thanks for your hint, I can boot the kernel thru lan9217 now, only a few
> > lines added.
> >
> > diff --git a/arch/arm/mach-mx3/mach-mx35_3ds.c
> > b/arch/arm/mach-mx3/mach-mx35_3ds.c
> > index 1c30d72..7e30a4e 100644
> > --- a/arch/arm/mach-mx3/mach-mx35_3ds.c
> > +++ b/arch/arm/mach-mx3/mach-mx35_3ds.c
> > @@ -35,6 +35,7 @@
> >  #include <mach/hardware.h>
> >  #include <mach/common.h>
> >  #include <mach/iomux-mx35.h>
> > +#include <mach/3ds_debugboard.h>
> >
> >  #include "devices-imx35.h"
> >  #include "devices.h"
> > @@ -86,6 +87,7 @@ static struct fsl_usb2_platform_data usb_pdata = {
> >  /*
> >   * Board specific initialization.
> >   */
> > +#define EXPIO_PARENT_INT 65
> >  static void __init mxc_board_init(void)
> >  {
> >         mxc_iomux_v3_setup_multiple_pads(mx35pdk_pads,
> > ARRAY_SIZE(mx35pdk_pads));
> > @@ -95,6 +97,10 @@ static void __init mxc_board_init(void)
> >         imx35_add_imx_uart0(&uart_pdata);
> >
> >         mxc_register_device(&mxc_otg_udc_device, &usb_pdata);
> > +
> > +       if (!mxc_expio_init(MX35_CS5_BASE_ADDR, EXPIO_PARENT_INT))
> > +               printk(KERN_WARNING "Init of the debugboard failed, all "
> > +                       "devices on the board are unusable.\n");
> If you intend this patch to go mainline please use pr_warn instead of
> printk(KERN_WARNING ...).
>
> Ok, but I don't want to use hard-code for EXPIO_PARENT_INT, and found there
is no related definition for MX35_GPIO1_1 pin, do I need to create it like
MX31_PIN_GPIO1_1?

Best Regards,
Xiao Jiang


> Best regards
> Uwe
>
> --
> Pengutronix e.K.                           | Uwe Kleine-K?nig            |
> Industrial Linux Solutions                 | http://www.pengutronix.de/  |
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20101015/c4c3ceb5/attachment-0001.html>