| Message ID | 1445860796-23045-1-git-send-email-a.hajda@samsung.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On 26.10.2015 20:59, Andrzej Hajda wrote: > DECON-TV(Display and Enhancement Controller for TV) is a variation > of DECON IP. Its main purpose is to produce video stream for HDMI IP. > > Signed-off-by: Andrzej Hajda <a.hajda@samsung.com> > --- > Hi Krzysztof, > > I have decided to skip cleanup part as it would require more work, > not related to this patchset. > > Regards > Andrzej > > Documentation/devicetree/bindings/video/exynos5433-decon.txt | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) Thanks for splitting this, looks good: Reviewed-by: Krzysztof Kozlowski <k.kozlowski@samsung.com> Best regards, Krzysztof -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, Oct 26, 2015 at 12:59:56PM +0100, Andrzej Hajda wrote: > DECON-TV(Display and Enhancement Controller for TV) is a variation > of DECON IP. Its main purpose is to produce video stream for HDMI IP. > > Signed-off-by: Andrzej Hajda <a.hajda@samsung.com> Acked-by: Rob Herring <robh@kernel.org> > --- > Hi Krzysztof, > > I have decided to skip cleanup part as it would require more work, > not related to this patchset. > > Regards > Andrzej > > Documentation/devicetree/bindings/video/exynos5433-decon.txt | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/video/exynos5433-decon.txt b/Documentation/devicetree/bindings/video/exynos5433-decon.txt > index 3dff78b..c9fd7b3 100644 > --- a/Documentation/devicetree/bindings/video/exynos5433-decon.txt > +++ b/Documentation/devicetree/bindings/video/exynos5433-decon.txt > @@ -5,7 +5,8 @@ Exynos series of SoCs which transfers the image data from a video memory > buffer to an external LCD interface. > > Required properties: > -- compatible: value should be "samsung,exynos5433-decon"; > +- compatible: value should be one of: > + "samsung,exynos5433-decon", "samsung,exynos5433-decon-tv"; > - reg: physical base address and length of the DECON registers set. > - interrupts: should contain a list of all DECON IP block interrupts in the > order: VSYNC, LCD_SYSTEM. The interrupt specifier format > -- > 1.9.1 > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Inki, It seems this patch and 04/10 did not get picked up yet. Could you merge it? Regards Andrzej On 10/26/2015 12:59 PM, Andrzej Hajda wrote: > DECON-TV(Display and Enhancement Controller for TV) is a variation > of DECON IP. Its main purpose is to produce video stream for HDMI IP. > > Signed-off-by: Andrzej Hajda <a.hajda@samsung.com> > --- > Hi Krzysztof, > > I have decided to skip cleanup part as it would require more work, > not related to this patchset. > > Regards > Andrzej > > Documentation/devicetree/bindings/video/exynos5433-decon.txt | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/Documentation/devicetree/bindings/video/exynos5433-decon.txt b/Documentation/devicetree/bindings/video/exynos5433-decon.txt > index 3dff78b..c9fd7b3 100644 > --- a/Documentation/devicetree/bindings/video/exynos5433-decon.txt > +++ b/Documentation/devicetree/bindings/video/exynos5433-decon.txt > @@ -5,7 +5,8 @@ Exynos series of SoCs which transfers the image data from a video memory > buffer to an external LCD interface. > > Required properties: > -- compatible: value should be "samsung,exynos5433-decon"; > +- compatible: value should be one of: > + "samsung,exynos5433-decon", "samsung,exynos5433-decon-tv"; > - reg: physical base address and length of the DECON registers set. > - interrupts: should contain a list of all DECON IP block interrupts in the > order: VSYNC, LCD_SYSTEM. The interrupt specifier format -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
+ Rob Herring 2016? 01? 14? 19:36? Andrzej Hajda ?(?) ? ?: > Hi Inki, > > It seems this patch and 04/10 did not get picked up yet. > Could you merge it? Yes, I can. However, 04/10 is required for acked-by from dt or SoC maintainer. Krzysztof and Rob, could you give me acked-by? Thanks, Inki Dae > > Regards > Andrzej > > On 10/26/2015 12:59 PM, Andrzej Hajda wrote: >> DECON-TV(Display and Enhancement Controller for TV) is a variation >> of DECON IP. Its main purpose is to produce video stream for HDMI IP. >> >> Signed-off-by: Andrzej Hajda <a.hajda@samsung.com> >> --- >> Hi Krzysztof, >> >> I have decided to skip cleanup part as it would require more work, >> not related to this patchset. >> >> Regards >> Andrzej >> >> Documentation/devicetree/bindings/video/exynos5433-decon.txt | 3 ++- >> 1 file changed, 2 insertions(+), 1 deletion(-) >> >> diff --git a/Documentation/devicetree/bindings/video/exynos5433-decon.txt b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >> index 3dff78b..c9fd7b3 100644 >> --- a/Documentation/devicetree/bindings/video/exynos5433-decon.txt >> +++ b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >> @@ -5,7 +5,8 @@ Exynos series of SoCs which transfers the image data from a video memory >> buffer to an external LCD interface. >> >> Required properties: >> -- compatible: value should be "samsung,exynos5433-decon"; >> +- compatible: value should be one of: >> + "samsung,exynos5433-decon", "samsung,exynos5433-decon-tv"; >> - reg: physical base address and length of the DECON registers set. >> - interrupts: should contain a list of all DECON IP block interrupts in the >> order: VSYNC, LCD_SYSTEM. The interrupt specifier format > > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 14.01.2016 19:49, Inki Dae wrote: > + Rob Herring > > 2016? 01? 14? 19:36? Andrzej Hajda ?(?) ? ?: >> Hi Inki, >> >> It seems this patch and 04/10 did not get picked up yet. >> Could you merge it? > > Yes, I can. However, 04/10 is required for acked-by from dt or SoC maintainer. > Krzysztof and Rob, could you give me acked-by? > > Thanks, > Inki Dae My ack is really not required, because this is a binding for a specific driver from video subsystem. It does not fall into the arm/mach-exynos nor the dts/*exynos*. Anyway I reviewed the patch on 28th of October and Rob acked it on 13th of November. Best regards, Krzysztof >> >> Regards >> Andrzej >> >> On 10/26/2015 12:59 PM, Andrzej Hajda wrote: >>> DECON-TV(Display and Enhancement Controller for TV) is a variation >>> of DECON IP. Its main purpose is to produce video stream for HDMI IP. >>> >>> Signed-off-by: Andrzej Hajda <a.hajda@samsung.com> >>> --- >>> Hi Krzysztof, >>> >>> I have decided to skip cleanup part as it would require more work, >>> not related to this patchset. >>> >>> Regards >>> Andrzej >>> >>> Documentation/devicetree/bindings/video/exynos5433-decon.txt | 3 ++- >>> 1 file changed, 2 insertions(+), 1 deletion(-) >>> >>> diff --git a/Documentation/devicetree/bindings/video/exynos5433-decon.txt b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>> index 3dff78b..c9fd7b3 100644 >>> --- a/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>> +++ b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>> @@ -5,7 +5,8 @@ Exynos series of SoCs which transfers the image data from a video memory >>> buffer to an external LCD interface. >>> >>> Required properties: >>> -- compatible: value should be "samsung,exynos5433-decon"; >>> +- compatible: value should be one of: >>> + "samsung,exynos5433-decon", "samsung,exynos5433-decon-tv"; >>> - reg: physical base address and length of the DECON registers set. >>> - interrupts: should contain a list of all DECON IP block interrupts in the >>> order: VSYNC, LCD_SYSTEM. The interrupt specifier format >> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> > -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
2016? 01? 18? 08:45? Krzysztof Kozlowski ?(?) ? ?: > On 14.01.2016 19:49, Inki Dae wrote: >> + Rob Herring >> >> 2016? 01? 14? 19:36? Andrzej Hajda ?(?) ? ?: >>> Hi Inki, >>> >>> It seems this patch and 04/10 did not get picked up yet. >>> Could you merge it? >> >> Yes, I can. However, 04/10 is required for acked-by from dt or SoC maintainer. >> Krzysztof and Rob, could you give me acked-by? >> >> Thanks, >> Inki Dae > > My ack is really not required, because this is a binding for a specific > driver from video subsystem. It does not fall into the arm/mach-exynos > nor the dts/*exynos*. > > Anyway I reviewed the patch on 28th of October and Rob acked it on 13th > of November. I meant 04/10 not 09/10 patch. :) Thanks, Inki Dae > > Best regards, > Krzysztof > > >>> >>> Regards >>> Andrzej >>> >>> On 10/26/2015 12:59 PM, Andrzej Hajda wrote: >>>> DECON-TV(Display and Enhancement Controller for TV) is a variation >>>> of DECON IP. Its main purpose is to produce video stream for HDMI IP. >>>> >>>> Signed-off-by: Andrzej Hajda <a.hajda@samsung.com> >>>> --- >>>> Hi Krzysztof, >>>> >>>> I have decided to skip cleanup part as it would require more work, >>>> not related to this patchset. >>>> >>>> Regards >>>> Andrzej >>>> >>>> Documentation/devicetree/bindings/video/exynos5433-decon.txt | 3 ++- >>>> 1 file changed, 2 insertions(+), 1 deletion(-) >>>> >>>> diff --git a/Documentation/devicetree/bindings/video/exynos5433-decon.txt b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>>> index 3dff78b..c9fd7b3 100644 >>>> --- a/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>>> +++ b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>>> @@ -5,7 +5,8 @@ Exynos series of SoCs which transfers the image data from a video memory >>>> buffer to an external LCD interface. >>>> >>>> Required properties: >>>> -- compatible: value should be "samsung,exynos5433-decon"; >>>> +- compatible: value should be one of: >>>> + "samsung,exynos5433-decon", "samsung,exynos5433-decon-tv"; >>>> - reg: physical base address and length of the DECON registers set. >>>> - interrupts: should contain a list of all DECON IP block interrupts in the >>>> order: VSYNC, LCD_SYSTEM. The interrupt specifier format >>> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> >>> >> > > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Inki, On 01/18/2016 10:54 AM, Inki Dae wrote: > > 2016? 01? 18? 08:45? Krzysztof Kozlowski ?(?) ? ?: >> On 14.01.2016 19:49, Inki Dae wrote: >>> + Rob Herring >>> >>> 2016? 01? 14? 19:36? Andrzej Hajda ?(?) ? ?: >>>> Hi Inki, >>>> >>>> It seems this patch and 04/10 did not get picked up yet. >>>> Could you merge it? >>> Yes, I can. However, 04/10 is required for acked-by from dt or SoC maintainer. >>> Krzysztof and Rob, could you give me acked-by? >>> >>> Thanks, >>> Inki Dae >> My ack is really not required, because this is a binding for a specific >> driver from video subsystem. It does not fall into the arm/mach-exynos >> nor the dts/*exynos*. >> >> Anyway I reviewed the patch on 28th of October and Rob acked it on 13th >> of November. > I meant 04/10 not 09/10 patch. :) There is document in kernel with clear explanation about the subject: Documentation/devicetree/bindings/submitting-patches.txt. See especially paragraph II.2 [1]. [1]: Documentation/devicetree/bindings/submitting-patches.txt#L50 Regards Andrzej > > Thanks, > Inki Dae > >> Best regards, >> Krzysztof >> >> >>>> Regards >>>> Andrzej >>>> >>>> On 10/26/2015 12:59 PM, Andrzej Hajda wrote: >>>>> DECON-TV(Display and Enhancement Controller for TV) is a variation >>>>> of DECON IP. Its main purpose is to produce video stream for HDMI IP. >>>>> >>>>> Signed-off-by: Andrzej Hajda <a.hajda@samsung.com> >>>>> --- >>>>> Hi Krzysztof, >>>>> >>>>> I have decided to skip cleanup part as it would require more work, >>>>> not related to this patchset. >>>>> >>>>> Regards >>>>> Andrzej >>>>> >>>>> Documentation/devicetree/bindings/video/exynos5433-decon.txt | 3 ++- >>>>> 1 file changed, 2 insertions(+), 1 deletion(-) >>>>> >>>>> diff --git a/Documentation/devicetree/bindings/video/exynos5433-decon.txt b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>>>> index 3dff78b..c9fd7b3 100644 >>>>> --- a/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>>>> +++ b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>>>> @@ -5,7 +5,8 @@ Exynos series of SoCs which transfers the image data from a video memory >>>>> buffer to an external LCD interface. >>>>> >>>>> Required properties: >>>>> -- compatible: value should be "samsung,exynos5433-decon"; >>>>> +- compatible: value should be one of: >>>>> + "samsung,exynos5433-decon", "samsung,exynos5433-decon-tv"; >>>>> - reg: physical base address and length of the DECON registers set. >>>>> - interrupts: should contain a list of all DECON IP block interrupts in the >>>>> order: VSYNC, LCD_SYSTEM. The interrupt specifier format >>>> -- >>>> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in >>>> the body of a message to majordomo@vger.kernel.org >>>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>>> >>>> >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in >> the body of a message to majordomo@vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
2016? 01? 18? 19:12? Andrzej Hajda ?(?) ? ?: > Hi Inki, > > On 01/18/2016 10:54 AM, Inki Dae wrote: >> >> 2016? 01? 18? 08:45? Krzysztof Kozlowski ?(?) ? ?: >>> On 14.01.2016 19:49, Inki Dae wrote: >>>> + Rob Herring >>>> >>>> 2016? 01? 14? 19:36? Andrzej Hajda ?(?) ? ?: >>>>> Hi Inki, >>>>> >>>>> It seems this patch and 04/10 did not get picked up yet. >>>>> Could you merge it? >>>> Yes, I can. However, 04/10 is required for acked-by from dt or SoC maintainer. >>>> Krzysztof and Rob, could you give me acked-by? >>>> >>>> Thanks, >>>> Inki Dae >>> My ack is really not required, because this is a binding for a specific >>> driver from video subsystem. It does not fall into the arm/mach-exynos >>> nor the dts/*exynos*. >>> >>> Anyway I reviewed the patch on 28th of October and Rob acked it on 13th >>> of November. >> I meant 04/10 not 09/10 patch. :) > > There is document in kernel with clear explanation about the subject: > Documentation/devicetree/bindings/submitting-patches.txt. > See especially paragraph II.2 [1]. > > [1]: Documentation/devicetree/bindings/submitting-patches.txt#L50 Got it. The document says, "For driver (not subsystem) bindings: If you are comfortable with the binding, and it hasn't received an Acked-by from the devicetree maintainers after a few weeks, go ahead and take it." Thanks, Inki Dae > > Regards > Andrzej > >> >> Thanks, >> Inki Dae >> >>> Best regards, >>> Krzysztof >>> >>> >>>>> Regards >>>>> Andrzej >>>>> >>>>> On 10/26/2015 12:59 PM, Andrzej Hajda wrote: >>>>>> DECON-TV(Display and Enhancement Controller for TV) is a variation >>>>>> of DECON IP. Its main purpose is to produce video stream for HDMI IP. >>>>>> >>>>>> Signed-off-by: Andrzej Hajda <a.hajda@samsung.com> >>>>>> --- >>>>>> Hi Krzysztof, >>>>>> >>>>>> I have decided to skip cleanup part as it would require more work, >>>>>> not related to this patchset. >>>>>> >>>>>> Regards >>>>>> Andrzej >>>>>> >>>>>> Documentation/devicetree/bindings/video/exynos5433-decon.txt | 3 ++- >>>>>> 1 file changed, 2 insertions(+), 1 deletion(-) >>>>>> >>>>>> diff --git a/Documentation/devicetree/bindings/video/exynos5433-decon.txt b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>>>>> index 3dff78b..c9fd7b3 100644 >>>>>> --- a/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>>>>> +++ b/Documentation/devicetree/bindings/video/exynos5433-decon.txt >>>>>> @@ -5,7 +5,8 @@ Exynos series of SoCs which transfers the image data from a video memory >>>>>> buffer to an external LCD interface. >>>>>> >>>>>> Required properties: >>>>>> -- compatible: value should be "samsung,exynos5433-decon"; >>>>>> +- compatible: value should be one of: >>>>>> + "samsung,exynos5433-decon", "samsung,exynos5433-decon-tv"; >>>>>> - reg: physical base address and length of the DECON registers set. >>>>>> - interrupts: should contain a list of all DECON IP block interrupts in the >>>>>> order: VSYNC, LCD_SYSTEM. The interrupt specifier format >>>>> -- >>>>> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in >>>>> the body of a message to majordomo@vger.kernel.org >>>>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>>>> >>>>> >>> -- >>> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in >>> the body of a message to majordomo@vger.kernel.org >>> More majordomo info at http://vger.kernel.org/majordomo-info.html >>> >>> > > -- > To unsubscribe from this list: send the line "unsubscribe devicetree" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > > -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/video/exynos5433-decon.txt b/Documentation/devicetree/bindings/video/exynos5433-decon.txt index 3dff78b..c9fd7b3 100644 --- a/Documentation/devicetree/bindings/video/exynos5433-decon.txt +++ b/Documentation/devicetree/bindings/video/exynos5433-decon.txt @@ -5,7 +5,8 @@ Exynos series of SoCs which transfers the image data from a video memory buffer to an external LCD interface. Required properties: -- compatible: value should be "samsung,exynos5433-decon"; +- compatible: value should be one of: + "samsung,exynos5433-decon", "samsung,exynos5433-decon-tv"; - reg: physical base address and length of the DECON registers set. - interrupts: should contain a list of all DECON IP block interrupts in the order: VSYNC, LCD_SYSTEM. The interrupt specifier format
DECON-TV(Display and Enhancement Controller for TV) is a variation of DECON IP. Its main purpose is to produce video stream for HDMI IP. Signed-off-by: Andrzej Hajda <a.hajda@samsung.com> --- Hi Krzysztof, I have decided to skip cleanup part as it would require more work, not related to this patchset. Regards Andrzej Documentation/devicetree/bindings/video/exynos5433-decon.txt | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)