Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20230710082534.762599c8@canb.auug.org.au> <54adffa8-b500-e1b4-7274-9a5354be8cfe@amd.com>
In-Reply-To: <54adffa8-b500-e1b4-7274-9a5354be8cfe@amd.com>
From:   Alex Deucher <alexdeucher@gmail.com>
Date:   Mon, 10 Jul 2023 09:24:35 -0400
Message-ID: <CADnq5_PvEKum2Wdmei7Zk8kBe_6NWzv2Oaqg7juPFriPOtRWgg@mail.gmail.com>
Subject: Re: linux-next: Signed-off-by missing for commit in the amdgpu tree
To:     Aurabindo Pillai <aurabindo.pillai@amd.com>
Cc:     Stephen Rothwell <sfr@canb.auug.org.au>,
        Wesley Chalmers <Wesley.Chalmers@amd.com>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Next Mailing List <linux-next@vger.kernel.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

On Mon, Jul 10, 2023 at 9:21=E2=80=AFAM Aurabindo Pillai
<aurabindo.pillai@amd.com> wrote:
>
>
>
> On 7/9/2023 6:25 PM, Stephen Rothwell wrote:
> > Hi all,
> >
> > Commit
> >
> >    aeddb7d13ad3 ("drm/amd/display: Block optimize on consecutive FAMS e=
nables")
> >
> > is missing a Signed-off-by from its author.
> >
>
> Hi Stephen,
>
> Not sure how the tags went missing, but here is the full commit
> (a7d8d8bcbd9af8d4aa1580bffd418af78384040f) information from
> amd-staging-drm-next:
>
>
>      drm/amd/display: Block optimize on consecutive FAMS enables
>
>      [WHY]
>      It is possible to commit state multiple times in rapid succession wi=
th
>      FAMS enabled; if each of these commits were to set optimized_require=
d,
>      then the user may see latency.
>
>      [HOW]
>      fw_based_mclk_switching is currently not used in dc->clk_mgr; use it
>      to track whether the current state has FAMS enabled;
>      if it has, then do not disable FAMS in prepare_bandwidth, and do
> not set
>      optimized_required.
>
>      Reviewed-by: Rodrigo Siqueira <Rodrigo.Siqueira@amd.com>
>      Signed-off-by: Wesley Chalmers <Wesley.Chalmers@amd.com>
>      Tested-by: Daniel Wheeler <daniel.wheeler@amd.com>
>      Signed-off-by: Alex Deucher <alexander.deucher@amd.com>
>      Acked-by: Hamza Mahfooz <hamza.mahfooz@amd.com>
>

I've fixed it up in my tree.

Alex