Re: [PATCH v10] Add Microchip MCP3422/3/4 high resolution ADC

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Otavio,

MCP3426/7/8 Are different beast, not so much different from
mcp3422/3/4. Actually I'm building a board to test out the code I'm
writing for MCP3426/7/8, and it should be ready in a few days.

After finalizing the code and the necessary QA, I will publish it on this ML.

Sincerly Angelo.


2013/11/20 Otavio Salvador <otavio@xxxxxxxxxxxxxxxx>:
> On Mon, Sep 2, 2013 at 11:00 AM, Angelo Compagnucci
> <angelo.compagnucci@xxxxxxxxx> wrote:
>> Kawabonga! It works! Really, but really (no joke here!) thank you
>> for your precious support and sorry for not being so smart to
>> understand what you were saying me. Sorry again for being so pedantic!
>>
>> I'm planning to write a tutorial based on my journey, something like
>> "Howto write your first IIO LDD", what do you think?
>>
>> Attached here, there is the latest version of the patch based on jour suggestions.
>> I think all of your concerns are solved now.
>>
>> Thank you again!
>>
>> Signed-off-by: Angelo Compagnucci <angelo.compagnucci@xxxxxxxxx>
> ...
>> +static const struct i2c_device_id mcp3422_id[] = {
>> +       { "mcp3422", 2 },
>> +       { "mcp3423", 3 },
>> +       { "mcp3424", 4 },
>> +       { }
>> +};
>> +MODULE_DEVICE_TABLE(i2c, mcp3422_id);
>> +
>> +#ifdef CONFIG_OF
>> +static const struct of_device_id mcp3422_of_match[] = {
>> +       { .compatible = "mcp3422" },
>> +       { }
>
> Shouldn't this be mcp,mcp3422?
>
> Another question, are you aware of need changes to use with mcp3428?
> Otherwise I will look at this.
>
> --
> Otavio Salvador                             O.S. Systems
> http://www.ossystems.com.br        http://code.ossystems.com.br
> Mobile: +55 (53) 9981-7854            Mobile: +1 (347) 903-9750



-- 
Profile: http://it.linkedin.com/in/compagnucciangelo
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux