It's probably better to start somewhere like drivers/staging for clean
up work.  Networking people are pretty busy with their own things but
staging is happy to take clean up patches.

You need to use a proper legal name (like you would for signing
documents for your From and Signed-off-by.

> > @@ -1444,7 +1444,7 @@ static int bind_mcastif_addr(struct socket *sock, 
> > struct net_device *dev)
> >     sin.sin_addr.s_addr  = addr;
> >     sin.sin_port         = 0;
> 
> I think you missed this one.
> should be
> -        sin.sin_port         = 0;
> +      sin.sin_port = 0

That was done deliberately.  Just leave that one as-is, please.

regards,
dan carpenter

Reply via email to