pio: check if return value of system_info_get_platform_bool is SYSTEM_INFO_ERROR_NONE... 41/155841/2 accepted/tizen/4.0/unified/20171017.212142 submit/tizen_4.0/20171016.114755 tizen_4.0.m2_release
authorkibak.yoon <kibak.yoon@samsung.com>
Mon, 16 Oct 2017 08:59:30 +0000 (17:59 +0900)
committerKibak Yoon <kibak.yoon@samsung.com>
Mon, 16 Oct 2017 09:52:00 +0000 (09:52 +0000)
Change-Id: I6515fa369ed5201af520e5956bd0a8716a0ee168
Signed-off-by: kibak.yoon <kibak.yoon@samsung.com>
src/peripheral_gpio.c
src/peripheral_i2c.c
src/peripheral_pwm.c
src/peripheral_spi.c
src/peripheral_uart.c

index a93bd54..408ed0a 100644 (file)
@@ -36,10 +36,13 @@ static int gpio_feature = GPIO_FEATURE_UNKNOWN;
 
 static bool __is_feature_supported()
 {
+       int ret = SYSTEM_INFO_ERROR_NONE;
        bool feature = false;
 
        if (gpio_feature == GPIO_FEATURE_UNKNOWN) {
-               system_info_get_platform_bool(PERIPHERAL_IO_GPIO_FEATURE, &feature);
+               ret = system_info_get_platform_bool(PERIPHERAL_IO_GPIO_FEATURE, &feature);
+               RETVM_IF(ret != SYSTEM_INFO_ERROR_NONE, false, "Failed to get system info");
+
                gpio_feature = (feature ? GPIO_FEATURE_TRUE : GPIO_FEATURE_FALSE);
        }
        return (gpio_feature == GPIO_FEATURE_TRUE ? true : false);
index dc3cbee..5d32bf3 100644 (file)
@@ -45,10 +45,13 @@ static int i2c_feature = I2C_FEATURE_UNKNOWN;
 
 static bool __is_feature_supported()
 {
+       int ret = SYSTEM_INFO_ERROR_NONE;
        bool feature = false;
 
        if (i2c_feature == I2C_FEATURE_UNKNOWN) {
-               system_info_get_platform_bool(PERIPHERAL_IO_I2C_FEATURE, &feature);
+               ret = system_info_get_platform_bool(PERIPHERAL_IO_I2C_FEATURE, &feature);
+               RETVM_IF(ret != SYSTEM_INFO_ERROR_NONE, false, "Failed to get system info");
+
                i2c_feature = (feature ? I2C_FEATURE_TRUE : I2C_FEATURE_FALSE);
        }
 
index 70c0992..fdd0de7 100644 (file)
@@ -35,10 +35,13 @@ static int pwm_feature = PWM_FEATURE_UNKNOWN;
 
 static bool __is_feature_supported()
 {
+       int ret = SYSTEM_INFO_ERROR_NONE;
        bool feature = false;
 
        if (pwm_feature == PWM_FEATURE_UNKNOWN) {
-               system_info_get_platform_bool(PERIPHERAL_IO_PWM_FEATURE, &feature);
+               ret = system_info_get_platform_bool(PERIPHERAL_IO_PWM_FEATURE, &feature);
+               RETVM_IF(ret != SYSTEM_INFO_ERROR_NONE, false, "Failed to get system info");
+
                pwm_feature = (feature ? PWM_FEATURE_TRUE : PWM_FEATURE_FALSE);
        }
 
index f5ac28e..34e777c 100644 (file)
@@ -36,10 +36,13 @@ static int spi_feature = SPI_FEATURE_UNKNOWN;
 
 static bool __is_feature_supported()
 {
+       int ret = SYSTEM_INFO_ERROR_NONE;
        bool feature = false;
 
        if (spi_feature == SPI_FEATURE_UNKNOWN) {
-               system_info_get_platform_bool(PERIPHERAL_IO_SPI_FEATURE, &feature);
+               ret = system_info_get_platform_bool(PERIPHERAL_IO_SPI_FEATURE, &feature);
+               RETVM_IF(ret != SYSTEM_INFO_ERROR_NONE, false, "Failed to get system info");
+
                spi_feature = (feature ? SPI_FEATURE_TRUE : SPI_FEATURE_FALSE);
        }
 
index 7c7d4da..10ef561 100644 (file)
@@ -35,10 +35,13 @@ static int uart_feature = UART_FEATURE_UNKNOWN;
 
 static bool __is_feature_supported()
 {
+       int ret = SYSTEM_INFO_ERROR_NONE;
        bool feature = false;
 
        if (uart_feature == UART_FEATURE_UNKNOWN) {
-               system_info_get_platform_bool(PERIPHERAL_IO_UART_FEATURE, &feature);
+               ret = system_info_get_platform_bool(PERIPHERAL_IO_UART_FEATURE, &feature);
+               RETVM_IF(ret != SYSTEM_INFO_ERROR_NONE, false, "Failed to get system info");
+
                uart_feature = (feature ? UART_FEATURE_TRUE : UART_FEATURE_FALSE);
        }