use of_property_read_u8() for u8 variables, also changed the return type to void as this function return type is nowhere used. Signed-off-by: Saurabh Sengar <saurabh.truth@xxxxxxxxx> --- drivers/thermal/samsung/exynos_tmu.c | 30 +++++++++++++----------------- 1 file changed, 13 insertions(+), 17 deletions(-) diff --git a/drivers/thermal/samsung/exynos_tmu.c b/drivers/thermal/samsung/exynos_tmu.c index fa61eff..b692c95 100644 --- a/drivers/thermal/samsung/exynos_tmu.c +++ b/drivers/thermal/samsung/exynos_tmu.c @@ -1128,20 +1128,16 @@ static int exynos_of_get_soc_type(struct device_node *np) return -EINVAL; } -static int exynos_of_sensor_conf(struct device_node *np, +static void exynos_of_sensor_conf(struct device_node *np, struct exynos_tmu_platform_data *pdata) { - u32 value; - int ret; - of_node_get(np); - ret = of_property_read_u32(np, "samsung,tmu_gain", &value); - pdata->gain = (u8)value; - of_property_read_u32(np, "samsung,tmu_reference_voltage", &value); - pdata->reference_voltage = (u8)value; - of_property_read_u32(np, "samsung,tmu_noise_cancel_mode", &value); - pdata->noise_cancel_mode = (u8)value; + of_property_read_u8(np, "samsung,tmu_gain", &pdata->gain); + of_property_read_u8(np, "samsung,tmu_reference_voltage", + &pdata->reference_voltage); + of_property_read_u8(np, "samsung,tmu_noise_cancel_mode", + &pdata->noise_cancel_mode); of_property_read_u32(np, "samsung,tmu_efuse_value", &pdata->efuse_value); @@ -1150,18 +1146,18 @@ static int exynos_of_sensor_conf(struct device_node *np, of_property_read_u32(np, "samsung,tmu_max_efuse_value", &pdata->max_efuse_value); - of_property_read_u32(np, "samsung,tmu_first_point_trim", &value); - pdata->first_point_trim = (u8)value; - of_property_read_u32(np, "samsung,tmu_second_point_trim", &value); - pdata->second_point_trim = (u8)value; - of_property_read_u32(np, "samsung,tmu_default_temp_offset", &value); - pdata->default_temp_offset = (u8)value; + of_property_read_u8(np, "samsung,tmu_first_point_trim", + &pdata->first_point_trim); + of_property_read_u8(np, "samsung,tmu_second_point_trim", + &pdata->second_point_trim); + of_property_read_u8(np, "samsung,tmu_default_temp_offset", + &pdata->default_temp_offset); of_property_read_u32(np, "samsung,tmu_cal_type", &pdata->cal_type); of_property_read_u32(np, "samsung,tmu_cal_mode", &pdata->cal_mode); of_node_put(np); - return 0; + return; } static int exynos_map_dt_data(struct platform_device *pdev) -- 1.9.1 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html