From: Rodrigo Siqueira Jordao <Rodrigo.Siqueira@amd.com>
To: "André Almeida" <andrealmeid@igalia.com>,
"Magali Lemes" <magalilemes00@gmail.com>
Cc: siqueirajordao@riseup.net, sunpeng.li@amd.com,
alexander.deucher@amd.com, harry.wentland@amd.com,
tales.aparecida@gmail.com, linux-kernel@vger.kernel.org,
amd-gfx@lists.freedesktop.org, mairacanal@riseup.net,
dri-devel@lists.freedesktop.org, isabbasso@riseup.net,
andrealmeid@riseup.net, christian.koenig@amd.com,
Xinhui.Pan@amd.com, airlied@linux.ie, daniel@ffwll.ch,
alex.hung@amd.com, mwen@igalia.com
Subject: Re: [PATCH v2 1/3] drm/amd/display: make variables static
Date: Tue, 2 Aug 2022 12:38:42 -0400 [thread overview]
Message-ID: <98cb62b2-e11b-4675-7017-7feba8f196e5@amd.com> (raw)
In-Reply-To: <7069d082-f508-af3a-8098-081e59a4e41f@igalia.com>
On 2022-08-02 09:31, André Almeida wrote:
>
>
> Às 22:06 de 29/07/22, Magali Lemes escreveu:
>> As "dcn3_1_soc", "dcn3_15_soc", and "dcn3_16_soc" are not used outside
>> of their corresponding "dcn3*_fpu.c", make them static and remove their
>> extern declaration.
>>
>> Fixes: 26f4712aedbd ("drm/amd/display: move FPU related code from dcn31 to dml/dcn31 folder")
>> Fixes: fa896297b31b ("drm/amd/display: move FPU related code from dcn315 to dml/dcn31 folder")
>> Fixes: 3f8951cc123f ("drm/amd/display: move FPU related code from dcn316 to dml/dcn31 folder")
>> Signed-off-by: Magali Lemes <magalilemes00@gmail.com>
>> Reviewed-by: Maíra Canal <mairacanal@riseup.net>
>> Reviewed-by: Melissa Wen <mwen@igalia.com>
>> ---
>
> Series is Reviewed-by: André Almeida <andrealmeid@igalia.com>
Series also
Reviewed-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
Applied to amd-staging-drm-next.
Thanks
Siqueira
prev parent reply other threads:[~2022-08-02 16:39 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-07-30 1:06 Magali Lemes
2022-07-30 1:07 ` [PATCH v2 2/3] drm/amd/display: remove header from source file Magali Lemes
2022-07-30 1:07 ` [PATCH v2 3/3] drm/amd/display: include missing headers Magali Lemes
2022-08-02 13:31 ` [PATCH v2 1/3] drm/amd/display: make variables static André Almeida
2022-08-02 16:38 ` Rodrigo Siqueira Jordao [this message]
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=98cb62b2-e11b-4675-7017-7feba8f196e5@amd.com \
--to=rodrigo.siqueira@amd.com \
--cc=Xinhui.Pan@amd.com \
--cc=airlied@linux.ie \
--cc=alex.hung@amd.com \
--cc=alexander.deucher@amd.com \
--cc=amd-gfx@lists.freedesktop.org \
--cc=andrealmeid@igalia.com \
--cc=andrealmeid@riseup.net \
--cc=christian.koenig@amd.com \
--cc=daniel@ffwll.ch \
--cc=dri-devel@lists.freedesktop.org \
--cc=harry.wentland@amd.com \
--cc=isabbasso@riseup.net \
--cc=linux-kernel@vger.kernel.org \
--cc=magalilemes00@gmail.com \
--cc=mairacanal@riseup.net \
--cc=mwen@igalia.com \
--cc=siqueirajordao@riseup.net \
--cc=sunpeng.li@amd.com \
--cc=tales.aparecida@gmail.com \
/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
all inboxes | Powered by JetHome®