pio: check if return value of system_info_get_platform_bool is SYSTEM_INFO_ERROR_NONE... 37/155837/3 accepted/tizen/unified/20171017.071211 submit/tizen/20171016.114740
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:19 +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 a93bd546dda456ea680e480bc13f08cc7477b03c..408ed0a2fca352213c50ea89c96a35ef0f01c6be 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 dc3cbee8096585c4892235d414811c30f309a338..5d32bf3a6606e268e3a8b4c0483519794afc4b71 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 70c09927103a68dc1bb74289c673d82180e96165..fdd0de778349e6f1ca95f46b80609923ac373704 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 f5ac28e7079ad3937204095d68da101082fdc5dd..34e777c74956259fb0e14115ae8cb48c9dfad1ac 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 7c7d4daf97b1665e66a56a05f3b73a4e5fc00bef..10ef5613aa5f3d197203c17d700ff29ba61268e4 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);
        }