diff for duplicates of <1488476152.2684.10.camel@synopsys.com> diff --git a/a/1.txt b/N1/1.txt index 1ff6319..fe787ad 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,41 +1,41 @@ Hi Jose, -On Wed, 2017-02-22@18:19 +0000, Jose Abreu wrote: +On Wed, 2017-02-22 at 18:19 +0000, Jose Abreu wrote: > Default clock frequency for ARC PGU does not match any > existing HDMI mode, instead the default value matches a > DVI mode. Change the clock frequency to 74.25MHz so that -> it matches HDMI mode 1280x720 at 60Hz +> it matches HDMI mode 1280x720@60Hz > -> Signed-off-by: Jose Abreu <joabreu at synopsys.com> -> Cc: Carlos Palminha <palminha at synopsys.com> -> Cc: Alexey Brodkin <abrodkin at synopsys.com> -> Cc: Rob Herring <robh+dt at kernel.org> -> Cc: Vineet Gupta <vgupta at synopsys.com> -> Cc: devicetree at vger.kernel.org -> Cc: linux-snps-arc at lists.infradead.org -> Cc: linux-kernel at vger.kernel.org +> Signed-off-by: Jose Abreu <joabreu@synopsys.com> +> Cc: Carlos Palminha <palminha@synopsys.com> +> Cc: Alexey Brodkin <abrodkin@synopsys.com> +> Cc: Rob Herring <robh+dt@kernel.org> +> Cc: Vineet Gupta <vgupta@synopsys.com> +> Cc: devicetree@vger.kernel.org +> Cc: linux-snps-arc@lists.infradead.org +> Cc: linux-kernel@vger.kernel.org > --- -> ?arch/arc/boot/dts/axs10x_mb.dtsi | 2 +- -> ?1 file changed, 1 insertion(+), 1 deletion(-) +> arch/arc/boot/dts/axs10x_mb.dtsi | 2 +- +> 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arc/boot/dts/axs10x_mb.dtsi b/arch/arc/boot/dts/axs10x_mb.dtsi > index 9d882b1..41cfb29 100644 > --- a/arch/arc/boot/dts/axs10x_mb.dtsi > +++ b/arch/arc/boot/dts/axs10x_mb.dtsi > @@ -51,7 +51,7 @@ -> ? pguclk: pguclk { -> ? #clock-cells = <0>; -> ? compatible = "fixed-clock"; +> pguclk: pguclk { +> #clock-cells = <0>; +> compatible = "fixed-clock"; > - clock-frequency = <74440000>; > + clock-frequency = <74250000>; -> ? }; -> ? }; +> }; +> }; Looks good to me but I have to note that with this change in place monitors that don't report [correct] EDID won't work just because if EDID is missing Linux kernel calculates pixel clock on its -own and for 1280x720 at 60?it gets 74.40 MHz which obviously differs from -yours 74.25 MHz and so?arc_pgu_crtc_atomic_check() returns?-EINVAL. +own and for 1280x720@60 it gets 74.40 MHz which obviously differs from +yours 74.25 MHz and so arc_pgu_crtc_atomic_check() returns -EINVAL. But with the fix I just sent, see http://lists.infradead.org/pipermail/linux-snps-arc/2017-March/002173.html @@ -46,4 +46,8 @@ support of multiple modes on AXS10x boards. That said... -Acked-by: Alexey Brodkin <abrodkin at synopsys.com> +Acked-by: Alexey Brodkin <abrodkin@synopsys.com> +_______________________________________________ +linux-snps-arc mailing list +linux-snps-arc@lists.infradead.org +http://lists.infradead.org/mailman/listinfo/linux-snps-arc diff --git a/a/content_digest b/N1/content_digest index 76e8bbc..a833f02 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,49 +1,55 @@ "ref\07ed6deaf1ae6d5819364c793835aa30316077a7e.1487787376.git.joabreu@synopsys.com\0" "ref\016273b3a26ec770f8d169f04f7cb9eb2fac55909.1487787376.git.joabreu@synopsys.com\0" - "From\0Alexey.Brodkin@synopsys.com (Alexey Brodkin)\0" - "Subject\0[PATCH 2/2] arc: axs10x: Fix ARC PGU default clock frequency\0" + "From\0Alexey Brodkin <Alexey.Brodkin@synopsys.com>\0" + "Subject\0Re: [PATCH 2/2] arc: axs10x: Fix ARC PGU default clock frequency\0" "Date\0Thu, 2 Mar 2017 17:35:52 +0000\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0Jose Abreu <Jose.Abreu@synopsys.com>\0" + "Cc\0devicetree@vger.kernel.org <devicetree@vger.kernel.org>" + Vineet Gupta <Vineet.Gupta1@synopsys.com> + Carlos Palminha <CARLOS.PALMINHA@synopsys.com> + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> + robh+dt@kernel.org <robh+dt@kernel.org> + " linux-snps-arc@lists.infradead.org <linux-snps-arc@lists.infradead.org>\0" "\00:1\0" "b\0" "Hi Jose,\n" "\n" - "On Wed, 2017-02-22@18:19 +0000, Jose Abreu wrote:\n" + "On Wed, 2017-02-22 at 18:19 +0000, Jose Abreu wrote:\n" "> Default clock frequency for ARC PGU does not match any\n" "> existing HDMI mode, instead the default value matches a\n" "> DVI mode. Change the clock frequency to 74.25MHz so that\n" - "> it matches HDMI mode 1280x720 at 60Hz\n" + "> it matches HDMI mode 1280x720@60Hz\n" "> \n" - "> Signed-off-by: Jose Abreu <joabreu at synopsys.com>\n" - "> Cc: Carlos Palminha <palminha at synopsys.com>\n" - "> Cc: Alexey Brodkin <abrodkin at synopsys.com>\n" - "> Cc: Rob Herring <robh+dt at kernel.org>\n" - "> Cc: Vineet Gupta <vgupta at synopsys.com>\n" - "> Cc: devicetree at vger.kernel.org\n" - "> Cc: linux-snps-arc at lists.infradead.org\n" - "> Cc: linux-kernel at vger.kernel.org\n" + "> Signed-off-by: Jose Abreu <joabreu@synopsys.com>\n" + "> Cc: Carlos Palminha <palminha@synopsys.com>\n" + "> Cc: Alexey Brodkin <abrodkin@synopsys.com>\n" + "> Cc: Rob Herring <robh+dt@kernel.org>\n" + "> Cc: Vineet Gupta <vgupta@synopsys.com>\n" + "> Cc: devicetree@vger.kernel.org\n" + "> Cc: linux-snps-arc@lists.infradead.org\n" + "> Cc: linux-kernel@vger.kernel.org\n" "> ---\n" - "> ?arch/arc/boot/dts/axs10x_mb.dtsi | 2 +-\n" - "> ?1 file changed, 1 insertion(+), 1 deletion(-)\n" + "> \302\240arch/arc/boot/dts/axs10x_mb.dtsi | 2 +-\n" + "> \302\2401 file changed, 1 insertion(+), 1 deletion(-)\n" "> \n" "> diff --git a/arch/arc/boot/dts/axs10x_mb.dtsi b/arch/arc/boot/dts/axs10x_mb.dtsi\n" "> index 9d882b1..41cfb29 100644\n" "> --- a/arch/arc/boot/dts/axs10x_mb.dtsi\n" "> +++ b/arch/arc/boot/dts/axs10x_mb.dtsi\n" "> @@ -51,7 +51,7 @@\n" - "> ?\t\t\tpguclk: pguclk {\n" - "> ?\t\t\t\t#clock-cells = <0>;\n" - "> ?\t\t\t\tcompatible = \"fixed-clock\";\n" + "> \302\240\t\t\tpguclk: pguclk {\n" + "> \302\240\t\t\t\t#clock-cells = <0>;\n" + "> \302\240\t\t\t\tcompatible = \"fixed-clock\";\n" "> -\t\t\t\tclock-frequency = <74440000>;\n" "> +\t\t\t\tclock-frequency = <74250000>;\n" - "> ?\t\t\t};\n" - "> ?\t\t};\n" + "> \302\240\t\t\t};\n" + "> \302\240\t\t};\n" "\n" "Looks good to me but I have to note that with this change in place\n" "monitors that don't report [correct] EDID won't work just because\n" "if EDID is missing Linux kernel calculates pixel clock on its\n" - "own and for 1280x720 at 60?it gets 74.40 MHz which obviously differs from\n" - "yours 74.25 MHz and so?arc_pgu_crtc_atomic_check() returns?-EINVAL.\n" + "own and for 1280x720@60\302\240it gets 74.40 MHz which obviously differs from\n" + "yours 74.25 MHz and so\302\240arc_pgu_crtc_atomic_check() returns\302\240-EINVAL.\n" "\n" "But with the fix I just sent, see\n" "http://lists.infradead.org/pipermail/linux-snps-arc/2017-March/002173.html\n" @@ -54,6 +60,10 @@ "\n" "That said...\n" "\n" - Acked-by: Alexey Brodkin <abrodkin at synopsys.com> + "Acked-by: Alexey Brodkin <abrodkin@synopsys.com>\n" + "_______________________________________________\n" + "linux-snps-arc mailing list\n" + "linux-snps-arc@lists.infradead.org\n" + http://lists.infradead.org/mailman/listinfo/linux-snps-arc -ea2448213b65c2d4033bdc1891530abbd59177c6b56e229f8c63a6f5b57af416 +5ccf708f9246b286fec5d29c359048c713934125066ac4c67134795b90e2385c
diff --git a/a/1.txt b/N2/1.txt index 1ff6319..2d74eda 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,41 +1,41 @@ Hi Jose, -On Wed, 2017-02-22@18:19 +0000, Jose Abreu wrote: +On Wed, 2017-02-22 at 18:19 +0000, Jose Abreu wrote: > Default clock frequency for ARC PGU does not match any > existing HDMI mode, instead the default value matches a > DVI mode. Change the clock frequency to 74.25MHz so that -> it matches HDMI mode 1280x720 at 60Hz +> it matches HDMI mode 1280x720@60Hz > -> Signed-off-by: Jose Abreu <joabreu at synopsys.com> -> Cc: Carlos Palminha <palminha at synopsys.com> -> Cc: Alexey Brodkin <abrodkin at synopsys.com> -> Cc: Rob Herring <robh+dt at kernel.org> -> Cc: Vineet Gupta <vgupta at synopsys.com> -> Cc: devicetree at vger.kernel.org -> Cc: linux-snps-arc at lists.infradead.org -> Cc: linux-kernel at vger.kernel.org +> Signed-off-by: Jose Abreu <joabreu@synopsys.com> +> Cc: Carlos Palminha <palminha@synopsys.com> +> Cc: Alexey Brodkin <abrodkin@synopsys.com> +> Cc: Rob Herring <robh+dt@kernel.org> +> Cc: Vineet Gupta <vgupta@synopsys.com> +> Cc: devicetree@vger.kernel.org +> Cc: linux-snps-arc@lists.infradead.org +> Cc: linux-kernel@vger.kernel.org > --- -> ?arch/arc/boot/dts/axs10x_mb.dtsi | 2 +- -> ?1 file changed, 1 insertion(+), 1 deletion(-) +> arch/arc/boot/dts/axs10x_mb.dtsi | 2 +- +> 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/arch/arc/boot/dts/axs10x_mb.dtsi b/arch/arc/boot/dts/axs10x_mb.dtsi > index 9d882b1..41cfb29 100644 > --- a/arch/arc/boot/dts/axs10x_mb.dtsi > +++ b/arch/arc/boot/dts/axs10x_mb.dtsi > @@ -51,7 +51,7 @@ -> ? pguclk: pguclk { -> ? #clock-cells = <0>; -> ? compatible = "fixed-clock"; +> pguclk: pguclk { +> #clock-cells = <0>; +> compatible = "fixed-clock"; > - clock-frequency = <74440000>; > + clock-frequency = <74250000>; -> ? }; -> ? }; +> }; +> }; Looks good to me but I have to note that with this change in place monitors that don't report [correct] EDID won't work just because if EDID is missing Linux kernel calculates pixel clock on its -own and for 1280x720 at 60?it gets 74.40 MHz which obviously differs from -yours 74.25 MHz and so?arc_pgu_crtc_atomic_check() returns?-EINVAL. +own and for 1280x720@60 it gets 74.40 MHz which obviously differs from +yours 74.25 MHz and so arc_pgu_crtc_atomic_check() returns -EINVAL. But with the fix I just sent, see http://lists.infradead.org/pipermail/linux-snps-arc/2017-March/002173.html @@ -46,4 +46,4 @@ support of multiple modes on AXS10x boards. That said... -Acked-by: Alexey Brodkin <abrodkin at synopsys.com> +Acked-by: Alexey Brodkin <abrodkin@synopsys.com> diff --git a/a/content_digest b/N2/content_digest index 76e8bbc..51ebbec 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,49 +1,55 @@ "ref\07ed6deaf1ae6d5819364c793835aa30316077a7e.1487787376.git.joabreu@synopsys.com\0" "ref\016273b3a26ec770f8d169f04f7cb9eb2fac55909.1487787376.git.joabreu@synopsys.com\0" - "From\0Alexey.Brodkin@synopsys.com (Alexey Brodkin)\0" - "Subject\0[PATCH 2/2] arc: axs10x: Fix ARC PGU default clock frequency\0" + "From\0Alexey Brodkin <Alexey.Brodkin@synopsys.com>\0" + "Subject\0Re: [PATCH 2/2] arc: axs10x: Fix ARC PGU default clock frequency\0" "Date\0Thu, 2 Mar 2017 17:35:52 +0000\0" - "To\0linux-snps-arc@lists.infradead.org\0" + "To\0Jose Abreu <Jose.Abreu@synopsys.com>\0" + "Cc\0Carlos Palminha <CARLOS.PALMINHA@synopsys.com>" + linux-kernel@vger.kernel.org <linux-kernel@vger.kernel.org> + Vineet Gupta <Vineet.Gupta1@synopsys.com> + robh+dt@kernel.org <robh+dt@kernel.org> + devicetree@vger.kernel.org <devicetree@vger.kernel.org> + " linux-snps-arc@lists.infradead.org <linux-snps-arc@lists.infradead.org>\0" "\00:1\0" "b\0" "Hi Jose,\n" "\n" - "On Wed, 2017-02-22@18:19 +0000, Jose Abreu wrote:\n" + "On Wed, 2017-02-22 at 18:19 +0000, Jose Abreu wrote:\n" "> Default clock frequency for ARC PGU does not match any\n" "> existing HDMI mode, instead the default value matches a\n" "> DVI mode. Change the clock frequency to 74.25MHz so that\n" - "> it matches HDMI mode 1280x720 at 60Hz\n" + "> it matches HDMI mode 1280x720@60Hz\n" "> \n" - "> Signed-off-by: Jose Abreu <joabreu at synopsys.com>\n" - "> Cc: Carlos Palminha <palminha at synopsys.com>\n" - "> Cc: Alexey Brodkin <abrodkin at synopsys.com>\n" - "> Cc: Rob Herring <robh+dt at kernel.org>\n" - "> Cc: Vineet Gupta <vgupta at synopsys.com>\n" - "> Cc: devicetree at vger.kernel.org\n" - "> Cc: linux-snps-arc at lists.infradead.org\n" - "> Cc: linux-kernel at vger.kernel.org\n" + "> Signed-off-by: Jose Abreu <joabreu@synopsys.com>\n" + "> Cc: Carlos Palminha <palminha@synopsys.com>\n" + "> Cc: Alexey Brodkin <abrodkin@synopsys.com>\n" + "> Cc: Rob Herring <robh+dt@kernel.org>\n" + "> Cc: Vineet Gupta <vgupta@synopsys.com>\n" + "> Cc: devicetree@vger.kernel.org\n" + "> Cc: linux-snps-arc@lists.infradead.org\n" + "> Cc: linux-kernel@vger.kernel.org\n" "> ---\n" - "> ?arch/arc/boot/dts/axs10x_mb.dtsi | 2 +-\n" - "> ?1 file changed, 1 insertion(+), 1 deletion(-)\n" + "> \302\240arch/arc/boot/dts/axs10x_mb.dtsi | 2 +-\n" + "> \302\2401 file changed, 1 insertion(+), 1 deletion(-)\n" "> \n" "> diff --git a/arch/arc/boot/dts/axs10x_mb.dtsi b/arch/arc/boot/dts/axs10x_mb.dtsi\n" "> index 9d882b1..41cfb29 100644\n" "> --- a/arch/arc/boot/dts/axs10x_mb.dtsi\n" "> +++ b/arch/arc/boot/dts/axs10x_mb.dtsi\n" "> @@ -51,7 +51,7 @@\n" - "> ?\t\t\tpguclk: pguclk {\n" - "> ?\t\t\t\t#clock-cells = <0>;\n" - "> ?\t\t\t\tcompatible = \"fixed-clock\";\n" + "> \302\240\t\t\tpguclk: pguclk {\n" + "> \302\240\t\t\t\t#clock-cells = <0>;\n" + "> \302\240\t\t\t\tcompatible = \"fixed-clock\";\n" "> -\t\t\t\tclock-frequency = <74440000>;\n" "> +\t\t\t\tclock-frequency = <74250000>;\n" - "> ?\t\t\t};\n" - "> ?\t\t};\n" + "> \302\240\t\t\t};\n" + "> \302\240\t\t};\n" "\n" "Looks good to me but I have to note that with this change in place\n" "monitors that don't report [correct] EDID won't work just because\n" "if EDID is missing Linux kernel calculates pixel clock on its\n" - "own and for 1280x720 at 60?it gets 74.40 MHz which obviously differs from\n" - "yours 74.25 MHz and so?arc_pgu_crtc_atomic_check() returns?-EINVAL.\n" + "own and for 1280x720@60\302\240it gets 74.40 MHz which obviously differs from\n" + "yours 74.25 MHz and so\302\240arc_pgu_crtc_atomic_check() returns\302\240-EINVAL.\n" "\n" "But with the fix I just sent, see\n" "http://lists.infradead.org/pipermail/linux-snps-arc/2017-March/002173.html\n" @@ -54,6 +60,6 @@ "\n" "That said...\n" "\n" - Acked-by: Alexey Brodkin <abrodkin at synopsys.com> + Acked-by: Alexey Brodkin <abrodkin@synopsys.com> -ea2448213b65c2d4033bdc1891530abbd59177c6b56e229f8c63a6f5b57af416 +4ae2027ce4002b9c97a78614eb1bbe67e6a5bc14521c2053e1d9dd26a507f6bf
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.