[meta-arago-distro][PATCH 1/2 v2] pinmux-utility: Add pinmux utility

[meta-arago-distro][PATCH 1/2 v2] pinmux-utility: Add pinmux utility

[Franklin S. Cooper Jr]

* Add Windows base pinmux utility
* Port pinmux-utility recipe from arago.
* The current recipe pulls in a zip file that doesn't contain
  license information. License information will be added in an
  updated recipe.

Signed-off-by: Franklin S. Cooper Jr <fcooper27jr@gmail.com>
---
 .../pinmux-utility/pinmux-utility.inc              |   19 +++++++++++++++
 .../pinmux-utility/pinmux-utility_2.3.1.0.bb       |   25 ++++++++++++++++++++
 2 files changed, 44 insertions(+), 0 deletions(-)
 create mode 100644 meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility.inc
 create mode 100644 meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility_2.3.1.0.bb

diff --git a/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility.inc b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility.inc
new file mode 100644
index 0000000..46d409d
--- /dev/null
+++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility.inc
@@ -0,0 +1,19 @@
+DESCRIPTION = "Texas Instruments Pinmux Utility"
+LICENSE = "BSD"
+
+PR = "r0"
+
+PACKAGE_ARCH = "${MACHINE_ARCH}"
+
+S = "${WORKDIR}/${P}"
+
+installdir = "host-tools/pinmux_utils/windows"
+
+do_install() {
+    install -d ${D}/${installdir}
+    install ${S}/Pin_Mux_Utility.msi ${D}/${installdir}
+    install ${S}/setup.exe ${D}/${installdir}
+    install ${S}/Release_Notes.txt ${D}/${installdir}
+}
+
+FILES_${PN} += "${installdir}/*"
diff --git a/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility_2.3.1.0.bb b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility_2.3.1.0.bb
new file mode 100644
index 0000000..e966f3a
--- /dev/null
+++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility_2.3.1.0.bb
@@ -0,0 +1,25 @@
+require pinmux-utility.inc
+
+PR_append = "-arago0"
+
+SRC_URI = "\
+    https://gforge.ti.com/gf/download/frsrelease/815/5471/PinMuxUtility_v2_03_01_00.zip;name=base;subdir=${P} \
+"
+SRC_URI[base.md5sum] = "31f28b06f72c042e5400fc53b4a29c0e"
+SRC_URI[base.sha256sum] = "2b0de3592c5f4b803a207d092dba227d10e1af65e0d4f06e1d8d23f317a452f9"
+
+# Currently for omapl138 devices we are using a different pinmux utility
+# program.  This requires a separate SRC_URI as well as a different
+# do_install.
+SRC_URI_omapl138 = "\
+    https://gforge.ti.com/gf/download/frsrelease/461/4210/Pin_Setup_AM18xx_01_00_1076_03.zip;name=omapl138;subdir=${P} \
+"
+
+do_install_omapl138() {
+    install -d ${D}/${installdir}
+    cp -rf ${S}/bin ${D}/${installdir}/
+    cp -rf ${S}/configurations ${D}/${installdir}/
+}
+
+SRC_URI[omapl138.md5sum] = "5a4eb834bde44c662aaf669882adbfe6"
+SRC_URI[omapl138.sha256sum] = "6701ca0e91761b9eb80b0097fbb2e2ce2cd8e0bcdc0bd18c34cd0221d9d450d0"
-- 
1.7.0.4

[meta-arago-distro][PATCH 2/2 v2] pinmux-utility: Add temporary license file

[Franklin S. Cooper Jr]

* Add temporary license file to the pinmux-utility. This file was
  grabbed from the license agreement screen when installing the non
  am18x version of the pinmux utility. A proper license file needs to
  be added to the zip file that is apart of the offical pinmux utility
  release.

Signed-off-by: Franklin S. Cooper Jr <fcooper27jr@gmail.com>
---
 .../pinmux-utility/pinmux-utility-2.3.1.0/LICENSE  |   30 ++++++++++++++++++++
 .../pinmux-utility/pinmux-utility_2.3.1.0.bb       |   13 ++++++++-
 2 files changed, 42 insertions(+), 1 deletions(-)
 create mode 100644 meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility-2.3.1.0/LICENSE

diff --git a/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility-2.3.1.0/LICENSE b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility-2.3.1.0/LICENSE
new file mode 100644
index 0000000..dcd680f
--- /dev/null
+++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility-2.3.1.0/LICENSE
@@ -0,0 +1,30 @@
+This distribution contains contributions or derivatives under copyright 
+as follows:
+Copyright (c) 2010, Texas Instruments Incorporated
+All rights reserved.
+Redistribution and use in source and binary forms, with or without 
+modification, are permitted provided that the following conditions are 
+met:
+- Redistributions of source code must retain the above copyright notice, 
+  this list of conditions and the following disclaimer.
+- Redistributions in binary form must reproduce the above copyright 
+  notice, this list of conditions and the following disclaimer in the 
+  documentation and/or other materials provided with the distribution.
+- Neither the name of Texas Instruments nor the names of its
+  contributors may be used to endorse or promote products derived 
+  from this software without specific prior written permission.
+THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS 
+AND CONTRIBUTORS "AS IS" AND ANY EXPRESS OR IMPLIED 
+WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED 
+WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A 
+PARTICULAR PURPOSE ARE DISCLAIMED.  
+IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS 
+BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, 
+EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT 
+NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR 
+SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS 
+INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF 
+LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT 
+(INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY 
+OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE 
+POSSIBILITY OF SUCH DAMAGE.
\ No newline at end of file
diff --git a/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility_2.3.1.0.bb b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility_2.3.1.0.bb
index e966f3a..fa20429 100644
--- a/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility_2.3.1.0.bb
+++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-utility_2.3.1.0.bb
@@ -1,6 +1,6 @@
 require pinmux-utility.inc
 
-PR_append = "-arago0"
+PR_append = "-arago1"
 
 SRC_URI = "\
     https://gforge.ti.com/gf/download/frsrelease/815/5471/PinMuxUtility_v2_03_01_00.zip;name=base;subdir=${P} \
@@ -23,3 +23,14 @@ do_install_omapl138() {
 
 SRC_URI[omapl138.md5sum] = "5a4eb834bde44c662aaf669882adbfe6"
 SRC_URI[omapl138.sha256sum] = "6701ca0e91761b9eb80b0097fbb2e2ce2cd8e0bcdc0bd18c34cd0221d9d450d0"
+
+# Temporarily add LICENSE file here since zip files don't contain any license information. This file was grabbed
+# from the License Agreement screen when installing the non am18x version of the pinmux utility.
+SRC_URI += "file://LICENSE"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=7f400ac8ac09799ccf153aab40deff94"
+
+do_unpack_extra() {
+    mv ${WORKDIR}/LICENSE ${WORKDIR}/${P}
+}
+
+addtask unpack_extra after do_unpack before do_patch
-- 
1.7.0.4

Re: [meta-arago-distro][PATCH 1/2 v2] pinmux-utility: Add pinmux utility

[Maupin, Chase]

Please start adding

---
Update in version x
	* item 1
	* item 2


That way we know why there is a new version.

Sincerely,
Chase Maupin
Software Applications
ARM MPU
e-mail: chase.maupin@ti.com
phone: (214) 567-2950

For support:
Forums - http://community.ti.com/forums/
Wiki - http://wiki.davincidsp.com/ 


> -----Original Message-----
> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> bounces@arago-project.org] On Behalf Of Franklin S. Cooper Jr
> Sent: Friday, September 07, 2012 2:10 PM
> To: meta-arago@arago-project.org
> Cc: Franklin S. Cooper Jr
> Subject: [meta-arago] [meta-arago-distro][PATCH 1/2 v2] pinmux-
> utility: Add pinmux utility
> 
> * Add Windows base pinmux utility
> * Port pinmux-utility recipe from arago.
> * The current recipe pulls in a zip file that doesn't contain
>   license information. License information will be added in an
>   updated recipe.
> 
> Signed-off-by: Franklin S. Cooper Jr <fcooper27jr@gmail.com>
> ---
>  .../pinmux-utility/pinmux-utility.inc              |   19
> +++++++++++++++
>  .../pinmux-utility/pinmux-utility_2.3.1.0.bb       |   25
> ++++++++++++++++++++
>  2 files changed, 44 insertions(+), 0 deletions(-)
>  create mode 100644 meta-arago-distro/recipes-arago/pinmux-
> utility/pinmux-utility.inc
>  create mode 100644 meta-arago-distro/recipes-arago/pinmux-
> utility/pinmux-utility_2.3.1.0.bb
> 
> diff --git a/meta-arago-distro/recipes-arago/pinmux-
> utility/pinmux-utility.inc b/meta-arago-distro/recipes-
> arago/pinmux-utility/pinmux-utility.inc
> new file mode 100644
> index 0000000..46d409d
> --- /dev/null
> +++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-
> utility.inc
> @@ -0,0 +1,19 @@
> +DESCRIPTION = "Texas Instruments Pinmux Utility"
> +LICENSE = "BSD"
> +
> +PR = "r0"
> +
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> +
> +S = "${WORKDIR}/${P}"
> +
> +installdir = "host-tools/pinmux_utils/windows"
> +
> +do_install() {
> +    install -d ${D}/${installdir}
> +    install ${S}/Pin_Mux_Utility.msi ${D}/${installdir}
> +    install ${S}/setup.exe ${D}/${installdir}
> +    install ${S}/Release_Notes.txt ${D}/${installdir}
> +}
> +
> +FILES_${PN} += "${installdir}/*"
> diff --git a/meta-arago-distro/recipes-arago/pinmux-
> utility/pinmux-utility_2.3.1.0.bb b/meta-arago-distro/recipes-
> arago/pinmux-utility/pinmux-utility_2.3.1.0.bb
> new file mode 100644
> index 0000000..e966f3a
> --- /dev/null
> +++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-
> utility_2.3.1.0.bb
> @@ -0,0 +1,25 @@
> +require pinmux-utility.inc
> +
> +PR_append = "-arago0"
> +
> +SRC_URI = "\
> +
> https://gforge.ti.com/gf/download/frsrelease/815/5471/PinMuxUtili
> ty_v2_03_01_00.zip;name=base;subdir=${P} \
> +"
> +SRC_URI[base.md5sum] = "31f28b06f72c042e5400fc53b4a29c0e"
> +SRC_URI[base.sha256sum] =
> "2b0de3592c5f4b803a207d092dba227d10e1af65e0d4f06e1d8d23f317a452f9
> "
> +
> +# Currently for omapl138 devices we are using a different pinmux
> utility
> +# program.  This requires a separate SRC_URI as well as a
> different
> +# do_install.
> +SRC_URI_omapl138 = "\
> +
> https://gforge.ti.com/gf/download/frsrelease/461/4210/Pin_Setup_A
> M18xx_01_00_1076_03.zip;name=omapl138;subdir=${P} \
> +"
> +
> +do_install_omapl138() {
> +    install -d ${D}/${installdir}
> +    cp -rf ${S}/bin ${D}/${installdir}/
> +    cp -rf ${S}/configurations ${D}/${installdir}/
> +}
> +
> +SRC_URI[omapl138.md5sum] = "5a4eb834bde44c662aaf669882adbfe6"
> +SRC_URI[omapl138.sha256sum] =
> "6701ca0e91761b9eb80b0097fbb2e2ce2cd8e0bcdc0bd18c34cd0221d9d450d0
> "
> --
> 1.7.0.4
> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [meta-arago-distro][PATCH 1/2 v2] pinmux-utility: Add pinmux utility

[Cooper Jr., Franklin]

I'm unsure about this. I would think that is the point of an app or library's release notes.

On Sep 8, 2012, at 10:52 AM, "Maupin, Chase" <chase.maupin@ti.com> wrote:

> Please start adding
> 
> ---
> Update in version x
>    * item 1
>    * item 2
> 
> 
> That way we know why there is a new version.
> 
> Sincerely,
> Chase Maupin
> Software Applications
> ARM MPU
> e-mail: chase.maupin@ti.com
> phone: (214) 567-2950
> 
> For support:
> Forums - http://community.ti.com/forums/
> Wiki - http://wiki.davincidsp.com/ 
> 
> 
>> -----Original Message-----
>> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
>> bounces@arago-project.org] On Behalf Of Franklin S. Cooper Jr
>> Sent: Friday, September 07, 2012 2:10 PM
>> To: meta-arago@arago-project.org
>> Cc: Franklin S. Cooper Jr
>> Subject: [meta-arago] [meta-arago-distro][PATCH 1/2 v2] pinmux-
>> utility: Add pinmux utility
>> 
>> * Add Windows base pinmux utility
>> * Port pinmux-utility recipe from arago.
>> * The current recipe pulls in a zip file that doesn't contain
>>  license information. License information will be added in an
>>  updated recipe.
>> 
>> Signed-off-by: Franklin S. Cooper Jr <fcooper27jr@gmail.com>
>> ---
>> .../pinmux-utility/pinmux-utility.inc              |   19
>> +++++++++++++++
>> .../pinmux-utility/pinmux-utility_2.3.1.0.bb       |   25
>> ++++++++++++++++++++
>> 2 files changed, 44 insertions(+), 0 deletions(-)
>> create mode 100644 meta-arago-distro/recipes-arago/pinmux-
>> utility/pinmux-utility.inc
>> create mode 100644 meta-arago-distro/recipes-arago/pinmux-
>> utility/pinmux-utility_2.3.1.0.bb
>> 
>> diff --git a/meta-arago-distro/recipes-arago/pinmux-
>> utility/pinmux-utility.inc b/meta-arago-distro/recipes-
>> arago/pinmux-utility/pinmux-utility.inc
>> new file mode 100644
>> index 0000000..46d409d
>> --- /dev/null
>> +++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-
>> utility.inc
>> @@ -0,0 +1,19 @@
>> +DESCRIPTION = "Texas Instruments Pinmux Utility"
>> +LICENSE = "BSD"
>> +
>> +PR = "r0"
>> +
>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>> +
>> +S = "${WORKDIR}/${P}"
>> +
>> +installdir = "host-tools/pinmux_utils/windows"
>> +
>> +do_install() {
>> +    install -d ${D}/${installdir}
>> +    install ${S}/Pin_Mux_Utility.msi ${D}/${installdir}
>> +    install ${S}/setup.exe ${D}/${installdir}
>> +    install ${S}/Release_Notes.txt ${D}/${installdir}
>> +}
>> +
>> +FILES_${PN} += "${installdir}/*"
>> diff --git a/meta-arago-distro/recipes-arago/pinmux-
>> utility/pinmux-utility_2.3.1.0.bb b/meta-arago-distro/recipes-
>> arago/pinmux-utility/pinmux-utility_2.3.1.0.bb
>> new file mode 100644
>> index 0000000..e966f3a
>> --- /dev/null
>> +++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-
>> utility_2.3.1.0.bb
>> @@ -0,0 +1,25 @@
>> +require pinmux-utility.inc
>> +
>> +PR_append = "-arago0"
>> +
>> +SRC_URI = "\
>> +
>> https://gforge.ti.com/gf/download/frsrelease/815/5471/PinMuxUtili
>> ty_v2_03_01_00.zip;name=base;subdir=${P} \
>> +"
>> +SRC_URI[base.md5sum] = "31f28b06f72c042e5400fc53b4a29c0e"
>> +SRC_URI[base.sha256sum] =
>> "2b0de3592c5f4b803a207d092dba227d10e1af65e0d4f06e1d8d23f317a452f9
>> "
>> +
>> +# Currently for omapl138 devices we are using a different pinmux
>> utility
>> +# program.  This requires a separate SRC_URI as well as a
>> different
>> +# do_install.
>> +SRC_URI_omapl138 = "\
>> +
>> https://gforge.ti.com/gf/download/frsrelease/461/4210/Pin_Setup_A
>> M18xx_01_00_1076_03.zip;name=omapl138;subdir=${P} \
>> +"
>> +
>> +do_install_omapl138() {
>> +    install -d ${D}/${installdir}
>> +    cp -rf ${S}/bin ${D}/${installdir}/
>> +    cp -rf ${S}/configurations ${D}/${installdir}/
>> +}
>> +
>> +SRC_URI[omapl138.md5sum] = "5a4eb834bde44c662aaf669882adbfe6"
>> +SRC_URI[omapl138.sha256sum] =
>> "6701ca0e91761b9eb80b0097fbb2e2ce2cd8e0bcdc0bd18c34cd0221d9d450d0
>> "
>> --
>> 1.7.0.4
>> 
>> _______________________________________________
>> meta-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [meta-arago-distro][PATCH 1/2 v2] pinmux-utility: Add pinmux utility

[Denys Dmytriyenko]

On Sat, Sep 08, 2012 at 04:16:42PM +0000, Cooper Jr., Franklin wrote:
> I'm unsure about this. I would think that is the point of an app or 
> library's release notes.

No, what Chase meant is for re-submition of the patches, it's common to 
explain what's different between v1 and v2 of the same patch. It's usually 
done after the --- line and before the actual diff, as that part is lost and 
won't go into the git commit description. See below...


> On Sep 8, 2012, at 10:52 AM, "Maupin, Chase" <chase.maupin@ti.com> wrote:
> 
> > Please start adding
> > 
> > ---
> > Update in version x
> >    * item 1
> >    * item 2
> > 
> > 
> > That way we know why there is a new version.
> > 
> >> -----Original Message-----
> >> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> >> bounces@arago-project.org] On Behalf Of Franklin S. Cooper Jr
> >> Sent: Friday, September 07, 2012 2:10 PM
> >> To: meta-arago@arago-project.org
> >> Cc: Franklin S. Cooper Jr
> >> Subject: [meta-arago] [meta-arago-distro][PATCH 1/2 v2] pinmux-
> >> utility: Add pinmux utility
> >> 
> >> * Add Windows base pinmux utility
> >> * Port pinmux-utility recipe from arago.
> >> * The current recipe pulls in a zip file that doesn't contain
> >>  license information. License information will be added in an
> >>  updated recipe.
> >> 
> >> Signed-off-by: Franklin S. Cooper Jr <fcooper27jr@gmail.com>
> >> ---

Here is the proper place for any comments you don't want to go into the commit 
description, such as the reason for resubmitting the patch. Like this:

* Version 2 of the patch addresses comments about blah.


> >> .../pinmux-utility/pinmux-utility.inc              |   19
> >> +++++++++++++++
> >> .../pinmux-utility/pinmux-utility_2.3.1.0.bb       |   25
> >> ++++++++++++++++++++
> >> 2 files changed, 44 insertions(+), 0 deletions(-)
> >> create mode 100644 meta-arago-distro/recipes-arago/pinmux-
> >> utility/pinmux-utility.inc
> >> create mode 100644 meta-arago-distro/recipes-arago/pinmux-
> >> utility/pinmux-utility_2.3.1.0.bb
> >> 
> >> diff --git a/meta-arago-distro/recipes-arago/pinmux-
> >> utility/pinmux-utility.inc b/meta-arago-distro/recipes-
> >> arago/pinmux-utility/pinmux-utility.inc
> >> new file mode 100644
> >> index 0000000..46d409d
> >> --- /dev/null
> >> +++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-
> >> utility.inc
> >> @@ -0,0 +1,19 @@
> >> +DESCRIPTION = "Texas Instruments Pinmux Utility"
> >> +LICENSE = "BSD"
> >> +
> >> +PR = "r0"
> >> +
> >> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> >> +
> >> +S = "${WORKDIR}/${P}"
> >> +
> >> +installdir = "host-tools/pinmux_utils/windows"
> >> +
> >> +do_install() {
> >> +    install -d ${D}/${installdir}
> >> +    install ${S}/Pin_Mux_Utility.msi ${D}/${installdir}
> >> +    install ${S}/setup.exe ${D}/${installdir}
> >> +    install ${S}/Release_Notes.txt ${D}/${installdir}
> >> +}
> >> +
> >> +FILES_${PN} += "${installdir}/*"
> >> diff --git a/meta-arago-distro/recipes-arago/pinmux-
> >> utility/pinmux-utility_2.3.1.0.bb b/meta-arago-distro/recipes-
> >> arago/pinmux-utility/pinmux-utility_2.3.1.0.bb
> >> new file mode 100644
> >> index 0000000..e966f3a
> >> --- /dev/null
> >> +++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-
> >> utility_2.3.1.0.bb
> >> @@ -0,0 +1,25 @@
> >> +require pinmux-utility.inc
> >> +
> >> +PR_append = "-arago0"
> >> +
> >> +SRC_URI = "\
> >> +
> >> https://gforge.ti.com/gf/download/frsrelease/815/5471/PinMuxUtili
> >> ty_v2_03_01_00.zip;name=base;subdir=${P} \
> >> +"
> >> +SRC_URI[base.md5sum] = "31f28b06f72c042e5400fc53b4a29c0e"
> >> +SRC_URI[base.sha256sum] =
> >> "2b0de3592c5f4b803a207d092dba227d10e1af65e0d4f06e1d8d23f317a452f9
> >> "
> >> +
> >> +# Currently for omapl138 devices we are using a different pinmux
> >> utility
> >> +# program.  This requires a separate SRC_URI as well as a
> >> different
> >> +# do_install.
> >> +SRC_URI_omapl138 = "\
> >> +
> >> https://gforge.ti.com/gf/download/frsrelease/461/4210/Pin_Setup_A
> >> M18xx_01_00_1076_03.zip;name=omapl138;subdir=${P} \
> >> +"
> >> +
> >> +do_install_omapl138() {
> >> +    install -d ${D}/${installdir}
> >> +    cp -rf ${S}/bin ${D}/${installdir}/
> >> +    cp -rf ${S}/configurations ${D}/${installdir}/
> >> +}
> >> +
> >> +SRC_URI[omapl138.md5sum] = "5a4eb834bde44c662aaf669882adbfe6"
> >> +SRC_URI[omapl138.sha256sum] =
> >> "6701ca0e91761b9eb80b0097fbb2e2ce2cd8e0bcdc0bd18c34cd0221d9d450d0
> >> "
> >> --
> >> 1.7.0.4
> >> 
> >> _______________________________________________
> >> meta-arago mailing list
> >> meta-arago@arago-project.org
> >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> 

Re: [meta-arago-distro][PATCH 1/2 v2] pinmux-utility: Add pinmux utility

[Maupin, Chase]

I meant a new version of your patch.

When you send a v2 or v3 it is a good idea to make a list of what you changed between versions.

Sincerely,
Chase Maupin
Software Applications
ARM MPU
e-mail: chase.maupin@ti.com
phone: (214) 567-2950

For support:
Forums - http://community.ti.com/forums/
Wiki - http://wiki.davincidsp.com/ 


> -----Original Message-----
> From: Cooper Jr., Franklin
> Sent: Saturday, September 08, 2012 11:17 AM
> To: Maupin, Chase
> Cc: Franklin S. Cooper Jr; meta-arago@arago-project.org; Franklin
> S. Cooper Jr
> Subject: Re: [meta-arago] [meta-arago-distro][PATCH 1/2 v2]
> pinmux-utility: Add pinmux utility
> 
> I'm unsure about this. I would think that is the point of an app
> or library's release notes.
> 
> On Sep 8, 2012, at 10:52 AM, "Maupin, Chase"
> <chase.maupin@ti.com> wrote:
> 
> > Please start adding
> >
> > ---
> > Update in version x
> >    * item 1
> >    * item 2
> >
> >
> > That way we know why there is a new version.
> >
> > Sincerely,
> > Chase Maupin
> > Software Applications
> > ARM MPU
> > e-mail: chase.maupin@ti.com
> > phone: (214) 567-2950
> >
> > For support:
> > Forums - http://community.ti.com/forums/
> > Wiki - http://wiki.davincidsp.com/
> >
> >
> >> -----Original Message-----
> >> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> >> bounces@arago-project.org] On Behalf Of Franklin S. Cooper Jr
> >> Sent: Friday, September 07, 2012 2:10 PM
> >> To: meta-arago@arago-project.org
> >> Cc: Franklin S. Cooper Jr
> >> Subject: [meta-arago] [meta-arago-distro][PATCH 1/2 v2]
> pinmux-
> >> utility: Add pinmux utility
> >>
> >> * Add Windows base pinmux utility
> >> * Port pinmux-utility recipe from arago.
> >> * The current recipe pulls in a zip file that doesn't contain
> >>  license information. License information will be added in an
> >>  updated recipe.
> >>
> >> Signed-off-by: Franklin S. Cooper Jr <fcooper27jr@gmail.com>
> >> ---
> >> .../pinmux-utility/pinmux-utility.inc              |   19
> >> +++++++++++++++
> >> .../pinmux-utility/pinmux-utility_2.3.1.0.bb       |   25
> >> ++++++++++++++++++++
> >> 2 files changed, 44 insertions(+), 0 deletions(-)
> >> create mode 100644 meta-arago-distro/recipes-arago/pinmux-
> >> utility/pinmux-utility.inc
> >> create mode 100644 meta-arago-distro/recipes-arago/pinmux-
> >> utility/pinmux-utility_2.3.1.0.bb
> >>
> >> diff --git a/meta-arago-distro/recipes-arago/pinmux-
> >> utility/pinmux-utility.inc b/meta-arago-distro/recipes-
> >> arago/pinmux-utility/pinmux-utility.inc
> >> new file mode 100644
> >> index 0000000..46d409d
> >> --- /dev/null
> >> +++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-
> >> utility.inc
> >> @@ -0,0 +1,19 @@
> >> +DESCRIPTION = "Texas Instruments Pinmux Utility"
> >> +LICENSE = "BSD"
> >> +
> >> +PR = "r0"
> >> +
> >> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> >> +
> >> +S = "${WORKDIR}/${P}"
> >> +
> >> +installdir = "host-tools/pinmux_utils/windows"
> >> +
> >> +do_install() {
> >> +    install -d ${D}/${installdir}
> >> +    install ${S}/Pin_Mux_Utility.msi ${D}/${installdir}
> >> +    install ${S}/setup.exe ${D}/${installdir}
> >> +    install ${S}/Release_Notes.txt ${D}/${installdir}
> >> +}
> >> +
> >> +FILES_${PN} += "${installdir}/*"
> >> diff --git a/meta-arago-distro/recipes-arago/pinmux-
> >> utility/pinmux-utility_2.3.1.0.bb b/meta-arago-distro/recipes-
> >> arago/pinmux-utility/pinmux-utility_2.3.1.0.bb
> >> new file mode 100644
> >> index 0000000..e966f3a
> >> --- /dev/null
> >> +++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-
> >> utility_2.3.1.0.bb
> >> @@ -0,0 +1,25 @@
> >> +require pinmux-utility.inc
> >> +
> >> +PR_append = "-arago0"
> >> +
> >> +SRC_URI = "\
> >> +
> >>
> https://gforge.ti.com/gf/download/frsrelease/815/5471/PinMuxUtili
> >> ty_v2_03_01_00.zip;name=base;subdir=${P} \
> >> +"
> >> +SRC_URI[base.md5sum] = "31f28b06f72c042e5400fc53b4a29c0e"
> >> +SRC_URI[base.sha256sum] =
> >>
> "2b0de3592c5f4b803a207d092dba227d10e1af65e0d4f06e1d8d23f317a452f9
> >> "
> >> +
> >> +# Currently for omapl138 devices we are using a different
> pinmux
> >> utility
> >> +# program.  This requires a separate SRC_URI as well as a
> >> different
> >> +# do_install.
> >> +SRC_URI_omapl138 = "\
> >> +
> >>
> https://gforge.ti.com/gf/download/frsrelease/461/4210/Pin_Setup_A
> >> M18xx_01_00_1076_03.zip;name=omapl138;subdir=${P} \
> >> +"
> >> +
> >> +do_install_omapl138() {
> >> +    install -d ${D}/${installdir}
> >> +    cp -rf ${S}/bin ${D}/${installdir}/
> >> +    cp -rf ${S}/configurations ${D}/${installdir}/
> >> +}
> >> +
> >> +SRC_URI[omapl138.md5sum] = "5a4eb834bde44c662aaf669882adbfe6"
> >> +SRC_URI[omapl138.sha256sum] =
> >>
> "6701ca0e91761b9eb80b0097fbb2e2ce2cd8e0bcdc0bd18c34cd0221d9d450d0
> >> "
> >> --
> >> 1.7.0.4
> >>
> >> _______________________________________________
> >> meta-arago mailing list
> >> meta-arago@arago-project.org
> >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> > _______________________________________________
> > meta-arago mailing list
> > meta-arago@arago-project.org
> > http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago

Re: [meta-arago-distro][PATCH 1/2 v2] pinmux-utility: Add pinmux utility

[Cooper Jr., Franklin]

That makes sense. Will do.

On Sep 8, 2012, at 12:24 PM, "Dmytriyenko, Denys" <denys@ti.com> wrote:

> On Sat, Sep 08, 2012 at 04:16:42PM +0000, Cooper Jr., Franklin wrote:
>> I'm unsure about this. I would think that is the point of an app or 
>> library's release notes.
> 
> No, what Chase meant is for re-submition of the patches, it's common to 
> explain what's different between v1 and v2 of the same patch. It's usually 
> done after the --- line and before the actual diff, as that part is lost and 
> won't go into the git commit description. See below...
> 
> 
>> On Sep 8, 2012, at 10:52 AM, "Maupin, Chase" <chase.maupin@ti.com> wrote:
>> 
>>> Please start adding
>>> 
>>> ---
>>> Update in version x
>>>   * item 1
>>>   * item 2
>>> 
>>> 
>>> That way we know why there is a new version.
>>> 
>>>> -----Original Message-----
>>>> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
>>>> bounces@arago-project.org] On Behalf Of Franklin S. Cooper Jr
>>>> Sent: Friday, September 07, 2012 2:10 PM
>>>> To: meta-arago@arago-project.org
>>>> Cc: Franklin S. Cooper Jr
>>>> Subject: [meta-arago] [meta-arago-distro][PATCH 1/2 v2] pinmux-
>>>> utility: Add pinmux utility
>>>> 
>>>> * Add Windows base pinmux utility
>>>> * Port pinmux-utility recipe from arago.
>>>> * The current recipe pulls in a zip file that doesn't contain
>>>> license information. License information will be added in an
>>>> updated recipe.
>>>> 
>>>> Signed-off-by: Franklin S. Cooper Jr <fcooper27jr@gmail.com>
>>>> ---
> 
> Here is the proper place for any comments you don't want to go into the commit 
> description, such as the reason for resubmitting the patch. Like this:
> 
> * Version 2 of the patch addresses comments about blah.
> 
> 
>>>> .../pinmux-utility/pinmux-utility.inc              |   19
>>>> +++++++++++++++
>>>> .../pinmux-utility/pinmux-utility_2.3.1.0.bb       |   25
>>>> ++++++++++++++++++++
>>>> 2 files changed, 44 insertions(+), 0 deletions(-)
>>>> create mode 100644 meta-arago-distro/recipes-arago/pinmux-
>>>> utility/pinmux-utility.inc
>>>> create mode 100644 meta-arago-distro/recipes-arago/pinmux-
>>>> utility/pinmux-utility_2.3.1.0.bb
>>>> 
>>>> diff --git a/meta-arago-distro/recipes-arago/pinmux-
>>>> utility/pinmux-utility.inc b/meta-arago-distro/recipes-
>>>> arago/pinmux-utility/pinmux-utility.inc
>>>> new file mode 100644
>>>> index 0000000..46d409d
>>>> --- /dev/null
>>>> +++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-
>>>> utility.inc
>>>> @@ -0,0 +1,19 @@
>>>> +DESCRIPTION = "Texas Instruments Pinmux Utility"
>>>> +LICENSE = "BSD"
>>>> +
>>>> +PR = "r0"
>>>> +
>>>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
>>>> +
>>>> +S = "${WORKDIR}/${P}"
>>>> +
>>>> +installdir = "host-tools/pinmux_utils/windows"
>>>> +
>>>> +do_install() {
>>>> +    install -d ${D}/${installdir}
>>>> +    install ${S}/Pin_Mux_Utility.msi ${D}/${installdir}
>>>> +    install ${S}/setup.exe ${D}/${installdir}
>>>> +    install ${S}/Release_Notes.txt ${D}/${installdir}
>>>> +}
>>>> +
>>>> +FILES_${PN} += "${installdir}/*"
>>>> diff --git a/meta-arago-distro/recipes-arago/pinmux-
>>>> utility/pinmux-utility_2.3.1.0.bb b/meta-arago-distro/recipes-
>>>> arago/pinmux-utility/pinmux-utility_2.3.1.0.bb
>>>> new file mode 100644
>>>> index 0000000..e966f3a
>>>> --- /dev/null
>>>> +++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-
>>>> utility_2.3.1.0.bb
>>>> @@ -0,0 +1,25 @@
>>>> +require pinmux-utility.inc
>>>> +
>>>> +PR_append = "-arago0"
>>>> +
>>>> +SRC_URI = "\
>>>> +
>>>> https://gforge.ti.com/gf/download/frsrelease/815/5471/PinMuxUtili
>>>> ty_v2_03_01_00.zip;name=base;subdir=${P} \
>>>> +"
>>>> +SRC_URI[base.md5sum] = "31f28b06f72c042e5400fc53b4a29c0e"
>>>> +SRC_URI[base.sha256sum] =
>>>> "2b0de3592c5f4b803a207d092dba227d10e1af65e0d4f06e1d8d23f317a452f9
>>>> "
>>>> +
>>>> +# Currently for omapl138 devices we are using a different pinmux
>>>> utility
>>>> +# program.  This requires a separate SRC_URI as well as a
>>>> different
>>>> +# do_install.
>>>> +SRC_URI_omapl138 = "\
>>>> +
>>>> https://gforge.ti.com/gf/download/frsrelease/461/4210/Pin_Setup_A
>>>> M18xx_01_00_1076_03.zip;name=omapl138;subdir=${P} \
>>>> +"
>>>> +
>>>> +do_install_omapl138() {
>>>> +    install -d ${D}/${installdir}
>>>> +    cp -rf ${S}/bin ${D}/${installdir}/
>>>> +    cp -rf ${S}/configurations ${D}/${installdir}/
>>>> +}
>>>> +
>>>> +SRC_URI[omapl138.md5sum] = "5a4eb834bde44c662aaf669882adbfe6"
>>>> +SRC_URI[omapl138.sha256sum] =
>>>> "6701ca0e91761b9eb80b0097fbb2e2ce2cd8e0bcdc0bd18c34cd0221d9d450d0
>>>> "
>>>> --
>>>> 1.7.0.4
>>>> 
>>>> _______________________________________________
>>>> meta-arago mailing list
>>>> meta-arago@arago-project.org
>>>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>>> _______________________________________________
>>> meta-arago mailing list
>>> meta-arago@arago-project.org
>>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>> _______________________________________________
>> meta-arago mailing list
>> meta-arago@arago-project.org
>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>> 

Re: [meta-arago-distro][PATCH 1/2 v2] pinmux-utility: Add pinmux utility

[Denys Dmytriyenko]

On Sat, Sep 08, 2012 at 05:27:39PM +0000, Cooper Jr., Franklin wrote:
> That makes sense. Will do.

BTW, some other things, see below...


> On Sep 8, 2012, at 12:24 PM, "Dmytriyenko, Denys" <denys@ti.com> wrote:
> 
> > On Sat, Sep 08, 2012 at 04:16:42PM +0000, Cooper Jr., Franklin wrote:
> >> I'm unsure about this. I would think that is the point of an app or 
> >> library's release notes.
> > 
> > No, what Chase meant is for re-submition of the patches, it's common to 
> > explain what's different between v1 and v2 of the same patch. It's usually 
> > done after the --- line and before the actual diff, as that part is lost and 
> > won't go into the git commit description. See below...
> > 
> > 
> >> On Sep 8, 2012, at 10:52 AM, "Maupin, Chase" <chase.maupin@ti.com> wrote:
> >> 
> >>> Please start adding
> >>> 
> >>> ---
> >>> Update in version x
> >>>   * item 1
> >>>   * item 2
> >>> 
> >>> 
> >>> That way we know why there is a new version.
> >>> 
> >>>> -----Original Message-----
> >>>> From: meta-arago-bounces@arago-project.org [mailto:meta-arago-
> >>>> bounces@arago-project.org] On Behalf Of Franklin S. Cooper Jr
> >>>> Sent: Friday, September 07, 2012 2:10 PM
> >>>> To: meta-arago@arago-project.org
> >>>> Cc: Franklin S. Cooper Jr
> >>>> Subject: [meta-arago] [meta-arago-distro][PATCH 1/2 v2] pinmux-
> >>>> utility: Add pinmux utility
> >>>> 
> >>>> * Add Windows base pinmux utility
> >>>> * Port pinmux-utility recipe from arago.
> >>>> * The current recipe pulls in a zip file that doesn't contain
> >>>> license information. License information will be added in an
> >>>> updated recipe.
> >>>> 
> >>>> Signed-off-by: Franklin S. Cooper Jr <fcooper27jr@gmail.com>

This gmail email is undeliverable for me. Moreover, you should probably sign 
off your changes with your TI address. Please do git-config in your 
repository, so your patches are generated with the correct From: and SOB: 
lines. You can still send them from gmail, though.


> >>>> ---
> > 
> > Here is the proper place for any comments you don't want to go into the commit 
> > description, such as the reason for resubmitting the patch. Like this:
> > 
> > * Version 2 of the patch addresses comments about blah.
> > 
> > 
> >>>> .../pinmux-utility/pinmux-utility.inc              |   19
> >>>> +++++++++++++++
> >>>> .../pinmux-utility/pinmux-utility_2.3.1.0.bb       |   25
> >>>> ++++++++++++++++++++
> >>>> 2 files changed, 44 insertions(+), 0 deletions(-)
> >>>> create mode 100644 meta-arago-distro/recipes-arago/pinmux-
> >>>> utility/pinmux-utility.inc
> >>>> create mode 100644 meta-arago-distro/recipes-arago/pinmux-
> >>>> utility/pinmux-utility_2.3.1.0.bb
> >>>> 
> >>>> diff --git a/meta-arago-distro/recipes-arago/pinmux-
> >>>> utility/pinmux-utility.inc b/meta-arago-distro/recipes-
> >>>> arago/pinmux-utility/pinmux-utility.inc
> >>>> new file mode 100644
> >>>> index 0000000..46d409d
> >>>> --- /dev/null
> >>>> +++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-
> >>>> utility.inc
> >>>> @@ -0,0 +1,19 @@
> >>>> +DESCRIPTION = "Texas Instruments Pinmux Utility"
> >>>> +LICENSE = "BSD"
> >>>> +
> >>>> +PR = "r0"
> >>>> +
> >>>> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> >>>> +
> >>>> +S = "${WORKDIR}/${P}"
> >>>> +
> >>>> +installdir = "host-tools/pinmux_utils/windows"
> >>>> +
> >>>> +do_install() {
> >>>> +    install -d ${D}/${installdir}
> >>>> +    install ${S}/Pin_Mux_Utility.msi ${D}/${installdir}
> >>>> +    install ${S}/setup.exe ${D}/${installdir}
> >>>> +    install ${S}/Release_Notes.txt ${D}/${installdir}
> >>>> +}
> >>>> +
> >>>> +FILES_${PN} += "${installdir}/*"
> >>>> diff --git a/meta-arago-distro/recipes-arago/pinmux-
> >>>> utility/pinmux-utility_2.3.1.0.bb b/meta-arago-distro/recipes-
> >>>> arago/pinmux-utility/pinmux-utility_2.3.1.0.bb
> >>>> new file mode 100644
> >>>> index 0000000..e966f3a
> >>>> --- /dev/null
> >>>> +++ b/meta-arago-distro/recipes-arago/pinmux-utility/pinmux-
> >>>> utility_2.3.1.0.bb
> >>>> @@ -0,0 +1,25 @@
> >>>> +require pinmux-utility.inc
> >>>> +
> >>>> +PR_append = "-arago0"
> >>>> +
> >>>> +SRC_URI = "\
> >>>> +
> >>>> https://gforge.ti.com/gf/download/frsrelease/815/5471/PinMuxUtili
> >>>> ty_v2_03_01_00.zip;name=base;subdir=${P} \
> >>>> +"
> >>>> +SRC_URI[base.md5sum] = "31f28b06f72c042e5400fc53b4a29c0e"
> >>>> +SRC_URI[base.sha256sum] =
> >>>> "2b0de3592c5f4b803a207d092dba227d10e1af65e0d4f06e1d8d23f317a452f9
> >>>> "
> >>>> +
> >>>> +# Currently for omapl138 devices we are using a different pinmux
> >>>> utility
> >>>> +# program.  This requires a separate SRC_URI as well as a
> >>>> different
> >>>> +# do_install.
> >>>> +SRC_URI_omapl138 = "\
> >>>> +
> >>>> https://gforge.ti.com/gf/download/frsrelease/461/4210/Pin_Setup_A
> >>>> M18xx_01_00_1076_03.zip;name=omapl138;subdir=${P} \
> >>>> +"
> >>>> +
> >>>> +do_install_omapl138() {
> >>>> +    install -d ${D}/${installdir}
> >>>> +    cp -rf ${S}/bin ${D}/${installdir}/
> >>>> +    cp -rf ${S}/configurations ${D}/${installdir}/
> >>>> +}
> >>>> +
> >>>> +SRC_URI[omapl138.md5sum] = "5a4eb834bde44c662aaf669882adbfe6"
> >>>> +SRC_URI[omapl138.sha256sum] =
> >>>> "6701ca0e91761b9eb80b0097fbb2e2ce2cd8e0bcdc0bd18c34cd0221d9d450d0
> >>>> "
> >>>> --
> >>>> 1.7.0.4
> >>>> 
> >>>> _______________________________________________
> >>>> meta-arago mailing list
> >>>> meta-arago@arago-project.org
> >>>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> >>> _______________________________________________
> >>> meta-arago mailing list
> >>> meta-arago@arago-project.org
> >>> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> >> _______________________________________________
> >> meta-arago mailing list
> >> meta-arago@arago-project.org
> >> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
> >> 
> _______________________________________________
> meta-arago mailing list
> meta-arago@arago-project.org
> http://arago-project.org/cgi-bin/mailman/listinfo/meta-arago
>