Re: [Qemu-devel] [PATCH] hw/net/e1000: Fix compiler warning

[Dmitry Fleytman]

Hi,

s/e1000/e1000e/ in commit message subject, except this looks ok.

Thanks for the patch,
Dmitry

Reviewed-by: Dmitry Fleytman <address@hidden>

> On 25 Jul 2016, at 06:48 AM, Changlong Xie <address@hidden> wrote:
> 
> slave:~/.xie/qemu-colo # gcc --version
> gcc (SUSE Linux) 4.3.4 [gcc-4_3-branch revision 152973]
> 
> slave:~/.xie/qemu-colo # make -j8
> CC    hw/net/e1000e_core.o
> hw/net/e1000e_core.c:56: warning: ‘e1000e_set_interrupt_cause’ declared 
> inline after being called
> hw/net/e1000e_core.c:56: warning: previous declaration of 
> ‘e1000e_set_interrupt_cause’ was here
> LINK  x86_64-softmmu/qemu-system-x86_64
> 
> Signed-off-by: Changlong Xie <address@hidden>
> ---
> hw/net/e1000e_core.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/hw/net/e1000e_core.c b/hw/net/e1000e_core.c
> index badb1fe..e0bd31c 100644
> --- a/hw/net/e1000e_core.c
> +++ b/hw/net/e1000e_core.c
> @@ -52,7 +52,7 @@
>                                      second according to spec 10.2.4.2 */
> #define E1000E_MAX_TX_FRAGS (64)
> 
> -static void
> +static inline void
> e1000e_set_interrupt_cause(E1000ECore *core, uint32_t val);
> 
> static inline void
> -- 
> 1.9.3
> 
> 
>