All of lore.kernel.org
 help / color / mirror / Atom feed
From: alexandre.belloni@free-electrons.com (Alexandre Belloni)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] ARM: multi_v7_defconfig: Add Atmel SAMA5 family
Date: Wed, 20 May 2015 23:22:24 +0200	[thread overview]
Message-ID: <20150520212224.GB3244@piout.net> (raw)
In-Reply-To: <CAOCHtYiYBsE=31E3ZNNnEkZrbFXmBmgwD8Qpt69iK6h6cW9shA@mail.gmail.com>

Hi,

On 20/05/2015 at 10:18:16 -0500, Robert Nelson wrote :
> On Wed, May 20, 2015 at 9:08 AM, Alexandre Belloni
> <alexandre.belloni@free-electrons.com> wrote:
> > Hi,
> >
> > On 20/05/2015 at 06:53:50 -0700, Kevin Hilman wrote :
> >> FYI... This patch is now in linux-next, and boot tests for all the
> >> sama5d[34] boards in kernelci.org have failed to boot.
> >>
> >>    http://kernelci.org/boot/all/job/next/kernel/next-20150520/
> >>
> >
> > Hum, I'm pretty sure you see the heartbeat led blinking ;)
> >
> > I think I told you a while ago on IRC and this is part of the commit
> > message. You have to use ttyAT0 instead of ttyS0 in your bootargs. I'm
> > not sure how I can make that more obvious, I'm open to any idea.
> >
> > I'm planning to send a series adding stdout-path to the atmel boards
> > when I'm back next week. Hopefully this will solve that issue.
> 
> Alexandre,
> 
> considering all the old boot-loaders, have you thought about adding a
> compatibility config similar to:
> 
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=00648d0282dcca1da453c0b20d27e24bf901a854
> 

We are not there yet. sama5_defconfig and at91_dt_defconfig will still
produce kernels using ttyS and not ttyAT. The switch will not happen
until you set CONFIG_SERIAL_ATMEL_TTYAT which has been present since
2.6.19. The issue is that it is mandatory to use it in a multi_v7
configuration because the 8250 driver is compiled in and is conflicting.


Kevin and Tyler jumped on the introduction of sama5 in multi_v7 and got
the issue. As long as you stick to the at91 only configurations, you
will be fine ;)
I hope this clarifies the thing :)

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

WARNING: multiple messages have this Message-ID (diff)
From: Alexandre Belloni <alexandre.belloni@free-electrons.com>
To: Robert Nelson <robertcnelson@gmail.com>
Cc: Kevin Hilman <khilman@kernel.org>,
	Nicolas Ferre <nicolas.ferre@atmel.com>,
	Arnd Bergmann <arnd@arndb.de>, Olof Johansson <olof@lixom.net>,
	Boris BREZILLON <boris.brezillon@free-electrons.com>,
	lkml <linux-kernel@vger.kernel.org>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH] ARM: multi_v7_defconfig: Add Atmel SAMA5 family
Date: Wed, 20 May 2015 23:22:24 +0200	[thread overview]
Message-ID: <20150520212224.GB3244@piout.net> (raw)
In-Reply-To: <CAOCHtYiYBsE=31E3ZNNnEkZrbFXmBmgwD8Qpt69iK6h6cW9shA@mail.gmail.com>

Hi,

On 20/05/2015 at 10:18:16 -0500, Robert Nelson wrote :
> On Wed, May 20, 2015 at 9:08 AM, Alexandre Belloni
> <alexandre.belloni@free-electrons.com> wrote:
> > Hi,
> >
> > On 20/05/2015 at 06:53:50 -0700, Kevin Hilman wrote :
> >> FYI... This patch is now in linux-next, and boot tests for all the
> >> sama5d[34] boards in kernelci.org have failed to boot.
> >>
> >>    http://kernelci.org/boot/all/job/next/kernel/next-20150520/
> >>
> >
> > Hum, I'm pretty sure you see the heartbeat led blinking ;)
> >
> > I think I told you a while ago on IRC and this is part of the commit
> > message. You have to use ttyAT0 instead of ttyS0 in your bootargs. I'm
> > not sure how I can make that more obvious, I'm open to any idea.
> >
> > I'm planning to send a series adding stdout-path to the atmel boards
> > when I'm back next week. Hopefully this will solve that issue.
> 
> Alexandre,
> 
> considering all the old boot-loaders, have you thought about adding a
> compatibility config similar to:
> 
> https://git.kernel.org/cgit/linux/kernel/git/torvalds/linux.git/commit/?id=00648d0282dcca1da453c0b20d27e24bf901a854
> 

We are not there yet. sama5_defconfig and at91_dt_defconfig will still
produce kernels using ttyS and not ttyAT. The switch will not happen
until you set CONFIG_SERIAL_ATMEL_TTYAT which has been present since
2.6.19. The issue is that it is mandatory to use it in a multi_v7
configuration because the 8250 driver is compiled in and is conflicting.


Kevin and Tyler jumped on the introduction of sama5 in multi_v7 and got
the issue. As long as you stick to the at91 only configurations, you
will be fine ;)
I hope this clarifies the thing :)

-- 
Alexandre Belloni, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com

  reply	other threads:[~2015-05-20 21:22 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2015-04-03  9:01 [PATCH] ARM: multi_v7_defconfig: Add Atmel SAMA5 family Alexandre Belloni
2015-04-03  9:01 ` Alexandre Belloni
2015-05-19  9:15 ` Nicolas Ferre
2015-05-19  9:15   ` Nicolas Ferre
2015-05-19 11:04   ` Arnd Bergmann
2015-05-19 11:04     ` Arnd Bergmann
2015-05-19 12:33     ` Alexandre Belloni
2015-05-19 12:33       ` Alexandre Belloni
2015-05-19 13:02       ` Nicolas Ferre
2015-05-19 13:02         ` Nicolas Ferre
2015-05-19 13:28         ` Nicolas Ferre
2015-05-19 13:28           ` Nicolas Ferre
2015-05-19 13:53           ` Nicolas Ferre
2015-05-19 13:53             ` Nicolas Ferre
2015-05-20 13:53             ` Kevin Hilman
2015-05-20 13:53               ` Kevin Hilman
2015-05-20 14:08               ` Alexandre Belloni
2015-05-20 14:08                 ` Alexandre Belloni
2015-05-20 14:59                 ` Kevin Hilman
2015-05-20 14:59                   ` Kevin Hilman
2015-05-20 21:29                   ` Alexandre Belloni
2015-05-20 21:29                     ` Alexandre Belloni
2015-05-20 15:18                 ` Robert Nelson
2015-05-20 15:18                   ` Robert Nelson
2015-05-20 21:22                   ` Alexandre Belloni [this message]
2015-05-20 21:22                     ` Alexandre Belloni

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=20150520212224.GB3244@piout.net \
    --to=alexandre.belloni@free-electrons.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.