Subject: problem with function vlan_get_protocol()

From: "jiangtao.jit" <[email-masked]>
Date: Sun, 17 Jul 2011 12:53:15 +0800

Hi,all

in file   inclue/linux/if_vlan.h

there is a statement in function vlan_get_protocol()

	if (likely(protop))
		protocol = *protop;

do we need an else branch on pointer protop failed ?  through the else is unlikely 

	if (likely(protop))
		protocol = *protop;
    else
		protocol = proto;

is it necessary ?

thanks

2011-07-17 
jiangtao.jit