On 3/31/22 16:48, Bjorn Andersson wrote:
On Thu 31 Mar 15:34 PDT 2022, Bart Van Assche wrote:
#if IS_ENABLED(CONFIG_DEVFREQ_GOV_SIMPLE_ONDEMAND)
static void ufs_qcom_config_scaling_param(struct ufs_hba *hba,
- struct devfreq_dev_profile *p,
- void *data)
+ struct devfreq_dev_profile *p,
+ struct devfreq_simple_ondemand_data *d)
This doesn't look to be properly indended to match the '('?
What does ./scripts/checkpatch.pl --strict say about the patch?
Other than that, the change looks good, so feel free to add my r-b once
you've double checked the indentation.
Reviewed-by: Bjorn Andersson <bjorn.andersson@xxxxxxxxxx>
Checkpatch doesn't verify this kind of indentation as far as I know.
Anyway, I will fix up the indentation when I repost this patch. Thanks
for having taken a look.
Bart.
