From: Harry Wentland <harry.wentland@amd.com>
To: Dan Carpenter
<dan.carpenter-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>,
Alex Deucher <alexander.deucher-5C7GfCeVMHo@public.gmane.org>
Cc: "David (ChunMing) Zhou"
<David1.Zhou-5C7GfCeVMHo@public.gmane.org>,
"Charlene Liu" <charlene.liu-5C7GfCeVMHo@public.gmane.org>,
"David Airlie" <airlied-cv59FeDIM0c@public.gmane.org>,
kernel-janitors-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org,
"Dmytro Laktyushkin"
<Dmytro.Laktyushkin-5C7GfCeVMHo@public.gmane.org>,
"Yongqiang Sun" <yongqiang.sun-5C7GfCeVMHo@public.gmane.org>,
"Tony Cheng" <tony.cheng-5C7GfCeVMHo@public.gmane.org>,
"Christian König" <christian.koenig-5C7GfCeVMHo@public.gmane.org>
Subject: Re: [PATCH] drm/amd/display: indent an if statement
Date: Tue, 21 Aug 2018 18:51:19 +0000 [thread overview]
Message-ID: <3d491f32-245c-30ac-4f13-39da721c3dbc@amd.com> (raw)
In-Reply-To: <20180814090944.qjm4vxyrjdqxu4d7-vMlZ3nK25oGAIHFoDFOv9A@public.gmane.org>
On 2018-08-14 05:09 AM, Dan Carpenter wrote:
> The if statement isn't indented and it makes static checkers complain.
>
> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Reviewed-by: Harry Wentland <harry.wentland@amd.com>
Harry
>
> diff --git a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
> index 4ca41d6e3bcf..d82ba58c720f 100644
> --- a/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
> +++ b/drivers/gpu/drm/amd/display/dc/core/dc_resource.c
> @@ -349,7 +349,7 @@ static bool is_dp_and_hdmi_sharable(
>
> if (stream1->clamping.c_depth != COLOR_DEPTH_888 ||
> stream2->clamping.c_depth != COLOR_DEPTH_888)
> - return false;
> + return false;
>
> return true;
>
>
next prev parent reply other threads:[~2018-08-21 18:51 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-11-04 6:12 [PATCH] drm/amd/display: indent an if statement Dan Carpenter
2017-11-04 9:16 ` walter harms
[not found] ` <59FD8567.4030500-fPG8STNUNVg@public.gmane.org>
2017-11-04 10:07 ` Christian König
2017-11-06 7:07 ` [PATCH] drm/amd/display: small cleanup in destruct() Dan Carpenter
2017-11-06 8:44 ` Christian König
2017-11-06 21:29 ` Alex Deucher
2017-11-06 6:42 ` [PATCH] drm/amd/display: indent an if statement Dan Carpenter
2018-08-14 9:09 ` Dan Carpenter
[not found] ` <20180814090944.qjm4vxyrjdqxu4d7-vMlZ3nK25oGAIHFoDFOv9A@public.gmane.org>
2018-08-21 18:51 ` Harry Wentland [this message]
2020-08-03 14:35 ` [PATCH] drm/amd/display: Indent " Dan Carpenter
2020-08-04 20:33 ` Alex Deucher
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=3d491f32-245c-30ac-4f13-39da721c3dbc@amd.com \
--to=harry.wentland@amd.com \
--cc=David1.Zhou-5C7GfCeVMHo@public.gmane.org \
--cc=Dmytro.Laktyushkin-5C7GfCeVMHo@public.gmane.org \
--cc=airlied-cv59FeDIM0c@public.gmane.org \
--cc=alexander.deucher-5C7GfCeVMHo@public.gmane.org \
--cc=amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=charlene.liu-5C7GfCeVMHo@public.gmane.org \
--cc=christian.koenig-5C7GfCeVMHo@public.gmane.org \
--cc=dan.carpenter-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org \
--cc=kernel-janitors-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
--cc=tony.cheng-5C7GfCeVMHo@public.gmane.org \
--cc=yongqiang.sun-5C7GfCeVMHo@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox