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

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

[Maupin, Chase]

>-----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?

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

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

[Maupin, Chase]

>-----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.

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

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

[Denys Dmytriyenko]

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
> 

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

[Denys Dmytriyenko]

On Wed, Sep 25, 2013 at 09:43:20AM -0400, Maupin, Chase wrote:
> >-----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?

Those lines only required if the EVM comes with a built-in LCD screen with 
touch panel (like AM37x or AM335x or AM18x). They overwrite the default 
settings of an external DVI/HDMI screen w/o touch panel (like OMAP4/5 or 
Beagles).


> >+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?
> 
> >+PPA_PROJ = "hyperseam"
> >--
> >1.8.1
> 

Re: [meta-ti patch v3 3/3] am43xx: Added the kernel recipe

[Maupin, Chase]

Same question as u-boot recipe about temporary.

>-----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 3/3] am43xx: Added the kernel recipe
>
>Signed-off-by: Deepak M <x0171695@ti.com>
>---
>v3:
>1. Changed the kernel recipe name to linux-am43xx
>2. Using the external source link
>
> recipes-kernel/linux/linux-am43xx/defconfig |  1 +
> recipes-kernel/linux/linux-am43xx_3.12.bb   | 31
>+++++++++++++++++++++++++++++
> 2 files changed, 32 insertions(+)
> create mode 100644 recipes-kernel/linux/linux-am43xx/defconfig
> create mode 100644 recipes-kernel/linux/linux-am43xx_3.12.bb
>
>diff --git a/recipes-kernel/linux/linux-am43xx/defconfig
>b/recipes-kernel/linux/linux-am43xx/defconfig
>new file mode 100644
>index 0000000..926b35a
>--- /dev/null
>+++ b/recipes-kernel/linux/linux-am43xx/defconfig
>@@ -0,0 +1 @@
>+use-kernel-config=omap2plus_defconfig
>diff --git a/recipes-kernel/linux/linux-am43xx_3.12.bb b/recipes-
>kernel/linux/linux-am43xx_3.12.bb
>new file mode 100644
>index 0000000..a4b0700
>--- /dev/null
>+++ b/recipes-kernel/linux/linux-am43xx_3.12.bb
>@@ -0,0 +1,31 @@
>+SECTION = "kernel"
>+DESCRIPTION = "Linux kernel for TI33x devices from PSP"
>+LICENSE = "GPLv2"
>+LIC_FILES_CHKSUM =
>"file://COPYING;md5=d7810fab7487fb0aad327b76f1be7cd7"
>+COMPATIBLE_MACHINE = "am437x-epos-evm"
>+
>+DEFAULT_PREFERENCE = "-1"
>+
>+inherit kernel
>+require recipes-kernel/linux/linux-dtb.inc
>+require setup-defconfig.inc
>+
>+KERNEL_EXTRA_ARGS += "LOADADDR=${UBOOT_ENTRYPOINT}"
>+
>+KERNEL_IMAGETYPE = "uImage"
>+KERNEL_DEVICETREE = "am43x-epos-evm.dtb"
>+
>+KERNEL_DTB = "arch/arm/boot/dts/am43x-epos-evm.dtb"
>+
>+
>+# The main PR is now using MACHINE_KERNEL_PR,
>+PR = "r1+gitr${SRCPV}"
>+
>+BRANCH   = "linux-3.12.y"
>+
>+SRC_URI = "git://git.ti.com/ti-linux-kernel/ti-linux-
>kernel.git;BRANCH=${BRANCH};protocol=ssh \
>+           file://defconfig \
>+"
>+
>+SRCREV = "272b98c6455f00884f0350f775c5342358ebb73f"
>+S = "${WORKDIR}/git"
>--
>1.8.1

Re: [meta-ti patch v3 3/3] am43xx: Added the kernel recipe

[Denys Dmytriyenko]

On Wed, Sep 25, 2013 at 06:09:50PM +0530, Deepak M wrote:
> Signed-off-by: Deepak M <x0171695@ti.com>
> ---
> v3:
> 1. Changed the kernel recipe name to linux-am43xx
> 2. Using the external source link
> 
>  recipes-kernel/linux/linux-am43xx/defconfig |  1 +
>  recipes-kernel/linux/linux-am43xx_3.12.bb   | 31 +++++++++++++++++++++++++++++
>  2 files changed, 32 insertions(+)
>  create mode 100644 recipes-kernel/linux/linux-am43xx/defconfig
>  create mode 100644 recipes-kernel/linux/linux-am43xx_3.12.bb

Again, 3.12 is not yet released, you can't call your recipe with 3.12.


> diff --git a/recipes-kernel/linux/linux-am43xx/defconfig b/recipes-kernel/linux/linux-am43xx/defconfig
> new file mode 100644
> index 0000000..926b35a
> --- /dev/null
> +++ b/recipes-kernel/linux/linux-am43xx/defconfig
> @@ -0,0 +1 @@
> +use-kernel-config=omap2plus_defconfig
> diff --git a/recipes-kernel/linux/linux-am43xx_3.12.bb b/recipes-kernel/linux/linux-am43xx_3.12.bb
> new file mode 100644
> index 0000000..a4b0700
> --- /dev/null
> +++ b/recipes-kernel/linux/linux-am43xx_3.12.bb
> @@ -0,0 +1,31 @@
> +SECTION = "kernel"
> +DESCRIPTION = "Linux kernel for TI33x devices from PSP"
> +LICENSE = "GPLv2"
> +LIC_FILES_CHKSUM = "file://COPYING;md5=d7810fab7487fb0aad327b76f1be7cd7"
> +COMPATIBLE_MACHINE = "am437x-epos-evm"
> +
> +DEFAULT_PREFERENCE = "-1"
> +
> +inherit kernel
> +require recipes-kernel/linux/linux-dtb.inc
> +require setup-defconfig.inc
> +
> +KERNEL_EXTRA_ARGS += "LOADADDR=${UBOOT_ENTRYPOINT}"
> +
> +KERNEL_IMAGETYPE = "uImage"
> +KERNEL_DEVICETREE = "am43x-epos-evm.dtb"
> +
> +KERNEL_DTB = "arch/arm/boot/dts/am43x-epos-evm.dtb"
> +
> +
> +# The main PR is now using MACHINE_KERNEL_PR,
> +PR = "r1+gitr${SRCPV}"
> +
> +BRANCH   = "linux-3.12.y"
> +
> +SRC_URI = "git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git;BRANCH=${BRANCH};protocol=ssh \
> +           file://defconfig \
> +"
> +
> +SRCREV = "272b98c6455f00884f0350f775c5342358ebb73f"
> +S = "${WORKDIR}/git"
> -- 
> 1.8.1
> 

Re: [meta-ti patch v3 3/3] am43xx: Added the kernel recipe

[Denys Dmytriyenko]

And BTW, you need to subscribe to meta-ti list in order to post to it...


On Thu, Sep 26, 2013 at 09:50:39AM -0400, Denys Dmytriyenko wrote:
> On Wed, Sep 25, 2013 at 06:09:50PM +0530, Deepak M wrote:
> > Signed-off-by: Deepak M <x0171695@ti.com>
> > ---
> > v3:
> > 1. Changed the kernel recipe name to linux-am43xx
> > 2. Using the external source link
> > 
> >  recipes-kernel/linux/linux-am43xx/defconfig |  1 +
> >  recipes-kernel/linux/linux-am43xx_3.12.bb   | 31 +++++++++++++++++++++++++++++
> >  2 files changed, 32 insertions(+)
> >  create mode 100644 recipes-kernel/linux/linux-am43xx/defconfig
> >  create mode 100644 recipes-kernel/linux/linux-am43xx_3.12.bb
> 
> Again, 3.12 is not yet released, you can't call your recipe with 3.12.
> 
> 
> > diff --git a/recipes-kernel/linux/linux-am43xx/defconfig b/recipes-kernel/linux/linux-am43xx/defconfig
> > new file mode 100644
> > index 0000000..926b35a
> > --- /dev/null
> > +++ b/recipes-kernel/linux/linux-am43xx/defconfig
> > @@ -0,0 +1 @@
> > +use-kernel-config=omap2plus_defconfig
> > diff --git a/recipes-kernel/linux/linux-am43xx_3.12.bb b/recipes-kernel/linux/linux-am43xx_3.12.bb
> > new file mode 100644
> > index 0000000..a4b0700
> > --- /dev/null
> > +++ b/recipes-kernel/linux/linux-am43xx_3.12.bb
> > @@ -0,0 +1,31 @@
> > +SECTION = "kernel"
> > +DESCRIPTION = "Linux kernel for TI33x devices from PSP"
> > +LICENSE = "GPLv2"
> > +LIC_FILES_CHKSUM = "file://COPYING;md5=d7810fab7487fb0aad327b76f1be7cd7"
> > +COMPATIBLE_MACHINE = "am437x-epos-evm"
> > +
> > +DEFAULT_PREFERENCE = "-1"
> > +
> > +inherit kernel
> > +require recipes-kernel/linux/linux-dtb.inc
> > +require setup-defconfig.inc
> > +
> > +KERNEL_EXTRA_ARGS += "LOADADDR=${UBOOT_ENTRYPOINT}"
> > +
> > +KERNEL_IMAGETYPE = "uImage"
> > +KERNEL_DEVICETREE = "am43x-epos-evm.dtb"
> > +
> > +KERNEL_DTB = "arch/arm/boot/dts/am43x-epos-evm.dtb"
> > +
> > +
> > +# The main PR is now using MACHINE_KERNEL_PR,
> > +PR = "r1+gitr${SRCPV}"
> > +
> > +BRANCH   = "linux-3.12.y"
> > +
> > +SRC_URI = "git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git;BRANCH=${BRANCH};protocol=ssh \
> > +           file://defconfig \
> > +"
> > +
> > +SRCREV = "272b98c6455f00884f0350f775c5342358ebb73f"
> > +S = "${WORKDIR}/git"
> > -- 
> > 1.8.1
> > 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti

Re: [meta-ti patch v3 3/3] am43xx: Added the kernel recipe

[Denys Dmytriyenko]

On Thu, Sep 26, 2013 at 09:50:39AM -0400, Denys Dmytriyenko wrote:
> On Wed, Sep 25, 2013 at 06:09:50PM +0530, Deepak M wrote:
> > Signed-off-by: Deepak M <x0171695@ti.com>
> > ---
> > v3:
> > 1. Changed the kernel recipe name to linux-am43xx
> > 2. Using the external source link
> > 
> >  recipes-kernel/linux/linux-am43xx/defconfig |  1 +
> >  recipes-kernel/linux/linux-am43xx_3.12.bb   | 31 +++++++++++++++++++++++++++++
> >  2 files changed, 32 insertions(+)
> >  create mode 100644 recipes-kernel/linux/linux-am43xx/defconfig
> >  create mode 100644 recipes-kernel/linux/linux-am43xx_3.12.bb
> 
> Again, 3.12 is not yet released, you can't call your recipe with 3.12.
> 
> 
> > diff --git a/recipes-kernel/linux/linux-am43xx/defconfig b/recipes-kernel/linux/linux-am43xx/defconfig
> > new file mode 100644
> > index 0000000..926b35a
> > --- /dev/null
> > +++ b/recipes-kernel/linux/linux-am43xx/defconfig
> > @@ -0,0 +1 @@
> > +use-kernel-config=omap2plus_defconfig
> > diff --git a/recipes-kernel/linux/linux-am43xx_3.12.bb b/recipes-kernel/linux/linux-am43xx_3.12.bb
> > new file mode 100644
> > index 0000000..a4b0700
> > --- /dev/null
> > +++ b/recipes-kernel/linux/linux-am43xx_3.12.bb
> > @@ -0,0 +1,31 @@
> > +SECTION = "kernel"
> > +DESCRIPTION = "Linux kernel for TI33x devices from PSP"
> > +LICENSE = "GPLv2"
> > +LIC_FILES_CHKSUM = "file://COPYING;md5=d7810fab7487fb0aad327b76f1be7cd7"
> > +COMPATIBLE_MACHINE = "am437x-epos-evm"
> > +
> > +DEFAULT_PREFERENCE = "-1"
> > +
> > +inherit kernel
> > +require recipes-kernel/linux/linux-dtb.inc
> > +require setup-defconfig.inc
> > +
> > +KERNEL_EXTRA_ARGS += "LOADADDR=${UBOOT_ENTRYPOINT}"
> > +
> > +KERNEL_IMAGETYPE = "uImage"
> > +KERNEL_DEVICETREE = "am43x-epos-evm.dtb"
> > +
> > +KERNEL_DTB = "arch/arm/boot/dts/am43x-epos-evm.dtb"
> > +
> > +
> > +# The main PR is now using MACHINE_KERNEL_PR,
> > +PR = "r1+gitr${SRCPV}"
> > +
> > +BRANCH   = "linux-3.12.y"
> > +
> > +SRC_URI = "git://git.ti.com/ti-linux-kernel/ti-linux-kernel.git;BRANCH=${BRANCH};protocol=ssh \

Any reason to use upstream linux-3.12.y branch instead of our integration 
ti-linux-3.12.y one? And we do have a recipe for that already:

http://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/tree/recipes-kernel/linux/linux-ti-staging_git.bb


> > +           file://defconfig \
> > +"
> > +
> > +SRCREV = "272b98c6455f00884f0350f775c5342358ebb73f"
> > +S = "${WORKDIR}/git"
> > -- 
> > 1.8.1
> > 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti

Re: [meta-ti patch v3 2/3] am43xx: Added the u-boot recipe

[Maupin, Chase]

>-----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 2/3] am43xx: Added the u-boot recipe
>
>Added the LIC_FILES_CHKSUM in the recipe as the default
>checksum was pointing to the COPYING file which is moved to the
>Licenses/gpl-2.0.txt in the current source.
>
>Signed-off-by: Deepak M <x0171695@ti.com>
>---
>v3:
>1. Changed the u-boot recipe name from u-boot-aegis to u-boot-
>am43xx.
>2. Using externel source link.
>
> recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb | 20
>++++++++++++++++++++
> 1 file changed, 20 insertions(+)
> create mode 100644 recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
>
>diff --git a/recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
>b/recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb

My assumption is this is a temporary recipe correct?  Once the unified u-boot recipe in is place you can then just append what you need on top right?

>new file mode 100644
>index 0000000..58eb99a
>--- /dev/null
>+++ b/recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
>@@ -0,0 +1,20 @@
>+require u-boot-ti.inc
>+
>+DESCRIPTION = "u-boot bootloader for ARM MPU devices"
>+LIC_FILES_CHKSUM = "file://Licenses/gpl-
>2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263"
>+COMPATIBLE_MACHINE = "am437x-epos-evm"
>+
>+DEFAULT_PREFERENCE = "-1"
>+
>+PV = "2013.10"
>+PR = "r0+gitr${SRCPV}"
>+
>+BRANCH = "v2013.10-rc2+"
>+
>+SRC_URI = "git://git.ti.com/~lokesh/ti-u-boot/u-boot-
>dev.git;protocol=git;branch=${BRANCH}"
>+
>+# Lock to the commit corresponding
>+SRCREV = "6514cf76a52ecce9b03ede098683a7eea97e2bce"
>+
>+# Set the name of the SPL that will built so that it is also
>packaged with u-boot.
>+SPL_BINARY = "MLO"
>--
>1.8.1

Re: [meta-ti patch v3 2/3] am43xx: Added the u-boot recipe

[Maupin, Chase]

>-----Original Message-----
>From: Muddegowda, Deepak
>Sent: Wednesday, September 25, 2013 8:46 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 2/3] am43xx: Added the u-boot
>recipe
>
>
>
>> -----Original Message-----
>> From: Maupin, Chase
>> Sent: Wednesday, September 25, 2013 7:14 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 2/3] am43xx: Added the u-boot
>recipe
>>
>> >-----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 2/3] am43xx: Added the u-boot recipe
>> >
>> >Added the LIC_FILES_CHKSUM in the recipe as the default
>checksum was
>> >pointing to the COPYING file which is moved to the
>Licenses/gpl-
>> 2.0.txt
>> >in the current source.
>> >
>> >Signed-off-by: Deepak M <x0171695@ti.com>
>> >---
>> >v3:
>> >1. Changed the u-boot recipe name from u-boot-aegis to u-boot-
>am43xx.
>> >2. Using externel source link.
>> >
>> > recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb | 20
>> >++++++++++++++++++++
>> > 1 file changed, 20 insertions(+)
>> > create mode 100644 recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
>> >
>> >diff --git a/recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
>> >b/recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
>>
>> My assumption is this is a temporary recipe correct?  Once the
>unified
>> u-boot recipe in is place you can then just append what you need
>on top
>> right?
>[Muddegowda, Deepak] Not exactly temporary, may change the SRC_URI
>in future that’s it.

OK.  Now my question is why?  If you are given a base u-boot recipe that supports am43xx devices why would you define your own instead of appending?

>>
>> >new file mode 100644
>> >index 0000000..58eb99a
>> >--- /dev/null
>> >+++ b/recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
>> >@@ -0,0 +1,20 @@
>> >+require u-boot-ti.inc
>> >+
>> >+DESCRIPTION = "u-boot bootloader for ARM MPU devices"
>> >+LIC_FILES_CHKSUM = "file://Licenses/gpl-
>> >2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263"
>> >+COMPATIBLE_MACHINE = "am437x-epos-evm"
>> >+
>> >+DEFAULT_PREFERENCE = "-1"
>> >+
>> >+PV = "2013.10"
>> >+PR = "r0+gitr${SRCPV}"
>> >+
>> >+BRANCH = "v2013.10-rc2+"
>> >+
>> >+SRC_URI = "git://git.ti.com/~lokesh/ti-u-boot/u-boot-
>> >dev.git;protocol=git;branch=${BRANCH}"
>> >+
>> >+# Lock to the commit corresponding
>> >+SRCREV = "6514cf76a52ecce9b03ede098683a7eea97e2bce"
>> >+
>> >+# Set the name of the SPL that will built so that it is also
>> >packaged with u-boot.
>> >+SPL_BINARY = "MLO"
>> >--
>> >1.8.1

Re: [meta-ti patch v3 2/3] am43xx: Added the u-boot recipe

[Denys Dmytriyenko]

On Thu, Sep 26, 2013 at 07:42:43AM -0400, Muddegowda, Deepak wrote:
> 
> 
> > -----Original Message-----
> > From: Maupin, Chase
> > Sent: Wednesday, September 25, 2013 7:18 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 2/3] am43xx: Added the u-boot recipe
> > 
> > >-----Original Message-----
> > >From: Muddegowda, Deepak
> > >Sent: Wednesday, September 25, 2013 8:46 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 2/3] am43xx: Added the u-boot recipe
> > >
> > >
> > >
> > >> -----Original Message-----
> > >> From: Maupin, Chase
> > >> Sent: Wednesday, September 25, 2013 7:14 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 2/3] am43xx: Added the u-boot
> > >recipe
> > >>
> > >> >-----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 2/3] am43xx: Added the u-boot recipe
> > >> >
> > >> >Added the LIC_FILES_CHKSUM in the recipe as the default
> > >checksum was
> > >> >pointing to the COPYING file which is moved to the
> > >Licenses/gpl-
> > >> 2.0.txt
> > >> >in the current source.
> > >> >
> > >> >Signed-off-by: Deepak M <x0171695@ti.com>
> > >> >---
> > >> >v3:
> > >> >1. Changed the u-boot recipe name from u-boot-aegis to u-boot-
> > >am43xx.
> > >> >2. Using externel source link.
> > >> >
> > >> > recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb | 20
> > >> >++++++++++++++++++++
> > >> > 1 file changed, 20 insertions(+)
> > >> > create mode 100644 recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
> > >> >
> > >> >diff --git a/recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
> > >> >b/recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
> > >>
> > >> My assumption is this is a temporary recipe correct?  Once the
> > >unified
> > >> u-boot recipe in is place you can then just append what you need
> > >on top
> > >> right?
> > >[Muddegowda, Deepak] Not exactly temporary, may change the SRC_URI in
> > >future that's it.
> > 
> > OK.  Now my question is why?  If you are given a base u-boot recipe
> > that supports am43xx devices why would you define your own instead of
> > appending?
> > 
> [Muddegowda, Deepak] But there was no recipe for the u-boot 2013.10. 

Here's one:
http://git.yoctoproject.org/cgit/cgit.cgi/meta-ti/tree/recipes-bsp/u-boot/u-boot-ti-staging_git.bb


> > >>
> > >> >new file mode 100644
> > >> >index 0000000..58eb99a
> > >> >--- /dev/null
> > >> >+++ b/recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
> > >> >@@ -0,0 +1,20 @@
> > >> >+require u-boot-ti.inc
> > >> >+
> > >> >+DESCRIPTION = "u-boot bootloader for ARM MPU devices"
> > >> >+LIC_FILES_CHKSUM = "file://Licenses/gpl-
> > >> >2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> > >> >+COMPATIBLE_MACHINE = "am437x-epos-evm"
> > >> >+
> > >> >+DEFAULT_PREFERENCE = "-1"
> > >> >+
> > >> >+PV = "2013.10"
> > >> >+PR = "r0+gitr${SRCPV}"
> > >> >+
> > >> >+BRANCH = "v2013.10-rc2+"
> > >> >+
> > >> >+SRC_URI = "git://git.ti.com/~lokesh/ti-u-boot/u-boot-
> > >> >dev.git;protocol=git;branch=${BRANCH}"
> > >> >+
> > >> >+# Lock to the commit corresponding
> > >> >+SRCREV = "6514cf76a52ecce9b03ede098683a7eea97e2bce"
> > >> >+
> > >> >+# Set the name of the SPL that will built so that it is also
> > >> >packaged with u-boot.
> > >> >+SPL_BINARY = "MLO"
> > >> >--
> > >> >1.8.1
> 

Re: [meta-ti patch v3 2/3] am43xx: Added the u-boot recipe

[Denys Dmytriyenko]

On Wed, Sep 25, 2013 at 06:09:49PM +0530, Deepak M wrote:
> Added the LIC_FILES_CHKSUM in the recipe as the default
> checksum was pointing to the COPYING file which is moved to the
> Licenses/gpl-2.0.txt in the current source.
> 
> Signed-off-by: Deepak M <x0171695@ti.com>
> ---
> v3:
> 1. Changed the u-boot recipe name from u-boot-aegis to u-boot-am43xx.
> 2. Using externel source link.
> 
>  recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb | 20 ++++++++++++++++++++
>  1 file changed, 20 insertions(+)
>  create mode 100644 recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
> 
> diff --git a/recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb b/recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
> new file mode 100644
> index 0000000..58eb99a
> --- /dev/null
> +++ b/recipes-bsp/u-boot/u-boot-am43xx_2013.10.bb
> @@ -0,0 +1,20 @@
> +require u-boot-ti.inc
> +
> +DESCRIPTION = "u-boot bootloader for ARM MPU devices"
> +LIC_FILES_CHKSUM = "file://Licenses/gpl-2.0.txt;md5=b234ee4d69f5fce4486a80fdaf4a4263"
> +COMPATIBLE_MACHINE = "am437x-epos-evm"
> +
> +DEFAULT_PREFERENCE = "-1"
> +
> +PV = "2013.10"

The version 2013.10 of u-boot hasn't been released yet, so you can't use that 
for PV.


> +PR = "r0+gitr${SRCPV}"
> +
> +BRANCH = "v2013.10-rc2+"
> +
> +SRC_URI = "git://git.ti.com/~lokesh/ti-u-boot/u-boot-dev.git;protocol=git;branch=${BRANCH}"
> +
> +# Lock to the commit corresponding
> +SRCREV = "6514cf76a52ecce9b03ede098683a7eea97e2bce"
> +
> +# Set the name of the SPL that will built so that it is also packaged with u-boot.
> +SPL_BINARY = "MLO"
> -- 
> 1.8.1
>