linux-kernel - Re: [patch] drm/amdgpu/cgs: cleanup some indenting

Open Source and information security mailing list archives

Date:	Mon, 4 Jan 2016 16:19:10 -0500
From:	Alex Deucher <alexdeucher@...il.com>
To:	Dan Carpenter <dan.carpenter@...cle.com>
Cc:	David Airlie <airlied@...ux.ie>, Jammy Zhou <Jammy.Zhou@....com>,
	kernel-janitors@...r.kernel.org,
	LKML <linux-kernel@...r.kernel.org>,
	Maling list - DRI developers 
	<dri-devel@...ts.freedesktop.org>,
	Eric Huang <JinHuiEric.Huang@....com>,
	Alex Deucher <alexander.deucher@....com>,
	Rex Zhu <Rex.Zhu@....com>,
	Christian König <christian.koenig@....com>
Subject: Re: [patch] drm/amdgpu/cgs: cleanup some indenting

On Mon, Jan 4, 2016 at 3:43 PM, Dan Carpenter <dan.carpenter@...cle.com> wrote:
> This code is indented too far.  Also we normally use spaces to align if
> statement conditions.
>
> Signed-off-by: Dan Carpenter <dan.carpenter@...cle.com>

Applied.  thanks!

Alex

>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c
> index 6fa0fea..59485d0 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_cgs.c
> @@ -843,15 +843,15 @@ static int amdgpu_cgs_acpi_eval_object(void *cgs_device,
>         if (info->input_count > 0) {
>                 if (info->pinput_argument == NULL)
>                         return -EINVAL;
> -                       argument = info->pinput_argument;
> -                       func_no = argument->value;
> -                       for (i = 0; i < info->input_count; i++) {
> -                               if (((argument->type == ACPI_TYPE_STRING) ||
> -                                       (argument->type == ACPI_TYPE_BUFFER))
> -                                       && (argument->pointer == NULL))
> -                                       return -EINVAL;
> -                               argument++;
> -                       }
> +               argument = info->pinput_argument;
> +               func_no = argument->value;
> +               for (i = 0; i < info->input_count; i++) {
> +                       if (((argument->type == ACPI_TYPE_STRING) ||
> +                            (argument->type == ACPI_TYPE_BUFFER)) &&
> +                           (argument->pointer == NULL))
> +                               return -EINVAL;
> +                       argument++;
> +               }
>         }
>
>         if (info->output_count > 0) {
> _______________________________________________
> dri-devel mailing list
> dri-devel@...ts.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives