lkml.org 
[lkml]   [2012]   [Sep]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] arm: introduce a DTS for Xen unprivileged virtual machines
On Thu, 20 Sep 2012, Dave Martin wrote:
> On Thu, Sep 20, 2012 at 11:06:03AM +0100, Pawel Moll wrote:
> > Morning,
> >
> > On Wed, 2012-09-19 at 18:44 +0100, Stefano Stabellini wrote:
> > > +/dts-v1/;
> > > +
> > > +/include/ "skeleton.dtsi"
> >
> > Any particular reason to include skeleton? And I think it would be
> > better to use #address-cells = <2> and #size-cells = <2>, to be ready
> > for LPAE addresses...
> >
> > > +/ {
> > > + model = "XENVM-4.2";
> > > + compatible = "xen,xenvm-4.2", "arm,vexpress";
> > > + interrupt-parent = <&gic>;
> > > +
> > > + chosen {
> > > + bootargs = "earlyprintk console=hvc0 root=/dev/xvda init=/sbin/init";
> > > + };
>
> Are you sure this default command line is appropriate?
>
> I don't normally like to see default command lines unless they really
> add something which is crucial for the board to work.
>
> None of those args looks vital to me. They are configuration choices
> and should be left up to the user.

Yes, you are right. I'll trim it down to "console=hvc0 root=/dev/xvda".


\
 
 \ /
  Last update: 2012-09-20 14:41    [W:0.125 / U:0.388 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site