From: marek.vasut@gmail.com (Marek Vasut)
To: linux-arm-kernel@lists.infradead.org
Subject: Issues when running DT enabled kernel on iMX28 from linux-next
Date: Fri, 18 May 2012 13:21:01 +0200 [thread overview]
Message-ID: <201205181321.01434.marek.vasut@gmail.com> (raw)
In-Reply-To: <20120518010138.GA5427@S2101-09.ap.freescale.net>
Dear Shawn Guo,
> On Thu, May 17, 2012 at 04:03:43PM -0700, Subodh Nijsure wrote:
> > Should one be able to run top of the linux-next on MX28 EVK hardware
> > with DT support enabled?
>
> Yes, I'm running it well with next-20120516.
>
> > I have attached boot log I see when I try to do that.
>
> One thing from the boot log worth checking is
>
> [ 0.820000] stmp_reset_block(f5010000): module reset timeout
>
> My wild guess is stmp_reset_block call on mmc0 failed on your end.
>
> > Further if one enables, CONFIG_MTD_NAND_GPMI_NAND=y
> > CONFIG_MTD_NAND_PLATFORM=y code in
> > drivers/mtd/nand/gpmi-nand/gpmi-nand.c doesn't compile because of
> >
> > this code:
> > pinctrl = devm_pinctrl_get_select_default(&pdev->dev);
> > if (IS_ERR(pinctrl)) {
> >
> > ret = PTR_ERR(pinctrl);
> > goto exit_pin;
> >
> > }
>
> Thanks for reporting it. It's my bad. I thought I had mxs_defconfig
> build testing cover the change. Actually, the gpmi is not enabled in
> mxs_defconfig yet. I will send the following fix to Arnd and Olof soon.
>
> @@ -486,7 +486,7 @@ static int __devinit acquire_resources(struct
> gpmi_nand_data *this) if (ret)
> goto exit_dma_channels;
>
> - pinctrl = devm_pinctrl_get_select_default(&pdev->dev);
> + pinctrl = devm_pinctrl_get_select_default(&this->pdev->dev);
> if (IS_ERR(pinctrl)) {
> ret = PTR_ERR(pinctrl);
> goto exit_pin;
Rebase your uboot on top of git://git.denx.de/u-boot-arm.git -master . u-boot-
imx is missing a patch and won't be updated for next 3 weeks (maintainer is on
vacation). u-boot-arm has a bugfix for this issue
Best regards,
Marek Vasut
next prev parent reply other threads:[~2012-05-18 11:21 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-05-17 23:03 Issues when running DT enabled kernel on iMX28 from linux-next Subodh Nijsure
2012-05-18 1:01 ` Shawn Guo
2012-05-18 1:50 ` Fabio Estevam
2012-05-18 1:58 ` Shawn Guo
2012-05-18 2:00 ` Fabio Estevam
2012-05-18 2:33 ` Shawn Guo
2012-05-18 3:05 ` Fabio Estevam
2012-05-18 11:21 ` Marek Vasut [this message]
2012-05-18 22:52 ` Subodh Nijsure
2012-05-19 1:23 ` Marek Vasut
2012-05-19 12:34 ` Shawn Guo
2012-05-19 16:02 ` Marek Vasut
2012-05-20 2:00 ` Shawn Guo
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=201205181321.01434.marek.vasut@gmail.com \
--to=marek.vasut@gmail.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.