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: <20220303230301.255049-1-suzuki.poulose@arm.com> <YipxKx+P7Dyc70vS@kroah.com>
In-Reply-To: <YipxKx+P7Dyc70vS@kroah.com>
From:   Mathieu Poirier <mathieu.poirier@linaro.org>
Date:   Thu, 10 Mar 2022 17:09:49 -0700
Message-ID: <CANLsYkxJd-wdnkvL1=WQOEQR2E5TJDK_5XR1QWkkOt8giRfkAw@mail.gmail.com>
Subject: Re: [GIT PULL] coresight changes for v5.18
To:     Greg KH <gregkh@linuxfoundation.org>
Cc:     Suzuki K Poulose <Suzuki.Poulose@arm.com>,
        coresight@lists.linaro.org, linux-kernel@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Thu, 10 Mar 2022 at 14:44, Greg KH <gregkh@linuxfoundation.org> wrote:
>
> On Thu, Mar 03, 2022 at 11:03:01PM +0000, Suzuki K Poulose wrote:
> > Hi Greg
> >
> > Please find the pull request for coresight subsystem for v5.18.
> >
> > Suzuki
> >
> > The following changes since commit dfd42facf1e4ada021b939b4e19c935dcdd55566:
> >
> >   Linux 5.17-rc3 (2022-02-06 12:20:50 -0800)
> >
> > are available in the Git repository at:
> >
> >   git://git.kernel.org/pub/scm/linux/kernel/git/coresight/linux.git tags/coresight-next-v5.18
> >
> > for you to fetch changes up to b54f53bc11a584713f79a704c70c488489f524b8:
> >
> >   coresight: Drop unused 'none' enum value for each component (2022-02-28 09:51:40 -0700)
>
> I have the following errors when pulling this tree and having the
> scripts check the commits:
>
> Commit 5340bf5df9d2 ("coresight: syscfg: Fix memleak on registration failure in cscfg_create_device")
>         committer Signed-off-by missing
>         author email:    linmq006@gmail.com
>         committer email: suzuki.poulose@arm.com
>         Signed-off-by: Miaoqian Lin <linmq006@gmail.com>
>         Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
>
> Commit 91a2f2941df2 ("coresight: Fix TRCCONFIGR.QE sysfs interface")
>         committer Signed-off-by missing
>         author email:    james.clark@arm.com
>         committer email: suzuki.poulose@arm.com
>         Signed-off-by: James Clark <james.clark@arm.com>
>         Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
>
> Commit 7f4cd3375906 ("coresight: trbe: Work around the trace data corruption")
>         committer Signed-off-by missing
>         author email:    anshuman.khandual@arm.com
>         committer email: suzuki.poulose@arm.com
>         Signed-off-by: Anshuman Khandual <anshuman.khandual@arm.com>
>         Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
>
> Commit 0ecf2c747437 ("coresight: trbe: Work around the invalid prohibited states")
>         committer Signed-off-by missing
>         author email:    anshuman.khandual@arm.com
>         committer email: suzuki.poulose@arm.com
>         Signed-off-by: Anshuman Khandual <anshuman.khandual@arm.com>
>         Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
>
> Commit 8b6927d0adad ("coresight: trbe: Work around the ignored system register writes")
>         committer Signed-off-by missing
>         author email:    anshuman.khandual@arm.com
>         committer email: suzuki.poulose@arm.com
>         Signed-off-by: Anshuman Khandual <anshuman.khandual@arm.com>
>         Signed-off-by: Mathieu Poirier <mathieu.poirier@linaro.org>
>
>
> What went wrong???
>

(Sigh)

I know what happened.  The tree was rebased from -rc1 to -rc3 to pick
up a dependency with the aarch64 tree that was needed for a patch.  In
doing so the original committer information was overwritten, which is
what the script is complaining about.

We will send you another pull request.

> linux-next didn't complain about this already?
>

It did not.

Thanks for the patience,
Mathieu