Re: [PATCH 2/2] mfd: add regulator cell to Cherry Trail Whiskey Cove PMIC

From: Andy Shevchenko
Date: Fri Oct 25 2019 - 04:07:01 EST


On Thu, Oct 24, 2019 at 02:29:39PM +0000, Andrey Zhizhikin wrote:
> Add a regulator mfd cell to Whiskey Cove PMIC driver, which is used to
> supply various voltage rails.
>
> In addition, make the initialization of this mfd driver early enough in
> order to provide regulator cell to mmc sub-system when it is initialized.

Doesn't deferred probe mechanism work for you?
MMC core returns that error till we have the driver initialized.

> + }, {
> + .name = "cht_wcove_regulator",
> + .id = CHT_WC_REGULATOR_VSDIO + 1,

> + .num_resources = 0,
> + .resources = NULL,

No need to put these.

> -static struct i2c_driver cht_wc_driver = {
> +static struct i2c_driver cht_wc_i2c_driver = {

Renaming is not explained in the commit message.

--
With Best Regards,
Andy Shevchenko