[PATCH] staging:sm750fb:Fixed no space and indent warnings

Sudip Mukherjee sudipm.mukherjee at gmail.com
Sun Mar 15 09:57:44 UTC 2015


On Sun, Mar 15, 2015 at 01:24:24AM -0700, Ragavendra wrote:
> ohh ok, is that the only fix that needs to be added. I don't mind fixing
> that and sending if required.
> 
> Anyway that brace needs to be on the same line as per the coding standards
> otherwise we get "ERROR: that open brace { should be on the previous line"
> in checkpath and I planned to fix this eventually. So do you want me to do
> this instead of the first one. Please advice.
the code was:
    if (physicalID == 0x718)
    {
         chip = SM718;
    }
so after your patch it should not be:
	if (physicalID == 0x718)
		{
	     ^^^^^^^^			
		chip = SM718;
	}
i have marked where the problem is. you are giving an extra tab before
the opening brace and introducing new style warnings there.
it should be :
	if (physicalID == 0x718)
	{
		chip = SM718;
	}

regards
sudip

> 
> -- 
> Thanks & regards,
> Ragavendra BN
> 


More information about the devel mailing list