Following up with complaints on inconsistent indentation from
Yangtao Li, this fixes indentation inconsistency.
In principle, this tries to put arguments aligned to the left
including the first argument except for in the case where
the first argument is on the far-right side.
Signed-off-by: MyungJoo Ham <[email protected]>
---
drivers/devfreq/devfreq.c | 49 +++++++++++++++++++++++------------------------
1 file changed, 24 insertions(+), 25 deletions(-)
diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c
index 4af608a..428a1de 100644
--- a/drivers/devfreq/devfreq.c
+++ b/drivers/devfreq/devfreq.c
@@ -528,7 +528,7 @@ void devfreq_interval_update(struct devfreq *devfreq, unsigned int *delay)
mutex_lock(&devfreq->lock);
if (!devfreq->stop_polling)
queue_delayed_work(devfreq_wq, &devfreq->work,
- msecs_to_jiffies(devfreq->profile->polling_ms));
+ msecs_to_jiffies(devfreq->profile->polling_ms));
}
out:
mutex_unlock(&devfreq->lock);
@@ -537,7 +537,7 @@ EXPORT_SYMBOL(devfreq_interval_update);
/**
* devfreq_notifier_call() - Notify that the device frequency requirements
- * has been changed out of devfreq framework.
+ * has been changed out of devfreq framework.
* @nb: the notifier_block (supposed to be devfreq->nb)
* @type: not used
* @devp: not used
@@ -683,12 +683,11 @@ struct devfreq *devfreq_add_device(struct device *dev,
goto err_out;
}
- devfreq->trans_table =
- devm_kzalloc(&devfreq->dev,
- array3_size(sizeof(unsigned int),
- devfreq->profile->max_state,
- devfreq->profile->max_state),
- GFP_KERNEL);
+ devfreq->trans_table = devm_kzalloc(&devfreq->dev,
+ array3_size(sizeof(unsigned int),
+ devfreq->profile->max_state,
+ devfreq->profile->max_state),
+ GFP_KERNEL);
if (!devfreq->trans_table) {
mutex_unlock(&devfreq->lock);
err = -ENOMEM;
@@ -696,9 +695,9 @@ struct devfreq *devfreq_add_device(struct device *dev,
}
devfreq->time_in_state = devm_kcalloc(&devfreq->dev,
- devfreq->profile->max_state,
- sizeof(unsigned long),
- GFP_KERNEL);
+ devfreq->profile->max_state,
+ sizeof(unsigned long),
+ GFP_KERNEL);
if (!devfreq->time_in_state) {
mutex_unlock(&devfreq->lock);
err = -ENOMEM;
@@ -1184,7 +1183,7 @@ static ssize_t available_governors_show(struct device *d,
*/
if (df->governor->immutable) {
count = scnprintf(&buf[count], DEVFREQ_NAME_LEN,
- "%s ", df->governor_name);
+ "%s ", df->governor_name);
/*
* The devfreq device shows the registered governor except for
* immutable governors such as passive governor .
@@ -1497,8 +1496,8 @@ EXPORT_SYMBOL(devfreq_recommended_opp);
/**
* devfreq_register_opp_notifier() - Helper function to get devfreq notified
- * for any changes in the OPP availability
- * changes
+ * for any changes in the OPP availability
+ * changes
* @dev: The devfreq user device. (parent of devfreq)
* @devfreq: The devfreq object.
*/
@@ -1510,8 +1509,8 @@ EXPORT_SYMBOL(devfreq_register_opp_notifier);
/**
* devfreq_unregister_opp_notifier() - Helper function to stop getting devfreq
- * notified for any changes in the OPP
- * availability changes anymore.
+ * notified for any changes in the OPP
+ * availability changes anymore.
* @dev: The devfreq user device. (parent of devfreq)
* @devfreq: The devfreq object.
*
@@ -1530,8 +1529,8 @@ static void devm_devfreq_opp_release(struct device *dev, void *res)
}
/**
- * devm_ devfreq_register_opp_notifier()
- * - Resource-managed devfreq_register_opp_notifier()
+ * devm_devfreq_register_opp_notifier() - Resource-managed
+ * devfreq_register_opp_notifier()
* @dev: The devfreq user device. (parent of devfreq)
* @devfreq: The devfreq object.
*/
@@ -1559,8 +1558,8 @@ int devm_devfreq_register_opp_notifier(struct device *dev,
EXPORT_SYMBOL(devm_devfreq_register_opp_notifier);
/**
- * devm_devfreq_unregister_opp_notifier()
- * - Resource-managed devfreq_unregister_opp_notifier()
+ * devm_devfreq_unregister_opp_notifier() - Resource-managed
+ * devfreq_unregister_opp_notifier()
* @dev: The devfreq user device. (parent of devfreq)
* @devfreq: The devfreq object.
*/
@@ -1579,8 +1578,8 @@ EXPORT_SYMBOL(devm_devfreq_unregister_opp_notifier);
* @list: DEVFREQ_TRANSITION_NOTIFIER.
*/
int devfreq_register_notifier(struct devfreq *devfreq,
- struct notifier_block *nb,
- unsigned int list)
+ struct notifier_block *nb,
+ unsigned int list)
{
int ret = 0;
@@ -1686,9 +1685,9 @@ EXPORT_SYMBOL(devm_devfreq_register_notifier);
* @list: DEVFREQ_TRANSITION_NOTIFIER.
*/
void devm_devfreq_unregister_notifier(struct device *dev,
- struct devfreq *devfreq,
- struct notifier_block *nb,
- unsigned int list)
+ struct devfreq *devfreq,
+ struct notifier_block *nb,
+ unsigned int list)
{
WARN_ON(devres_release(dev, devm_devfreq_notifier_release,
devm_devfreq_dev_match, devfreq));
--
2.7.4
On 19. 1. 21. 오전 11:19, MyungJoo Ham wrote:
> Following up with complaints on inconsistent indentation from
> Yangtao Li, this fixes indentation inconsistency.
>
> In principle, this tries to put arguments aligned to the left
> including the first argument except for in the case where
> the first argument is on the far-right side.
>
> Signed-off-by: MyungJoo Ham <[email protected]>
> ---
> drivers/devfreq/devfreq.c | 49 +++++++++++++++++++++++------------------------
> 1 file changed, 24 insertions(+), 25 deletions(-)
Looks better before.
Reviewed-by: Chanwoo Choi <[email protected]>
>
> diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c
> index 4af608a..428a1de 100644
> --- a/drivers/devfreq/devfreq.c
> +++ b/drivers/devfreq/devfreq.c
> @@ -528,7 +528,7 @@ void devfreq_interval_update(struct devfreq *devfreq, unsigned int *delay)
> mutex_lock(&devfreq->lock);
> if (!devfreq->stop_polling)
> queue_delayed_work(devfreq_wq, &devfreq->work,
> - msecs_to_jiffies(devfreq->profile->polling_ms));
> + msecs_to_jiffies(devfreq->profile->polling_ms));
> }
> out:
> mutex_unlock(&devfreq->lock);
> @@ -537,7 +537,7 @@ EXPORT_SYMBOL(devfreq_interval_update);
>
> /**
> * devfreq_notifier_call() - Notify that the device frequency requirements
> - * has been changed out of devfreq framework.
> + * has been changed out of devfreq framework.
> * @nb: the notifier_block (supposed to be devfreq->nb)
> * @type: not used
> * @devp: not used
> @@ -683,12 +683,11 @@ struct devfreq *devfreq_add_device(struct device *dev,
> goto err_out;
> }
>
> - devfreq->trans_table =
> - devm_kzalloc(&devfreq->dev,
> - array3_size(sizeof(unsigned int),
> - devfreq->profile->max_state,
> - devfreq->profile->max_state),
> - GFP_KERNEL);
> + devfreq->trans_table = devm_kzalloc(&devfreq->dev,
> + array3_size(sizeof(unsigned int),
> + devfreq->profile->max_state,
> + devfreq->profile->max_state),
> + GFP_KERNEL);
> if (!devfreq->trans_table) {
> mutex_unlock(&devfreq->lock);
> err = -ENOMEM;
> @@ -696,9 +695,9 @@ struct devfreq *devfreq_add_device(struct device *dev,
> }
>
> devfreq->time_in_state = devm_kcalloc(&devfreq->dev,
> - devfreq->profile->max_state,
> - sizeof(unsigned long),
> - GFP_KERNEL);
> + devfreq->profile->max_state,
> + sizeof(unsigned long),
> + GFP_KERNEL);
> if (!devfreq->time_in_state) {
> mutex_unlock(&devfreq->lock);
> err = -ENOMEM;
> @@ -1184,7 +1183,7 @@ static ssize_t available_governors_show(struct device *d,
> */
> if (df->governor->immutable) {
> count = scnprintf(&buf[count], DEVFREQ_NAME_LEN,
> - "%s ", df->governor_name);
> + "%s ", df->governor_name);
> /*
> * The devfreq device shows the registered governor except for
> * immutable governors such as passive governor .
> @@ -1497,8 +1496,8 @@ EXPORT_SYMBOL(devfreq_recommended_opp);
>
> /**
> * devfreq_register_opp_notifier() - Helper function to get devfreq notified
> - * for any changes in the OPP availability
> - * changes
> + * for any changes in the OPP availability
> + * changes
> * @dev: The devfreq user device. (parent of devfreq)
> * @devfreq: The devfreq object.
> */
> @@ -1510,8 +1509,8 @@ EXPORT_SYMBOL(devfreq_register_opp_notifier);
>
> /**
> * devfreq_unregister_opp_notifier() - Helper function to stop getting devfreq
> - * notified for any changes in the OPP
> - * availability changes anymore.
> + * notified for any changes in the OPP
> + * availability changes anymore.
> * @dev: The devfreq user device. (parent of devfreq)
> * @devfreq: The devfreq object.
> *
> @@ -1530,8 +1529,8 @@ static void devm_devfreq_opp_release(struct device *dev, void *res)
> }
>
> /**
> - * devm_ devfreq_register_opp_notifier()
> - * - Resource-managed devfreq_register_opp_notifier()
> + * devm_devfreq_register_opp_notifier() - Resource-managed
> + * devfreq_register_opp_notifier()
> * @dev: The devfreq user device. (parent of devfreq)
> * @devfreq: The devfreq object.
> */
> @@ -1559,8 +1558,8 @@ int devm_devfreq_register_opp_notifier(struct device *dev,
> EXPORT_SYMBOL(devm_devfreq_register_opp_notifier);
>
> /**
> - * devm_devfreq_unregister_opp_notifier()
> - * - Resource-managed devfreq_unregister_opp_notifier()
> + * devm_devfreq_unregister_opp_notifier() - Resource-managed
> + * devfreq_unregister_opp_notifier()
> * @dev: The devfreq user device. (parent of devfreq)
> * @devfreq: The devfreq object.
> */
> @@ -1579,8 +1578,8 @@ EXPORT_SYMBOL(devm_devfreq_unregister_opp_notifier);
> * @list: DEVFREQ_TRANSITION_NOTIFIER.
> */
> int devfreq_register_notifier(struct devfreq *devfreq,
> - struct notifier_block *nb,
> - unsigned int list)
> + struct notifier_block *nb,
> + unsigned int list)
> {
> int ret = 0;
>
> @@ -1686,9 +1685,9 @@ EXPORT_SYMBOL(devm_devfreq_register_notifier);
> * @list: DEVFREQ_TRANSITION_NOTIFIER.
> */
> void devm_devfreq_unregister_notifier(struct device *dev,
> - struct devfreq *devfreq,
> - struct notifier_block *nb,
> - unsigned int list)
> + struct devfreq *devfreq,
> + struct notifier_block *nb,
> + unsigned int list)
> {
> WARN_ON(devres_release(dev, devm_devfreq_notifier_release,
> devm_devfreq_dev_match, devfreq));
>
--
Best Regards,
Chanwoo Choi
Samsung Electronics
On Mon, Jan 21, 2019 at 10:28 AM Chanwoo Choi <[email protected]> wrote:
>
> On 19. 1. 21. 오전 11:19, MyungJoo Ham wrote:
> > Following up with complaints on inconsistent indentation from
> > Yangtao Li, this fixes indentation inconsistency.
> >
> > In principle, this tries to put arguments aligned to the left
> > including the first argument except for in the case where
> > the first argument is on the far-right side.
Acked-by: Yangtao Li <[email protected]>
> >
> > Signed-off-by: MyungJoo Ham <[email protected]>
> > ---
> > drivers/devfreq/devfreq.c | 49 +++++++++++++++++++++++------------------------
> > 1 file changed, 24 insertions(+), 25 deletions(-)
>
> Looks better before.
> Reviewed-by: Chanwoo Choi <[email protected]>
>
> >
> > diff --git a/drivers/devfreq/devfreq.c b/drivers/devfreq/devfreq.c
> > index 4af608a..428a1de 100644
> > --- a/drivers/devfreq/devfreq.c
> > +++ b/drivers/devfreq/devfreq.c
> > @@ -528,7 +528,7 @@ void devfreq_interval_update(struct devfreq *devfreq, unsigned int *delay)
> > mutex_lock(&devfreq->lock);
> > if (!devfreq->stop_polling)
> > queue_delayed_work(devfreq_wq, &devfreq->work,
> > - msecs_to_jiffies(devfreq->profile->polling_ms));
> > + msecs_to_jiffies(devfreq->profile->polling_ms));
> > }
> > out:
> > mutex_unlock(&devfreq->lock);
> > @@ -537,7 +537,7 @@ EXPORT_SYMBOL(devfreq_interval_update);
> >
> > /**
> > * devfreq_notifier_call() - Notify that the device frequency requirements
> > - * has been changed out of devfreq framework.
> > + * has been changed out of devfreq framework.
> > * @nb: the notifier_block (supposed to be devfreq->nb)
> > * @type: not used
> > * @devp: not used
> > @@ -683,12 +683,11 @@ struct devfreq *devfreq_add_device(struct device *dev,
> > goto err_out;
> > }
> >
> > - devfreq->trans_table =
> > - devm_kzalloc(&devfreq->dev,
> > - array3_size(sizeof(unsigned int),
> > - devfreq->profile->max_state,
> > - devfreq->profile->max_state),
> > - GFP_KERNEL);
> > + devfreq->trans_table = devm_kzalloc(&devfreq->dev,
> > + array3_size(sizeof(unsigned int),
> > + devfreq->profile->max_state,
> > + devfreq->profile->max_state),
> > + GFP_KERNEL);
> > if (!devfreq->trans_table) {
> > mutex_unlock(&devfreq->lock);
> > err = -ENOMEM;
> > @@ -696,9 +695,9 @@ struct devfreq *devfreq_add_device(struct device *dev,
> > }
> >
> > devfreq->time_in_state = devm_kcalloc(&devfreq->dev,
> > - devfreq->profile->max_state,
> > - sizeof(unsigned long),
> > - GFP_KERNEL);
> > + devfreq->profile->max_state,
> > + sizeof(unsigned long),
> > + GFP_KERNEL);
> > if (!devfreq->time_in_state) {
> > mutex_unlock(&devfreq->lock);
> > err = -ENOMEM;
> > @@ -1184,7 +1183,7 @@ static ssize_t available_governors_show(struct device *d,
> > */
> > if (df->governor->immutable) {
> > count = scnprintf(&buf[count], DEVFREQ_NAME_LEN,
> > - "%s ", df->governor_name);
> > + "%s ", df->governor_name);
> > /*
> > * The devfreq device shows the registered governor except for
> > * immutable governors such as passive governor .
> > @@ -1497,8 +1496,8 @@ EXPORT_SYMBOL(devfreq_recommended_opp);
> >
> > /**
> > * devfreq_register_opp_notifier() - Helper function to get devfreq notified
> > - * for any changes in the OPP availability
> > - * changes
> > + * for any changes in the OPP availability
> > + * changes
> > * @dev: The devfreq user device. (parent of devfreq)
> > * @devfreq: The devfreq object.
> > */
> > @@ -1510,8 +1509,8 @@ EXPORT_SYMBOL(devfreq_register_opp_notifier);
> >
> > /**
> > * devfreq_unregister_opp_notifier() - Helper function to stop getting devfreq
> > - * notified for any changes in the OPP
> > - * availability changes anymore.
> > + * notified for any changes in the OPP
> > + * availability changes anymore.
> > * @dev: The devfreq user device. (parent of devfreq)
> > * @devfreq: The devfreq object.
> > *
> > @@ -1530,8 +1529,8 @@ static void devm_devfreq_opp_release(struct device *dev, void *res)
> > }
> >
> > /**
> > - * devm_ devfreq_register_opp_notifier()
> > - * - Resource-managed devfreq_register_opp_notifier()
> > + * devm_devfreq_register_opp_notifier() - Resource-managed
> > + * devfreq_register_opp_notifier()
> > * @dev: The devfreq user device. (parent of devfreq)
> > * @devfreq: The devfreq object.
> > */
> > @@ -1559,8 +1558,8 @@ int devm_devfreq_register_opp_notifier(struct device *dev,
> > EXPORT_SYMBOL(devm_devfreq_register_opp_notifier);
> >
> > /**
> > - * devm_devfreq_unregister_opp_notifier()
> > - * - Resource-managed devfreq_unregister_opp_notifier()
> > + * devm_devfreq_unregister_opp_notifier() - Resource-managed
> > + * devfreq_unregister_opp_notifier()
> > * @dev: The devfreq user device. (parent of devfreq)
> > * @devfreq: The devfreq object.
> > */
> > @@ -1579,8 +1578,8 @@ EXPORT_SYMBOL(devm_devfreq_unregister_opp_notifier);
> > * @list: DEVFREQ_TRANSITION_NOTIFIER.
> > */
> > int devfreq_register_notifier(struct devfreq *devfreq,
> > - struct notifier_block *nb,
> > - unsigned int list)
> > + struct notifier_block *nb,
> > + unsigned int list)
> > {
> > int ret = 0;
> >
> > @@ -1686,9 +1685,9 @@ EXPORT_SYMBOL(devm_devfreq_register_notifier);
> > * @list: DEVFREQ_TRANSITION_NOTIFIER.
> > */
> > void devm_devfreq_unregister_notifier(struct device *dev,
> > - struct devfreq *devfreq,
> > - struct notifier_block *nb,
> > - unsigned int list)
> > + struct devfreq *devfreq,
> > + struct notifier_block *nb,
> > + unsigned int list)
> > {
> > WARN_ON(devres_release(dev, devm_devfreq_notifier_release,
> > devm_devfreq_dev_match, devfreq));
> >
>
> --
> Best Regards,
> Chanwoo Choi
> Samsung Electronics