Re: [thud][PATCH] cpsw9g-eth-fw: Add CPSW 9G support for J7

Re: [thud][PATCH] cpsw9g-eth-fw: Add CPSW 9G support for J7

[Denys Dmytriyenko]

Thanks, but please subscribe to the list before posting.


On Thu, Sep 12, 2019 at 10:04:05PM +0530, Santhana Bharathi wrote:
> Signed-off-by: Santhana Bharathi <santa@ti.com>
> ---
>  recipes-bsp/cpsw9g-eth-fw/cpsw9g-eth-fw_git.bb | 47 ++++++++++++++++++++++++++
>  1 file changed, 47 insertions(+)
>  create mode 100644 recipes-bsp/cpsw9g-eth-fw/cpsw9g-eth-fw_git.bb
> 
> diff --git a/recipes-bsp/cpsw9g-eth-fw/cpsw9g-eth-fw_git.bb b/recipes-bsp/cpsw9g-eth-fw/cpsw9g-eth-fw_git.bb
> new file mode 100644
> index 0000000..8677034
> --- /dev/null
> +++ b/recipes-bsp/cpsw9g-eth-fw/cpsw9g-eth-fw_git.bb
> @@ -0,0 +1,47 @@
> +Summary = "R5 PSDK CPSW9G ethernet switch firmware"

Uppercase


> +LICENSE = "TI-TFL"
> +LIC_FILES_CHKSUM = "file://LICENSE.ti;md5=04ad0a015d4bb63c2b9e7b112debf3db"
> +
> +PV = "1.0-git${SRCPV}"

1.0+git...


> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> +
> +inherit deploy
> +inherit update-alternatives
> +
> +PROTOCOL = "git"
> +BRANCH = "master"
> +SRCREV = "df7f399319ee401dfbda9317878128a0f718507f"
> +
> +SRC_URI = "git://git.ti.com/glsdk/ti-eth-fw.git;protocol=${PROTOCOL};branch=${BRANCH}"
> +
> +S = "${WORKDIR}/git"
> +
> +CPSW9G_FW_DIR = "${S}/ethfw"
> +CPSW9G_FW_FILENAME = "app_remoteswitchcfg_server.xer5f"
> +
> +# make sure that lib/firmware, and all its contents are part of the package
> +FILES_${PN} += "${base_libdir}/firmware"
> +
> +do_install() {
> +  install -d ${D}${base_libdir}/firmware
> +  install -m 0755 ${CPSW9G_FW_DIR}/${CPSW9G_FW_FILENAME} ${D}${base_libdir}/firmware
> +}
> +
> +TARGET_MAIN_R5FSS0_0 = "j7-main-r5f0_0-fw"
> +ALTERNATIVE_${PN} = "j7-main-r5f0_0-fw"
> +ALTERNATIVE_LINK_NAME[j7-main-r5f0_0-fw] = "${base_libdir}/firmware/${TARGET_MAIN_R5FSS0_0}"
> +ALTERNATIVE_TARGET[j7-main-r5f0_0-fw] = "${base_libdir}/firmware/${CPSW9G_FW_FILENAME}"
> +ALTERNATIVE_PRIORITY = "17"
> +
> +# This is used to prevent the build system to strip the executables
> +INHIBIT_PACKAGE_STRIP = "1"
> +# This is used to prevent the build system to split the debug info in a separate file
> +INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
> +# As it likely to be a different arch from the Yocto build, disable checking by adding "arch" to INSANE_SKIP
> +INSANE_SKIP_${PN} += "arch"

No need to comment each and every line here - those comments were provided in 
the example to explain what they do, but they don't need to be copied over and 
over in every new recipe...


> +# we don't want to configure and build the source code
> +do_compile[noexec] = "1"
> +do_configure[noexec] = "1"
> +
> -- 
> 1.9.1
> 

Re: [thud][PATCH] cpsw9g-eth-fw: Add CPSW 9G support for J7

[Denys Dmytriyenko]

On Thu, Sep 12, 2019 at 12:40:43PM -0400, Denys Dmytriyenko wrote:
> Thanks, but please subscribe to the list before posting.
> 
> 
> On Thu, Sep 12, 2019 at 10:04:05PM +0530, Santhana Bharathi wrote:
> > Signed-off-by: Santhana Bharathi <santa@ti.com>
> > ---
> >  recipes-bsp/cpsw9g-eth-fw/cpsw9g-eth-fw_git.bb | 47 ++++++++++++++++++++++++++
> >  1 file changed, 47 insertions(+)
> >  create mode 100644 recipes-bsp/cpsw9g-eth-fw/cpsw9g-eth-fw_git.bb
> > 
> > diff --git a/recipes-bsp/cpsw9g-eth-fw/cpsw9g-eth-fw_git.bb b/recipes-bsp/cpsw9g-eth-fw/cpsw9g-eth-fw_git.bb
> > new file mode 100644
> > index 0000000..8677034
> > --- /dev/null
> > +++ b/recipes-bsp/cpsw9g-eth-fw/cpsw9g-eth-fw_git.bb
> > @@ -0,0 +1,47 @@
> > +Summary = "R5 PSDK CPSW9G ethernet switch firmware"
> 
> Uppercase
> 
> 
> > +LICENSE = "TI-TFL"
> > +LIC_FILES_CHKSUM = "file://LICENSE.ti;md5=04ad0a015d4bb63c2b9e7b112debf3db"
> > +
> > +PV = "1.0-git${SRCPV}"
> 
> 1.0+git...
> 
> 
> > +PACKAGE_ARCH = "${MACHINE_ARCH}"
> > +
> > +inherit deploy
> > +inherit update-alternatives
> > +
> > +PROTOCOL = "git"
> > +BRANCH = "master"
> > +SRCREV = "df7f399319ee401dfbda9317878128a0f718507f"
> > +
> > +SRC_URI = "git://git.ti.com/glsdk/ti-eth-fw.git;protocol=${PROTOCOL};branch=${BRANCH}"
> > +
> > +S = "${WORKDIR}/git"
> > +
> > +CPSW9G_FW_DIR = "${S}/ethfw"
> > +CPSW9G_FW_FILENAME = "app_remoteswitchcfg_server.xer5f"
> > +
> > +# make sure that lib/firmware, and all its contents are part of the package
> > +FILES_${PN} += "${base_libdir}/firmware"
> > +
> > +do_install() {
> > +  install -d ${D}${base_libdir}/firmware
> > +  install -m 0755 ${CPSW9G_FW_DIR}/${CPSW9G_FW_FILENAME} ${D}${base_libdir}/firmware

Also, why is it setting 755 permissions?


> > +}
> > +
> > +TARGET_MAIN_R5FSS0_0 = "j7-main-r5f0_0-fw"
> > +ALTERNATIVE_${PN} = "j7-main-r5f0_0-fw"
> > +ALTERNATIVE_LINK_NAME[j7-main-r5f0_0-fw] = "${base_libdir}/firmware/${TARGET_MAIN_R5FSS0_0}"
> > +ALTERNATIVE_TARGET[j7-main-r5f0_0-fw] = "${base_libdir}/firmware/${CPSW9G_FW_FILENAME}"
> > +ALTERNATIVE_PRIORITY = "17"
> > +
> > +# This is used to prevent the build system to strip the executables
> > +INHIBIT_PACKAGE_STRIP = "1"
> > +# This is used to prevent the build system to split the debug info in a separate file
> > +INHIBIT_PACKAGE_DEBUG_SPLIT = "1"
> > +# As it likely to be a different arch from the Yocto build, disable checking by adding "arch" to INSANE_SKIP
> > +INSANE_SKIP_${PN} += "arch"
> 
> No need to comment each and every line here - those comments were provided in 
> the example to explain what they do, but they don't need to be copied over and 
> over in every new recipe...
> 
> 
> > +# we don't want to configure and build the source code
> > +do_compile[noexec] = "1"
> > +do_configure[noexec] = "1"
> > +
> > -- 
> > 1.9.1
> > 
> -- 
> _______________________________________________
> meta-ti mailing list
> meta-ti@yoctoproject.org
> https://lists.yoctoproject.org/listinfo/meta-ti