On Mar 16, 2016 20:28, Daniel Baluta wrote: > On Wed, Mar 16, 2016 at 6:25 PM, Slawomir Stepien <sst@xxxxxxxxx> wrote: > > On Mar 16, 2016 13:30, Peter Meerwald-Stadler wrote: > >> On Wed, 16 Mar 2016, Slawomir Stepien wrote: > >> > >> > The following functionalities are supported: > >> > - write, read from volatile and non volatile memory > >> > - increase and decrease commands > >> > - read from status register > >> > - write and read to tcon register > >> > > >> > Datasheet: http://ww1.microchip.com/downloads/en/DeviceDoc/22059b.pdf > > > > Thank you for all your comments. > > > >> the driver naming is a mess: the subject says MCP414X, the file name is > >> mcp41xx, the DT compatible string says mcp4113x -- this does not match > > > > OK. I will change that to mcp414x in version 2. > > Filename shouldn't be generic (e.g ending with xx). It should be a > specific chip name > (a good candidate is the first in alphabetical order), because there > could be chips falling > in the same name category but with a different driver. OK got it. Please wait for the version 2. -- Slawomir Stepien -- 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
