On Tue, May 10, 2011 at 3:35 PM, Ben Dooks <ben-i2c@xxxxxxxxx> wrote:
> On Sat, May 07, 2011 at 02:53:05PM +0300, Eran Duchan wrote:
>> +static int __devinit i2c_bit_platform_probe(struct platform_device *pdev)
>> +{
>> + struct i2c_algo_bit_data *bit_data;
>> + struct i2c_adapter *adap;
>> + int ret;
>> +
>> + if (pdev->dev.platform_data == NULL)
>> + ret = -ENXIO;
>
> minor point, this will probably not get shown by the bus layer.
> Not sure what's best here, some people don't like -ENOENT here.
I think ENXIO is fine though. Many drivers use it to indicate "I don't
have enough information to initialize this device".
Havard
--
To unsubscribe from this list: send the line "unsubscribe linux-i2c" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
