Re: [PATCH] clk: qcom: gdsc: Remove direct runtime PM calls

Johan Hovold
2022-11-03T13:21:59Z

On Wed, Nov 02, 2022 at 09:53:49AM -0700, Stephen Boyd wrote:
> Quoting Johan Hovold (2022-11-02 03:52:39)

> > > Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org>
> > > Cc: Johan Hovold <johan+linaro@kernel.org>
> > > Cc: Ulf Hansson <ulf.hansson@linaro.org>
> > > Cc: Taniya Das <quic_tdas@quicinc.com>
> > > Cc: Satya Priya <quic_c_skakit@quicinc.com>
> > > Cc: Douglas Anderson <dianders@chromium.org>
> > > Cc: Matthias Kaehlcke <mka@chromium.org>
> > > Reported-by: Stephen Boyd <swboyd@chromium.org>
> >
> > We typically don't add Reported-by tags for bugs we find and fix
> > ourselves.
>
> Heh, I didn't see anything like that in Documentation/ so it seems fine.
> I debugged my problem and reported it.

I'd say the documentation is pretty clear on this matter:

Reported-by: names a user who reported a problem which is fixed by this
patch; this tag is used to give credit to the (often underappreciated)
people who test our code and let us know when things do not work
correctly.

- Documentation/process/5.Posting.rst

The Reported-by tag gives credit to people who find bugs and report
them and it hopefully inspires them to help us again in the future.
Please note that if the bug was reported in private, then ask for
permission first before using the Reported-by tag.

- Documentation/process/submitting-patches.rst

Just like you don't add a Tested-by tag for every patch you submit, it
is implied that you found the issue you fix unless you explicitly
attribute that to a third party using Reported-by.

This is the first time I see anyone trying to use Reported-by this way,
and even if you think the documentation isn't clear enough on this, our
praxis is.

Johan