Re: [meta-ti patch v3 1/3] am43xx : Added machine config for am437x EPOS machine

[Denys Dmytriyenko <denys@ti.com>]

On Wed, Sep 25, 2013 at 09:45:42AM -0400, Maupin, Chase wrote:
> >-----Original Message-----
> >From: Muddegowda, Deepak
> >Sent: Wednesday, September 25, 2013 8:45 AM
> >To: Maupin, Chase; Srinivasan, Nageswari; Heroor, Siddharth;
> >Cooper Jr., Franklin; Dmytriyenko, Denys; Mankad, Maulik Ojas;
> >epos_dev@list.ti.com - Internal mailing list for EPOS dev team
> >(May contain non-TIers); meta-ti@yoctoproject.org
> >Subject: RE: [meta-ti patch v3 1/3] am43xx : Added machine config
> >for am437x EPOS machine
> >
> >
> >
> >> -----Original Message-----
> >> From: Maupin, Chase
> >> Sent: Wednesday, September 25, 2013 7:13 PM
> >> To: Muddegowda, Deepak; Srinivasan, Nageswari; Heroor,
> >Siddharth;
> >> Cooper Jr., Franklin; Dmytriyenko, Denys; Mankad, Maulik Ojas;
> >> epos_dev@list.ti.com - Internal mailing list for EPOS dev team
> >(May
> >> contain non-TIers); meta-ti@yoctoproject.org
> >> Subject: RE: [meta-ti patch v3 1/3] am43xx : Added machine
> >config for
> >> am437x EPOS machine
> >>
> >> >-----Original Message-----
> >> >From: Muddegowda, Deepak
> >> >Sent: Wednesday, September 25, 2013 7:40 AM
> >> >To: Srinivasan, Nageswari; Maupin, Chase; Heroor, Siddharth;
> >Cooper
> >> >Jr., Franklin; Dmytriyenko, Denys; Mankad, Maulik Ojas;
> >Muddegowda,
> >> >Deepak; epos_dev@list.ti.com - Internal mailing list for EPOS
> >dev team
> >> >(May contain non-TIers); meta- ti@yoctoproject.org
> >> >Subject: [meta-ti patch v3 1/3] am43xx : Added machine config
> >for
> >> >am437x EPOS machine
> >> >
> >> >As this belongs to the ti43x SOC_FAMILY inluding the ti43x`s
> >include.
> >> >
> >> >Signed-off-by: Deepak M <x0171695@ti.com>
> >> >---
> >> >v3 :
> >> >1. Changed the machine name to am437x-epos-evm 2. Changed the
> >> >UBOOT_MACHINE 3. Adding the meta-ti@yoctoproject.org list for
> >review.
> >> >
> >> > conf/machine/am437x-epos-evm.conf | 38
> >> >++++++++++++++++++++++++++++++++++++++
> >> > 1 file changed, 38 insertions(+)
> >> > create mode 100644 conf/machine/am437x-epos-evm.conf
> >> >
> >> >diff --git a/conf/machine/am437x-epos-evm.conf
> >> >b/conf/machine/am437x-epos-evm.conf
> >> >new file mode 100644
> >> >index 0000000..a9adf7e
> >> >--- /dev/null
> >> >+++ b/conf/machine/am437x-epos-evm.conf
> >> >@@ -0,0 +1,38 @@
> >> >+#@TYPE: Machine
> >> >+#@NAME: AM43xx EVM
> >> >+#@DESCRIPTION: Machine configuration for the TI AM43xx EVM
> >> >+
> >> >+require conf/machine/include/ti43x.inc
> >> >+
> >> >+# Use built-in LCD by default
> >> >+XSERVER += "xf86-input-tslib"
> >> >+GUI_MACHINE_CLASS = "smallscreen"
> >> >+MACHINE_FEATURES += "touchscreen"
> >>
> >> Denys, input on these lines above?
> >>
> >> >+
> >> >+IMAGE_FSTYPES += "ubi tar.gz"
> >> >+
> >> >+SERIAL_CONSOLE = "115200 ttyO0"
> >> >+
> >> >+# UBI information.  Note that this is board and kernel
> >specific.
> >> >Changes
> >> >+# in your kernel port may require changes in these variables.
> >> >For more
> >> >+# details about this board please see
> >> >+# http://processors.wiki.ti.com/index.php/UBIFS_Support
> >> >+
> >> >+# do ubiattach /dev/ubi_ctrl -m 7 -O 2048 # From dmesg:
> >> >+# UBI: smallest flash I/O unit:    2048
> >> >+# UBI: logical eraseblock size:    126976 bytes
> >> >+# from ubiattach stdout:
> >> >+# UBI device number 0, total 1988 LEBs MKUBIFS_ARGS = "-F -m
> >2048 -e
> >> >+126976 -c 3836"
> >> >+
> >> >+# do ubiattach /dev/ubi_ctrl -m 7 -O 2048 # from dmesg:
> >> >+# UBI: smallest flash I/O unit:    2048
> >> >+# UBI: physical eraseblock size:   131072 bytes (128 KiB)
> >> >+# UBI: sub-page size:              512
> >> >+# UBI: VID header offset:          2048 (aligned 2048)
> >> >+UBINIZE_ARGS = "-m 2048 -p 128KiB -s 512 -O 2048"
> >> >+
> >> >+UBOOT_MACHINE = "am43xx_evm"
> >>
> >> Maybe a silly question but why the change in UBOOT_MACHINE?
> >[Muddegowda, Deepak] Previously I was using haps config, so placed
> >it with the am43xx_evm :)
> 
> I guess my question is why the default am43xx_evm_config setting in the include file doesn't work.

And the default should work - am43xx_evm is not a proper target for u-boot 
configuration, am43xx_evm_config is.

> >>
> >> >+PPA_PROJ = "hyperseam"
> >> >--
> >> >1.8.1
>