From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tomi Valkeinen Date: Wed, 26 Jun 2013 13:36:27 +0000 Subject: Re: [PATCH v2] video: of_display_timing.h: Declare 'display_timing' Message-Id: <51CAEE5B.4020108@ti.com> MIME-Version: 1 Content-Type: multipart/mixed; boundary="----enig2QGIFDDPFPDKVXJUWNGOC" List-Id: References: <1372253665-23148-1-git-send-email-fabio.estevam@freescale.com> In-Reply-To: <1372253665-23148-1-git-send-email-fabio.estevam@freescale.com> To: linux-fbdev@vger.kernel.org ------enig2QGIFDDPFPDKVXJUWNGOC Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: quoted-printable On 26/06/13 16:34, Fabio Estevam wrote: > Commit ffa3fd21de ("videomode: implement public of_get_display_timing()= ") causes > the following build warning: >=20 > include/video/of_display_timing.h:18:10: warning: 'struct display_timin= g' declared inside parameter list [enabled by default] > include/video/of_display_timing.h:18:10: warning: its scope is only thi= s definition or declaration, which is probably not what you want [enabled= by default] >=20 > Declare 'display_timing' to avoid the build warning. >=20 > Signed-off-by: Fabio Estevam > --- > Changes since v1: > - Add 'struct display_timing;' instead of '#include