[PATCH 4/4] staging: sm750fb: fix undeclared function

Giedrius Statkevičius giedrius.statkevicius at gmail.com
Sun Mar 8 13:02:45 UTC 2015


On 2015.03.08 14:31, Sudip Mukherjee wrote:
> kbuild test robot reported that for microblaze-allyesconfig
> chan_to_field() and lynxfb_ops_set_par() were not defined. These two
> functions were defined under CONFIG_PM, so for any archtecture if
> CONFIG_PM is not defined we will have this error.
> while moving the lynxfb_suspend() function some very obvious
> checkpatch errors were taken care of.

Such as? If you do you could also fix up the poor things with that
function too such as the comment style here (no space between ; and the
comment):

> +		if (info)
> +			fb_set_suspend(info, 1);/* 1 means do suspend*/
> +

or here (extra unneeded spaces):

> +
> +		/* set chip to sleep mode	*/
> +		if (share->suspend)
> +			(*share->suspend)(share);

-- 
Thanks,
Giedrius


More information about the devel mailing list