On 7/17/20 9:56 PM, David Miller wrote:
From: Murali Karicheri <m-karicheri2@xxxxxx>
Date: Fri, 17 Jul 2020 11:15:05 -0400
@@ -32,7 +33,9 @@ static int hsr_newlink(struct net *src_net, struct net_device *dev,
struct netlink_ext_ack *extack)
{
struct net_device *link[2];
- unsigned char multicast_spec, hsr_version;
+ unsigned char multicast_spec;
+ enum hsr_version proto_version;
+ u8 proto = HSR_PROTOCOL_HSR;
Please use reverse christmas tree ordering for local variables.
Ok.
Thank you.
--
Murali Karicheri
Texas Instruments