On Tue, 07 Feb 2017, Jose Abreu <Jose.Abreu@xxxxxxxxxxxx> wrote:
>> +bool drm_scdc_check_scrambling_status(struct i2c_adapter *adapter)
>> +{
>> + u8 status;
>> + int ret;
>> +
>> + ret = drm_scdc_readb(adapter, SCDC_SCRAMBLER_STATUS, &status);
>> + if (ret < 0) {
>> + DRM_ERROR("Failed to read scrambling status, error %d\n", ret);
>> + return false;
>> + }
>> +
>> + return status & SCDC_SCRAMBLING_STATUS;
>
> "return (status & SCDC_SCRAMBLING_STATUS) > 0;" ?
What's the point in that?
BR,
Jani.
--
Jani Nikula, Intel Open Source Technology Center
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx
