>-----Original Message----- >From: [email protected] >[mailto:[email protected]] On >Behalf Of David Brownell >Sent: Wednesday, September 02, 2009 9:58 AM >To: Barry Song >Cc: [email protected]; [email protected]; >[email protected]; >[email protected]; >[email protected]; [email protected] >Subject: Re: [Uclinux-dist-devel] [PATCH 1/2] add ad714x >platform_datadefinition > >On Monday 31 August 2009, Barry Song wrote: >> --- /dev/null >> +++ b/include/linux/spi/ad714x.h > >Doesn't belong under SPI since this is also used by the >I2C version of the chip. Likely include/linux/input/*h >is best.
I really wanted to place it to include/linux/input, but the include/linux/input doesn't exist and current input header files are placed in the parent dir "include/linux". > > >> @@ -0,0 +1,60 @@ >> +/* >> + * include/linux/ad714x.h > >Never include the directories. And the coding style says >this should have a twitter-sized description of the driver >in that line, like > > * ad714x.h -- platform data for capacitance sensor driver > >It's good policy to include a brief GPL statement at the top >of every source file too, including a Copyright (C) you. > > >> + * >> + * AD714x is very flexible, it can be used as buttons, scrollwheel, >> + * slider, touchpad at the same time. That depends on the boards. >> + * The platform_data for the device's "struct device" holds this >> + * information. >> + */ >> + >> + >> +#ifndef AD714X_HW_H >> +#define AD714X_HW_H > >As noted: __LINUX_INPUT_AD714X_H is the style to use here. > >_______________________________________________ >Uclinux-dist-devel mailing list >[email protected] >https://blackfin.uclinux.org/mailman/listinfo/uclinux-dist-devel > ------------------------------------------------------------------------------ Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day trial. Simplify your report design, integration and deployment - and focus on what you do best, core application coding. Discover what's new with Crystal Reports now. http://p.sf.net/sfu/bobj-july _______________________________________________ spi-devel-general mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/spi-devel-general
