On Fri, Dec 30, 2011 at 03:05:53PM +0900, Sangbeom Kim wrote:Ah, I knew there was a reason why I hadn't reviewed this! I wasexpecting a new version with these:> +static int s5m8767_reg_enable_suspend(struct regulator_dev *rdev)> +static int s5m8767_reg_disable_suspend(struct regulator_dev *rdev)removed. Otherwise this looks good.