[yocto] [meta-raspberrypi][PATCH v2] rpi-config: fix invalid shell variable name

Andrea Galbusera gizero at gmail.com
Tue Apr 4 10:35:50 PDT 2017


Commit da32aac introduced an invalid shell variable name in do_deploy():
according to bash manpage variable names cannot contain dots. Replace
dot with underscore to fix it.

Signed-off-by: Andrea Galbusera <gizero at gmail.com>
---

v2: also update README accordingly
Signed-off-by: Andrea Galbusera <gizero at gmail.com>
---
 README                                  | 2 +-
 recipes-bsp/bootfiles/rpi-config_git.bb | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/README b/README
index c58bc99..9fdd5eb 100644
--- a/README
+++ b/README
@@ -247,7 +247,7 @@ List of currently supported models:
 If you would like to use the Waveshare "C" 1024×600, 7 inch Capacitive Touch
 Screen LCD, HDMI interface (http://www.waveshare.com/7inch-HDMI-LCD-C.htm)
 Rev 2.1, please set the following in your local.conf
-WAVESHARE_1024X600_C_2.1 = "1"
+WAVESHARE_1024X600_C_2_1 = "1"
 
 3.P. Enable UART
 ===============
diff --git a/recipes-bsp/bootfiles/rpi-config_git.bb b/recipes-bsp/bootfiles/rpi-config_git.bb
index 8adc938..af55983 100644
--- a/recipes-bsp/bootfiles/rpi-config_git.bb
+++ b/recipes-bsp/bootfiles/rpi-config_git.bb
@@ -112,7 +112,7 @@ do_deploy() {
     fi
 
     # Waveshare "C" 1024x600 7" Rev2.1 IPS capacitive touch (http://www.waveshare.com/7inch-HDMI-LCD-C.htm)
-    if [ "${WAVESHARE_1024X600_C_2.1}" = "1" ]; then
+    if [ "${WAVESHARE_1024X600_C_2_1}" = "1" ]; then
         echo "# Waveshare \"C\" 1024x600 7\" Rev2.1 IPS capacitive touch screen" >> ${DEPLOYDIR}/bcm2835-bootfiles/config.txt
         echo "max_usb_current=1" >> ${DEPLOYDIR}/bcm2835-bootfiles/config.txt
         echo "hdmi_group=2" >> ${DEPLOYDIR}/bcm2835-bootfiles/config.txt
-- 
2.7.4




More information about the yocto mailing list