Messages in this thread | | | Date | Thu, 13 Feb 2020 11:57:25 -0800 | From | Matthias Kaehlcke <> | Subject | Re: [PATCH V4 0/4] Enable video on sc7180 |
| |
On Thu, Feb 13, 2020 at 11:26:27AM -0800, Bjorn Andersson wrote: > On Thu 13 Feb 10:53 PST 2020, Matthias Kaehlcke wrote: > > > Hi Bjorn, > > > > On Mon, Feb 03, 2020 at 10:02:40AM -0800, Bjorn Andersson wrote: > > > On Tue 14 Jan 04:53 PST 2020, Dikshita Agarwal wrote: > > > > > > > Hello, > > > > > > > > Changes since v3: > > > > > > > > - addressed DT and DT schema review comments. > > > > > > > > - renamed DT schema file. > > > > > > > > v3 can be found at [1]. > > > > These changes depend on patch series [2] - [6]. > > > > > > > > Thanks, > > > > Dikshita > > > > > > > > > > Picked up the dts patches for 5.7, with Stan's acks > > > > I can't seem to find the patches in the QCOM repo, neither in > > 'arm64-for-5.7' nor 'for-next'. Am I looking at the wrong place or > > maybe you forget to push these? > > > > Thanks for the question Matthias, I was looking for this email as I > rebased onto v5.6-rc1 earlier this week, but got distracted. > > I pulled them in, but in the rebase I realized that we don't have the > interconnects in place, so in it's current form these patches doesn't > compile. > > Seems we're waiting for rather trivial respin of > https://lore.kernel.org/linux-arm-msm/1577782737-32068-1-git-send-email-okukatla@codeaurora.org/ > to get this settled.
Hm, there has been no response to the comments in more than a month, also the series depends on another ('Split SDM845 interconnect nodes and consolidate RPMh support' https://patchwork.kernel.org/project/linux-arm-msm/list/?series=238831), even though that isn't mentioned anywhere.
IIUC the interconnect configuration isn't strictly required to get the video codec to work. I wonder if it would make sense to respin this series to remove the interconnect properties. They could be added in a separate patch after the ICC support has landed.
Dikshita/Bjorn, what do you think?
| |