[PATCH v6 3/3] ARM: shmobile: lager: enable nfsroot in DTS

[Simon Horman]

On Mon, Aug 05, 2013 at 07:18:40PM -0700, Kuninori Morimoto wrote:
> 
> Hi Simon
> 
> > > > >  	chosen {
> > > > > -		bootargs = "console=ttySC6,115200 ignore_loglevel";
> > > > > +		bootargs = "console=ttySC6,115200 ignore_loglevel root=/dev/nfs ip=dhcp nfsroot=,rsize=4096,wsize=4096 rw";
> > > > 
> (snip)
> > The purpose of adding NFS information to bootargs in r8a7790-lager.dts is
> > to cause the lager board to use NFS root when booting from using the C
> > version of the board code which lives in board-lager.c. This is in contrast
> > to the DT reverence version of the board code, which uses DT more heavily,
> > does not initialise sh-eth because DT bindings are not available (as far as
> > I know) and is not effected by r8a7790-lager.dts.
> > 
> > Historically this kind of NFS information was added to the command line in
> > the defconfig.  However, as part of the long journey to consolidate (reduce
> > the number of) shmobile defconfigs it now goes in DT instead.
> 
> Thank you.
> I don't have any objection to bootargs having NFS information.
> My question was only for "rsize/wsize".
> Sorry for my confusion English.
> 
> > With regards to the "rsize/wsize" settings, I will remove them.
> > However, I do see them present in the following files and I believe
> > historically they were present in some of the shmobile defconfigs.
> > 
> > emev2-kzm9d-reference.dts
> > emev2-kzm9d.dts
> > r8a7740-armadillo800eva-reference.dts
> > r8a7740-armadillo800eva.dts
> 
> I see.
> I don't know why these defconfigs have "rsize/wsize".
> But, this information was needed on old kernel version somehow (I forgot when, which version),
> and (I guess) there is no such issue now.
> We can remove it ?

Yes, I think we can remove it.
Would you like to make some patches?