On 09/24/2015 06:57 PM, Guenter Roeck wrote:
> On 09/24/2015 05:46 PM, York Sun wrote:
>> SI5338 is a programmable clock generator. It has 4 sets of inputs,
>> PLL, multisynth and dividers to make 4 outputs. This driver splits
>> them into multiple clocks to comply with common clock framework.
>>
>> See Documentation/devicetree/bindings/clock/silabs,si5338.txt for
>> details.
>>
<snip>

>> +static inline u8 si5338_reg_read(struct si5338_driver_data *drvdata, u16 
>> reg)
>> +{
>> +    u32 val;
>> +    int ret;
>> +
>> +    ret = regmap_read(drvdata->regmap, reg, &val);
>> +    if (ret) {
>> +            dev_err(&drvdata->client->dev,
>> +                    "unable to read from reg %02x: return %d\n", reg, ret);
>> +            return 0;
> 
> Why hide the error ?

I need to comb through all the return values as you suggested. Thanks.

York
--
To unsubscribe from this list: send the line "unsubscribe linux-clk" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to