[Buildroot] [PATCH v2 2/2] board: Add a readme file for Toradex Colibri Vybrid module

[maitysanchayan at gmail.com]

Hello Thomas,

On 16-05-31 23:23:42, Thomas Petazzoni wrote:
> Hello,
> 
> On Sat, 12 Dec 2015 12:36:58 +0530, Sanchayan Maity wrote:
> > Add a readme file documenting how to use the build output images
> > using and cross referencing the update procedure as used by Toradex.
> 
> This patch should be squashed/merged into your previous patch. I.e, we
> want a single patch adding the defconfig/readme for the Toradex Colibri
> Vybrid module.
> 
> We also need your Signed-off-by here.

Will fix it.

> 
> > ---
> >  board/toradex/colibri-vf/readme.txt | 23 +++++++++++++++++++++++
> >  1 file changed, 23 insertions(+)
> >  create mode 100644 board/toradex/colibri-vf/readme.txt
> > 
> > diff --git a/board/toradex/colibri-vf/readme.txt b/board/toradex/colibri-vf/readme.txt
> > new file mode 100644
> > index 0000000..a403c03
> > --- /dev/null
> > +++ b/board/toradex/colibri-vf/readme.txt
> > @@ -0,0 +1,23 @@
> 
> Please add a short introduction paragraph here that indicates which HW
> platforms are supported, if possible with links to the official website
> for the platform.

Ok.

> 
> > +Build instructions
> > +------------------
> > +
> > +As a regular user configure and then build:
> > +
> > +$ make colibri_vf_defconfig
> > +$ make
> > +
> > +This configuration will use a linux kernel and uboot from the git trees
> > +maintained by Toradex and build a minimal root file system.
> > +
> > +Flashing the Vybrid module
> > +--------------------------
> > +
> > +Details can be found at:
> > +http://developer.toradex.com/knowledge-base/flashing-linux-on-vybrid-modules
> > +
> > +The above build configuration would result in building of rootfs.ubifs image
> 
> would result -> will result
> 
> > +and u-boot-nand.imx uboot binary. Prepare the SD card as per the instructions
> > +on the above article. Replace the ubifs.img file in SD card with rootfs.ubifs
> > +image file, rename it to ubifs.img and replace the u-boot-nand.imx uboot binary
> > +on the SD card with the one generated by buildroot.
> 
> It is really not great that you have to prepare a complete SD card with
> those weird instructions, and then replace some files with what
> Buildroot has produced. What is so special about this SD card that
> Buildroot cannot produce?

Will work on including our update process within this.

> 
> > Then follow the updation
> 
> updation -> update
> 
> > +procedure on the above article.
> 
> Could you fix the above issues, and send an updated version?

Yes, will try to address all the issues and send updated version.

Thanks for your feedback.

Regards,
Sanchayan.

> 
> Thanks a lot!
> 
> Thomas
> -- 
> Thomas Petazzoni, CTO, Free Electrons
> Embedded Linux, Kernel and Android engineering
> http://free-electrons.com