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@xxxxxxxxxxxxxx/ > 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?