All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bryan O'Donoghue <bryan.odonoghue@linaro.org>
To: "Krishna chaitanya chundru" <quic_krichai@quicinc.com>,
	"Bjorn Andersson" <andersson@kernel.org>,
	"Konrad Dybcio" <konrad.dybcio@linaro.org>,
	"Bjorn Helgaas" <bhelgaas@google.com>,
	"Lorenzo Pieralisi" <lpieralisi@kernel.org>,
	"Krzysztof Wilczyński" <kw@linux.com>,
	"Rob Herring" <robh@kernel.org>,
	"Krzysztof Kozlowski" <krzysztof.kozlowski+dt@linaro.org>,
	"Conor Dooley" <conor+dt@kernel.org>,
	"Manivannan Sadhasivam" <mani@kernel.org>,
	"Rob Herring" <robh+dt@kernel.org>,
	"Johan Hovold" <johan+linaro@kernel.org>,
	"Brian Masney" <bmasney@redhat.com>,
	"Georgi Djakov" <djakov@kernel.org>
Cc: linux-arm-msm@vger.kernel.org, vireshk@kernel.org,
	quic_vbadigan@quicinc.com, quic_skananth@quicinc.com,
	quic_nitegupt@quicinc.com, linux-pci@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	stable@vger.kernel.org
Subject: Re: [PATCH v6 3/6] PCI: qcom: Add missing icc bandwidth vote for cpu to PCIe path
Date: Fri, 12 Jan 2024 15:17:28 +0000	[thread overview]
Message-ID: <fecfd2d9-7302-4eb6-92d0-c2efbe824bf4@linaro.org> (raw)
In-Reply-To: <20240112-opp_support-v6-3-77bbf7d0cc37@quicinc.com>

On 12/01/2024 14:22, Krishna chaitanya chundru wrote:
> CPU-PCIe path consits for registers PCIe BAR space, config space.
> As there is less access on this path compared to pcie to mem path
> add minimum vote i.e GEN1x1 bandwidth always.
> 
> In suspend remove the cpu vote after register space access is done.
> 
> Fixes: c4860af88d0c ("PCI: qcom: Add basic interconnect support")

If this patch is a Fixes then don't you need the accompanying dts change 
as a parallel Fixes too ?

i.e. without the dts update - you won't have the nodes in the dts to 
consume => applying this code to the stable kernel absent the dts will 
result in no functional change and therefore no bugfix.

I'm not sure if you are asked to put a Fixes here but it seems to be it 
should either be dropped or require a parallel Fixes: tag for the dts 
and yaml changes.

What is the bug this change fixes in the backport ?

> cc: stable@vger.kernel.org
> Signed-off-by: Krishna chaitanya chundru <quic_krichai@quicinc.com>

---
bod

  reply	other threads:[~2024-01-12 15:17 UTC|newest]

Thread overview: 58+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-01-12 14:21 [PATCH v6 0/6] PCI: qcom: Add support for OPP Krishna chaitanya chundru
2024-01-12 14:22 ` [PATCH v6 1/6] dt-bindings: PCI: qcom: Add interconnects path as required property Krishna chaitanya chundru
2024-01-12 16:55   ` Conor Dooley
2024-01-12 17:12     ` Dmitry Baryshkov
2024-01-12 17:27       ` Conor Dooley
2024-01-19 22:34   ` Rob Herring
2024-01-29 15:22   ` Manivannan Sadhasivam
2024-01-12 14:22 ` [PATCH v6 2/6] arm64: dts: qcom: sm8450: Add interconnect path to PCIe node Krishna chaitanya chundru
2024-01-29 15:24   ` Manivannan Sadhasivam
2024-01-12 14:22 ` [PATCH v6 3/6] PCI: qcom: Add missing icc bandwidth vote for cpu to PCIe path Krishna chaitanya chundru
2024-01-12 15:17   ` Bryan O'Donoghue [this message]
2024-01-12 22:33     ` Konrad Dybcio
2024-01-16 10:52       ` Johan Hovold
2024-01-17  9:13         ` Konrad Dybcio
2024-01-16  4:52     ` Krishna Chaitanya Chundru
2024-01-16 10:06       ` Bryan O'Donoghue
2024-01-12 15:30   ` Dmitry Baryshkov
2024-01-16  4:57     ` Krishna Chaitanya Chundru
2024-01-17  6:39       ` Manivannan Sadhasivam
2024-01-29 14:10         ` Krishna Chaitanya Chundru
2024-01-12 15:59   ` Johan Hovold
2024-01-12 22:37     ` Konrad Dybcio
2024-01-16 10:54       ` Johan Hovold
2024-01-16  5:04     ` Krishna Chaitanya Chundru
2024-01-16 10:46       ` Johan Hovold
2024-01-12 16:47   ` Bjorn Helgaas
2024-01-16  5:06     ` Krishna Chaitanya Chundru
2024-01-12 14:22 ` [PATCH v6 4/6] dt-bindings: pci: qcom: Add opp table Krishna chaitanya chundru
2024-01-12 14:22 ` [PATCH v6 5/6] arm64: dts: qcom: sm8450: Add opp table support to PCIe Krishna chaitanya chundru
2024-01-29 16:04   ` Manivannan Sadhasivam
2024-01-30  6:11     ` Viresh Kumar
2024-01-30  7:14       ` Manivannan Sadhasivam
2024-01-30  8:36         ` Viresh Kumar
2024-01-30  9:48           ` Manivannan Sadhasivam
2024-01-30  9:55             ` Viresh Kumar
2024-01-30 13:16               ` Manivannan Sadhasivam
2024-01-31  5:23                 ` Viresh Kumar
2024-01-31  8:46                   ` Manivannan Sadhasivam
2024-01-31 10:00                     ` Viresh Kumar
2024-02-01 14:45                   ` Konrad Dybcio
2024-02-02  7:33                     ` Viresh Kumar
2024-02-09 21:14                       ` Konrad Dybcio
2024-02-19  7:02                         ` Krishna Chaitanya Chundru
2024-02-19 10:28                         ` Viresh Kumar
2024-02-19 12:38                           ` Manivannan Sadhasivam
2024-01-12 14:22 ` [PATCH v6 6/6] PCI: qcom: Add OPP support to scale performance state of power domain Krishna chaitanya chundru
2024-01-12 15:33   ` Dmitry Baryshkov
2024-01-16  5:17     ` Krishna Chaitanya Chundru
2024-01-16  9:55       ` Dmitry Baryshkov
2024-01-16 11:00         ` Johan Hovold
2024-02-01 11:54         ` Manivannan Sadhasivam
2024-02-01 11:58           ` Dmitry Baryshkov
2024-02-01 12:07             ` Manivannan Sadhasivam
2024-01-12 16:50   ` Bjorn Helgaas
2024-01-16  5:07     ` Krishna Chaitanya Chundru
2024-01-12 22:44   ` Konrad Dybcio
2024-01-16  5:18     ` Krishna Chaitanya Chundru
2024-01-29 16:00   ` Manivannan Sadhasivam

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=fecfd2d9-7302-4eb6-92d0-c2efbe824bf4@linaro.org \
    --to=bryan.odonoghue@linaro.org \
    --cc=andersson@kernel.org \
    --cc=bhelgaas@google.com \
    --cc=bmasney@redhat.com \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=djakov@kernel.org \
    --cc=johan+linaro@kernel.org \
    --cc=konrad.dybcio@linaro.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=kw@linux.com \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-pci@vger.kernel.org \
    --cc=lpieralisi@kernel.org \
    --cc=mani@kernel.org \
    --cc=quic_krichai@quicinc.com \
    --cc=quic_nitegupt@quicinc.com \
    --cc=quic_skananth@quicinc.com \
    --cc=quic_vbadigan@quicinc.com \
    --cc=robh+dt@kernel.org \
    --cc=robh@kernel.org \
    --cc=stable@vger.kernel.org \
    --cc=vireshk@kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.