[PATCH V5 15/17] regulator: Regulator driver for the Mediatek DVFSRC

[Henry Chen]

Hi Mark,

Thank you for reviewing.

On Mon, 2020-09-14 at 13:34 +0100, Mark Brown wrote:
> On Mon, Sep 14, 2020 at 11:04:42AM +0800, Henry Chen wrote:
> 
> > +// SPDX-License-Identifier: GPL-2.0
> > +/*
> > + * Copyright (c) 2019 MediaTek Inc.
> > + */
> 
> Please make the entire comment a C++ one so things look more
> intentional, and are you sure about that copyright date?
Ok, it will be modified on next version.
> 
> > +static int dvfsrc_set_voltage_sel(struct regulator_dev *rdev,
> > +				  unsigned int selector)
> > +{
> > +	struct device *dvfsrc_dev = to_dvfsrc_dev(rdev);
> > +	int id = rdev_get_id(rdev);
> > +
> > +	switch (id) {
> > +	case DVFSRC_ID_VCORE:
> > +		mtk_dvfsrc_send_request(dvfsrc_dev,
> > +					MTK_DVFSRC_CMD_VCORE_REQUEST,
> > +					selector);
> > +	break;
> > +	case DVFSRC_ID_VSCP:
> > +		mtk_dvfsrc_send_request(dvfsrc_dev,
> > +					MTK_DVFSRC_CMD_VSCP_REQUEST,
> > +					selector);
> > +	break;
> 
> Just have two separate operations rather than the switch statement, and
> where you do have switch statements please follow the kernel coding
> style.

Ok, it will be modified on next version.