Hi Fabio Estevam,

On Wednesday 14 September 2011 06:09 PM, Fabio Estevam wrote:
On Wed, Sep 14, 2011 at 9:07 AM, Tushar Behera<tushar.beh...@linaro.org>  wrote:
...

Will this be better?

static void lcd_hv070wsa_set_power(struct plat_lcd_data *pd, \
                                        unsigned int power)
{
        int ret;
        unsigned long flag = power ? GPIOF_OUT_INIT_HIGH : \
                                        GPIOF_OUT_INIT_LOW;

        ret = gpio_request_one(EXYNOS4_GPE3(4), flag, "GPE3_4");

        if (ret)
                printk(KERN_ERR "Could not request gpio for LCD power");
}

Looks better. You can use pr_err instead of printk(KERN_ERR .

Thanks for your review comments. I will send the next version with these changes.
Regards,

Fabio Estevam


--
Tushar Behera

_______________________________________________
linaro-dev mailing list
linaro-dev@lists.linaro.org
http://lists.linaro.org/mailman/listinfo/linaro-dev

Reply via email to