Re: [PATCH 8/9] bq27x00: Add miscdevice for each battery with ioctl for reading registers

[Pali Rohár]

On Sunday 27 November 2011 11:54:16 Mark Brown wrote:
> On Sat, Nov 26, 2011 at 02:04:48AM +0400, Anton Vorontsov wrote:
> > What would be more practical, is to allow I2C core to provide
> > userspace interface even for already bound I2C devices.
> >
> > That could be some kind of CONFIG_I2C_UNSAFE_DEBUG: when
> > selected I2C core would allow access to all I2C devices. But still,
> > the niche for such a feature is tiny, so I doubt that it is worth
> > doing at all.
>
> I don't know if it's applicable here but if this is a device that has
> registers that fit within regmap then the regmap API provides a debugfs
> interface for dumping the register map as standard.

I do not know regmap api, but I need interface which is easy readable from C
or shell scripts and which can read some registers (numbers 0-N).

These patches I written for Nokia N900. Now I need interface available in
kernel 2.6.28, which is only version working with Maemo (default N900 system).
Of course upstreaming *all* N900 drivers is needed, because I belive that one
day I will download vanilla kernel and run it on N900.


Anton Vorontsov, is simple debugfs interface (files - or only one file - which
have names of register numbers) acceptable for upstreaming (instead this
miscdevice with ioctl)?

--
Pali Rohár
pali.rohar@xxxxxxxxx

Attachment: signature.asc
Description: This is a digitally signed message part.