Re: Common code for TSC 2101 and 2102

[Tony Lindgren <tony@atomide.com>]

* David Brownell <david-b@pacbell.net> [070713 13:05]:
> On Friday 13 July 2007, andrzej zaborowski wrote:
> 
> > > do not add new typedefs
> > > #1796: FILE: include/linux/spi/tsc210x.h:54:
> > > +typedef void (*tsc210x_touch_t)(void *context, int touching);
> > >
> > > do not add new typedefs
> > > #1797: FILE: include/linux/spi/tsc210x.h:55:
> > > +typedef void (*tsc210x_coords_t)(void *context, int x, int y, int z1, int z2);
> > >
> > > do not add new typedefs
> > > #1798: FILE: include/linux/spi/tsc210x.h:56:
> > > +typedef void (*tsc210x_ports_t)(void *context, int bat[], int aux[]);
> > >
> > > do not add new typedefs
> > > #1799: FILE: include/linux/spi/tsc210x.h:57:
> > > +typedef void (*tsc210x_temp_t)(void *context, int temp);
> > 
> > I wonder what is used instead.
> 
> I think the "no new typedefs" rule gets relaxed for function
> call prototypes.  But to answer your question:  it's usually
> more clear to just put the function prototype into the relevant
> type declarator than to define a new type.

I'll be pushing Andrzej's patch today.

Tony