From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mauro Carvalho Chehab Subject: Re: [git:media_tree/master] [media] ARM: dts: exynos: add HDMI controller phandle to exynos4.dtsi Date: Tue, 11 Apr 2017 05:54:13 -0300 Message-ID: <20170411055413.07957d91@vento.lan> References: Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-samsung-soc-owner@vger.kernel.org To: Krzysztof Kozlowski Cc: linuxtv-commits@linuxtv.org, Marek Szyprowski , linux-samsung-soc@vger.kernel.org, devicetree@vger.kernel.org, Hans Verkuil , linux-media@vger.kernel.org List-Id: devicetree@vger.kernel.org Em Tue, 11 Apr 2017 10:36:58 +0200 Krzysztof Kozlowski escreveu: > On Mon, Apr 10, 2017 at 6:12 PM, Mauro Carvalho Chehab > wrote: > > This is an automatic generated email to let you know that the following patch were queued: > > > > Subject: [media] ARM: dts: exynos: add HDMI controller phandle to exynos4.dtsi > > Author: Hans Verkuil > > Date: Tue Dec 13 12:37:16 2016 -0200 > > > > Add the new hdmi phandle to exynos4.dtsi. This phandle is needed by the > > s5p-cec driver to initialize the CEC notifier framework. > > > > Tested with my Odroid U3. > > > > Signed-off-by: Hans Verkuil > > Tested-by: Marek Szyprowski > > CC: linux-samsung-soc@vger.kernel.org > > CC: devicetree@vger.kernel.org > > CC: Krzysztof Kozlowski > > Signed-off-by: Mauro Carvalho Chehab > > > > arch/arm/boot/dts/exynos4.dtsi | 1 + > > 1 file changed, 1 insertion(+) > > > > Mauro, you should not apply it. It is already going through samsung-soc [1]. > if you need this patch for bisectability or any other reasons, I > provided a tag with it here: > https://www.spinics.net/lists/devicetree/msg171182.html > > Please drop the patch because now it will get duplicated. Having exactly the same patch applied on multiple trees usually is not a problem, provided that it doesn't rise a non-trivial conflict. I avoid rebase the tree where this patch is applied, as rebasing it affect the workflow of other developers. I'm afraid that, if I revert this patch, it will cause more harm than good. So, I guess the best solution to fix the issue would be to pull from a stable branch on your tree with has this patch and solve conflicts, if any. This way, nothing will popup when merging upstream. Regards, Mauro > > Best regards, > Krzysztof > > [1] https://www.spinics.net/lists/arm-kernel/msg575229.html Thanks, Mauro