Re: [PATCH v2] itg3200: add dt support.

From: NeilBrown
Date: Tue Nov 19 2013 - 00:49:19 EST


On Tue, 19 Nov 2013 02:49:38 +0100 Sebastian Reichel <sre@xxxxxxxxxx> wrote:

> Hi,
>
> On Tue, Nov 19, 2013 at 11:30:13AM +1100, NeilBrown wrote:
> > No new configuration, just a 'compatible' string and
> > documentation.
>
> itg3200 looks like a candidate for the list of trivial i2c
> devices [0] to me.
>
> [0] Documentation/devicetree/bindings/i2c/trivial-devices.txt
>
> -- Sebastian

Hmmm... a file that isn't referenced anywhere else in the kernel
documentation so is unlikely to be found except by people who know it is
there....

I can see that having lots of files for trivial devices is rather clumsy, but
there must be a better way.
I don't suppose we could just put the documentation in the device-driver file
in some format similar to kernel-doc and just extract it if it is wanted
separately? I think people read code a lot more than they read
documentation. I know I do.

But I can re-send with an update to trivial-devices.txt is that is what is
wanted.

Thanks,
NeilBrown

Attachment: signature.asc
Description: PGP signature