On Wed, Feb 06, 2019 at 09:31:27AM -0800, Stephen Boyd wrote: > Quoting Matti Vaittinen (2019-02-06 01:39:38) > > @@ -86,9 +87,21 @@ static int bd71837_clk_probe(struct platform_device > > *pdev) > > dev_err(&pdev->dev, "No parent clk found\n"); > > return -EINVAL; > > } > > - > > - c->reg = BD718XX_REG_OUT32K; > > - c->mask = BD718XX_OUT32K_EN; > > + switch (mfd->chip_type) { > > + case ROHM_CHIP_TYPE_BD71837: > > + case ROHM_CHIP_TYPE_BD71847: > > + > > Nitpick: why the extra newline above? For no obvious reason. I'll fix this as I need to create new version anyways. I'll squash the first 3 patches and fix some issues pointed out by the 0-Day test suite.
Br, Matti Vaittinen -- Matti Vaittinen, Linux device drivers ROHM Semiconductors, Finland SWDC Kiviharjunlenkki 1E 90220 OULU FINLAND ~~~ "I don't think so," said Rene Descartes. Just then, he vanished ~~~