Re: [PATCH 2/3] w35und: merge wbhal_s.h and wbhal_f.h header files

From: Pekka Enberg
Date: Fri Jan 23 2009 - 09:22:46 EST


On Wed, 2009-01-21 at 09:45 -0800, Greg KH wrote:
> On Mon, Jan 12, 2009 at 06:03:13PM +0200, Pekka J Enberg wrote:
> > From: Pekka Enberg <penberg@xxxxxxxxxxxxxx>
> >
> > No need to keep function definitions separate from the rest so unify wbhal_s.h
> > and wbhal_f.h header files.
> >
> > Cc: Pavel Machek <pavel@xxxxxxx>
> > Signed-off-by: Pekka Enberg <penberg@xxxxxxxxxxxxxx>
>
> I get a build error with this patch applied:
>
> CC [M] drivers/staging/winbond/mds.o
> In file included from drivers/staging/winbond/mds.c:3:
> drivers/staging/winbond/mds_f.h:4:19: error: wbhal.h: No such file or directory
> In file included from drivers/staging/winbond/mds_f.h:5,
> from drivers/staging/winbond/mds.c:3:
> drivers/staging/winbond/core.h:23: error: field âsHwDataâ has incomplete type
> In file included from drivers/staging/winbond/mds.c:3:
> drivers/staging/winbond/mds_f.h:10: error: expected declaration specifiers or â...â before âPT02_DESCRIPTORâ
> drivers/staging/winbond/mds_f.h:11: error: expected declaration specifiers or â...â before âPDESCRIPTORâ
> drivers/staging/winbond/mds_f.h:19: error: expected declaration specifiers or â...â before âPDESCRIPTORâ
> drivers/staging/winbond/mds_f.h:20: error: expected declaration specifiers or â...â before âPDESCRIPTORâ
> In file included from drivers/staging/winbond/mds.c:4:
>
>
> And it goes on...
>
> I'll drop this one from my queue.

OK, sorry about that. I'll wait for the other patches to land in
mainline and respin that one. Thanks!

Pekka

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/