* [PATCH v10 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel @ 2020-03-27 7:36 ` Harigovindan P 0 siblings, 0 replies; 14+ messages in thread From: Harigovindan P @ 2020-03-27 7:36 UTC (permalink / raw) To: dri-devel, linux-arm-msm, freedreno, devicetree Cc: Harigovindan P, robdclark, seanpaul, sean Adding support for visionox rm69299 panel driver and adding bindings for the same panel. Harigovindan P (2): dt-bindings: display: add visionox rm69299 panel variant drm/panel: add support for rm69299 visionox panel driver .../display/panel/visionox,rm69299.yaml | 58 ++++ drivers/gpu/drm/panel/Kconfig | 8 + drivers/gpu/drm/panel/Makefile | 1 + .../gpu/drm/panel/panel-visionox-rm69299.c | 295 ++++++++++++++++++ 4 files changed, 362 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml create mode 100644 drivers/gpu/drm/panel/panel-visionox-rm69299.c -- 2.25.1 ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v10 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel @ 2020-03-27 7:36 ` Harigovindan P 0 siblings, 0 replies; 14+ messages in thread From: Harigovindan P @ 2020-03-27 7:36 UTC (permalink / raw) To: dri-devel, linux-arm-msm, freedreno, devicetree Cc: sean, seanpaul, Harigovindan P Adding support for visionox rm69299 panel driver and adding bindings for the same panel. Harigovindan P (2): dt-bindings: display: add visionox rm69299 panel variant drm/panel: add support for rm69299 visionox panel driver .../display/panel/visionox,rm69299.yaml | 58 ++++ drivers/gpu/drm/panel/Kconfig | 8 + drivers/gpu/drm/panel/Makefile | 1 + .../gpu/drm/panel/panel-visionox-rm69299.c | 295 ++++++++++++++++++ 4 files changed, 362 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml create mode 100644 drivers/gpu/drm/panel/panel-visionox-rm69299.c -- 2.25.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* [PATCH v10 1/2] dt-bindings: display: add visionox rm69299 panel variant 2020-03-27 7:36 ` Harigovindan P @ 2020-03-27 7:36 ` Harigovindan P -1 siblings, 0 replies; 14+ messages in thread From: Harigovindan P @ 2020-03-27 7:36 UTC (permalink / raw) To: dri-devel, linux-arm-msm, freedreno, devicetree Cc: Harigovindan P, robdclark, seanpaul, sean Add bindings for visionox rm69299 panel. Signed-off-by: Harigovindan P <harigovi@codeaurora.org> --- Changes in v2: - Removed unwanted properties from description. - Creating source files without execute permissions(Rob Herring). Changes in v3: - Changing txt file into yaml Changes in v4: - Updating license identifier. - Moving yaml file inside panel directory. - Removing pinctrl entries. - Adding documentation for reset-gpios. Changes in v5: - No changes. Updated 2/2 Patch. Changes in v6: - Removing patternProperties. - Added " |" after description. - Setting port and reset-gpios to true. - Removing @ae94000 for dsi node. Changes in v7: - Added reg property. Changes in v8: - Rearranged additionalProperties. - Dropping improper reg property. Changes in v9: - Adding additionalProperties at the same level as 'properties' - Adding properties for "ports" which includes: -> #address-cells -> #size-cells -> port@0 Changes in v10: - Changed file mode - Making port as required - Fixing example by removing ports and unit address in port - Removing indentation for 'additionalProperties' .../display/panel/visionox,rm69299.yaml | 58 +++++++++++++++++++ 1 file changed, 58 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml diff --git a/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml b/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml new file mode 100644 index 000000000000..1a44caae5f9b --- /dev/null +++ b/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml @@ -0,0 +1,58 @@ +# SPDX-License-Identifier: GPL-2.0-only or BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/display/panel/visionox,rm69299.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Visionox model RM69299 Panels Device Tree Bindings. + +maintainers: + - Harigovindan P <harigovi@codeaurora.org> + +description: | + This binding is for display panels using a Visionox RM692999 panel. + +allOf: + - $ref: panel-common.yaml# + +properties: + compatible: + const: visionox,rm69299-1080p-display + + vdda-supply: + description: | + Phandle of the regulator that provides the vdda supply voltage. + + vdd3p3-supply: + description: | + Phandle of the regulator that provides the vdd3p3 supply voltage. + + port: true + + reset-gpios: true + +additionalProperties: false + +required: + - compatible + - vdda-supply + - vdd3p3-supply + - reset-gpios + - port + +examples: + - | + panel { + compatible = "visionox,rm69299-1080p-display"; + + vdda-supply = <&src_pp1800_l8c>; + vdd3p3-supply = <&src_pp2800_l18a>; + + reset-gpios = <&pm6150l_gpio 3 0>; + port { + panel0_in: endpoint { + remote-endpoint = <&dsi0_out>; + }; + }; + }; +... -- 2.25.1 ^ permalink raw reply related [flat|nested] 14+ messages in thread
* [PATCH v10 1/2] dt-bindings: display: add visionox rm69299 panel variant @ 2020-03-27 7:36 ` Harigovindan P 0 siblings, 0 replies; 14+ messages in thread From: Harigovindan P @ 2020-03-27 7:36 UTC (permalink / raw) To: dri-devel, linux-arm-msm, freedreno, devicetree Cc: sean, seanpaul, Harigovindan P Add bindings for visionox rm69299 panel. Signed-off-by: Harigovindan P <harigovi@codeaurora.org> --- Changes in v2: - Removed unwanted properties from description. - Creating source files without execute permissions(Rob Herring). Changes in v3: - Changing txt file into yaml Changes in v4: - Updating license identifier. - Moving yaml file inside panel directory. - Removing pinctrl entries. - Adding documentation for reset-gpios. Changes in v5: - No changes. Updated 2/2 Patch. Changes in v6: - Removing patternProperties. - Added " |" after description. - Setting port and reset-gpios to true. - Removing @ae94000 for dsi node. Changes in v7: - Added reg property. Changes in v8: - Rearranged additionalProperties. - Dropping improper reg property. Changes in v9: - Adding additionalProperties at the same level as 'properties' - Adding properties for "ports" which includes: -> #address-cells -> #size-cells -> port@0 Changes in v10: - Changed file mode - Making port as required - Fixing example by removing ports and unit address in port - Removing indentation for 'additionalProperties' .../display/panel/visionox,rm69299.yaml | 58 +++++++++++++++++++ 1 file changed, 58 insertions(+) create mode 100644 Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml diff --git a/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml b/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml new file mode 100644 index 000000000000..1a44caae5f9b --- /dev/null +++ b/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml @@ -0,0 +1,58 @@ +# SPDX-License-Identifier: GPL-2.0-only or BSD-2-Clause +%YAML 1.2 +--- +$id: http://devicetree.org/schemas/display/panel/visionox,rm69299.yaml# +$schema: http://devicetree.org/meta-schemas/core.yaml# + +title: Visionox model RM69299 Panels Device Tree Bindings. + +maintainers: + - Harigovindan P <harigovi@codeaurora.org> + +description: | + This binding is for display panels using a Visionox RM692999 panel. + +allOf: + - $ref: panel-common.yaml# + +properties: + compatible: + const: visionox,rm69299-1080p-display + + vdda-supply: + description: | + Phandle of the regulator that provides the vdda supply voltage. + + vdd3p3-supply: + description: | + Phandle of the regulator that provides the vdd3p3 supply voltage. + + port: true + + reset-gpios: true + +additionalProperties: false + +required: + - compatible + - vdda-supply + - vdd3p3-supply + - reset-gpios + - port + +examples: + - | + panel { + compatible = "visionox,rm69299-1080p-display"; + + vdda-supply = <&src_pp1800_l8c>; + vdd3p3-supply = <&src_pp2800_l18a>; + + reset-gpios = <&pm6150l_gpio 3 0>; + port { + panel0_in: endpoint { + remote-endpoint = <&dsi0_out>; + }; + }; + }; +... -- 2.25.1 _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH v10 1/2] dt-bindings: display: add visionox rm69299 panel variant 2020-03-27 7:36 ` Harigovindan P @ 2020-03-28 20:40 ` Sam Ravnborg -1 siblings, 0 replies; 14+ messages in thread From: Sam Ravnborg @ 2020-03-28 20:40 UTC (permalink / raw) To: Harigovindan P Cc: dri-devel, linux-arm-msm, freedreno, devicetree, sean, seanpaul Hi Harigovindan On Fri, Mar 27, 2020 at 01:06:35PM +0530, Harigovindan P wrote: > Add bindings for visionox rm69299 panel. > > Signed-off-by: Harigovindan P <harigovi@codeaurora.org> > --- > > Changes in v2: > - Removed unwanted properties from description. > - Creating source files without execute permissions(Rob Herring). > Changes in v3: > - Changing txt file into yaml > Changes in v4: > - Updating license identifier. > - Moving yaml file inside panel directory. > - Removing pinctrl entries. > - Adding documentation for reset-gpios. > Changes in v5: > - No changes. Updated 2/2 Patch. > Changes in v6: > - Removing patternProperties. > - Added " |" after description. > - Setting port and reset-gpios to true. > - Removing @ae94000 for dsi node. > Changes in v7: > - Added reg property. > Changes in v8: > - Rearranged additionalProperties. > - Dropping improper reg property. > Changes in v9: > - Adding additionalProperties at the same level as > 'properties' > - Adding properties for "ports" which includes: > -> #address-cells > -> #size-cells > -> port@0 > Changes in v10: > - Changed file mode > - Making port as required > - Fixing example by removing ports and unit address in port > - Removing indentation for 'additionalProperties' Fixed ident in a few places (one space => two sapces), and applied to drm-misc-next. Sam > > .../display/panel/visionox,rm69299.yaml | 58 +++++++++++++++++++ > 1 file changed, 58 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml > > diff --git a/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml b/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml > new file mode 100644 > index 000000000000..1a44caae5f9b > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml > @@ -0,0 +1,58 @@ > +# SPDX-License-Identifier: GPL-2.0-only or BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/display/panel/visionox,rm69299.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Visionox model RM69299 Panels Device Tree Bindings. > + > +maintainers: > + - Harigovindan P <harigovi@codeaurora.org> > + > +description: | > + This binding is for display panels using a Visionox RM692999 panel. > + > +allOf: > + - $ref: panel-common.yaml# > + > +properties: > + compatible: > + const: visionox,rm69299-1080p-display > + > + vdda-supply: > + description: | > + Phandle of the regulator that provides the vdda supply voltage. > + > + vdd3p3-supply: > + description: | > + Phandle of the regulator that provides the vdd3p3 supply voltage. > + > + port: true > + > + reset-gpios: true > + > +additionalProperties: false > + > +required: > + - compatible > + - vdda-supply > + - vdd3p3-supply > + - reset-gpios > + - port > + > +examples: > + - | > + panel { > + compatible = "visionox,rm69299-1080p-display"; > + > + vdda-supply = <&src_pp1800_l8c>; > + vdd3p3-supply = <&src_pp2800_l18a>; > + > + reset-gpios = <&pm6150l_gpio 3 0>; > + port { > + panel0_in: endpoint { > + remote-endpoint = <&dsi0_out>; > + }; > + }; > + }; > +... > -- > 2.25.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v10 1/2] dt-bindings: display: add visionox rm69299 panel variant @ 2020-03-28 20:40 ` Sam Ravnborg 0 siblings, 0 replies; 14+ messages in thread From: Sam Ravnborg @ 2020-03-28 20:40 UTC (permalink / raw) To: Harigovindan P Cc: freedreno, devicetree, linux-arm-msm, dri-devel, seanpaul, sean Hi Harigovindan On Fri, Mar 27, 2020 at 01:06:35PM +0530, Harigovindan P wrote: > Add bindings for visionox rm69299 panel. > > Signed-off-by: Harigovindan P <harigovi@codeaurora.org> > --- > > Changes in v2: > - Removed unwanted properties from description. > - Creating source files without execute permissions(Rob Herring). > Changes in v3: > - Changing txt file into yaml > Changes in v4: > - Updating license identifier. > - Moving yaml file inside panel directory. > - Removing pinctrl entries. > - Adding documentation for reset-gpios. > Changes in v5: > - No changes. Updated 2/2 Patch. > Changes in v6: > - Removing patternProperties. > - Added " |" after description. > - Setting port and reset-gpios to true. > - Removing @ae94000 for dsi node. > Changes in v7: > - Added reg property. > Changes in v8: > - Rearranged additionalProperties. > - Dropping improper reg property. > Changes in v9: > - Adding additionalProperties at the same level as > 'properties' > - Adding properties for "ports" which includes: > -> #address-cells > -> #size-cells > -> port@0 > Changes in v10: > - Changed file mode > - Making port as required > - Fixing example by removing ports and unit address in port > - Removing indentation for 'additionalProperties' Fixed ident in a few places (one space => two sapces), and applied to drm-misc-next. Sam > > .../display/panel/visionox,rm69299.yaml | 58 +++++++++++++++++++ > 1 file changed, 58 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml > > diff --git a/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml b/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml > new file mode 100644 > index 000000000000..1a44caae5f9b > --- /dev/null > +++ b/Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml > @@ -0,0 +1,58 @@ > +# SPDX-License-Identifier: GPL-2.0-only or BSD-2-Clause > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/display/panel/visionox,rm69299.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Visionox model RM69299 Panels Device Tree Bindings. > + > +maintainers: > + - Harigovindan P <harigovi@codeaurora.org> > + > +description: | > + This binding is for display panels using a Visionox RM692999 panel. > + > +allOf: > + - $ref: panel-common.yaml# > + > +properties: > + compatible: > + const: visionox,rm69299-1080p-display > + > + vdda-supply: > + description: | > + Phandle of the regulator that provides the vdda supply voltage. > + > + vdd3p3-supply: > + description: | > + Phandle of the regulator that provides the vdd3p3 supply voltage. > + > + port: true > + > + reset-gpios: true > + > +additionalProperties: false > + > +required: > + - compatible > + - vdda-supply > + - vdd3p3-supply > + - reset-gpios > + - port > + > +examples: > + - | > + panel { > + compatible = "visionox,rm69299-1080p-display"; > + > + vdda-supply = <&src_pp1800_l8c>; > + vdd3p3-supply = <&src_pp2800_l18a>; > + > + reset-gpios = <&pm6150l_gpio 3 0>; > + port { > + panel0_in: endpoint { > + remote-endpoint = <&dsi0_out>; > + }; > + }; > + }; > +... > -- > 2.25.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v10 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel 2020-03-27 7:36 ` Harigovindan P @ 2020-03-28 20:40 ` Sam Ravnborg -1 siblings, 0 replies; 14+ messages in thread From: Sam Ravnborg @ 2020-03-28 20:40 UTC (permalink / raw) To: Harigovindan P Cc: dri-devel, linux-arm-msm, freedreno, devicetree, sean, seanpaul Hi Harigovindan On Fri, Mar 27, 2020 at 01:06:34PM +0530, Harigovindan P wrote: > Adding support for visionox rm69299 panel driver and adding bindings for the same panel. > > Harigovindan P (2): > dt-bindings: display: add visionox rm69299 panel variant > drm/panel: add support for rm69299 visionox panel driver I have only the first patch, which is now applied. Please resend second patch as it is lost somewhere. Sam > > .../display/panel/visionox,rm69299.yaml | 58 ++++ > drivers/gpu/drm/panel/Kconfig | 8 + > drivers/gpu/drm/panel/Makefile | 1 + > .../gpu/drm/panel/panel-visionox-rm69299.c | 295 ++++++++++++++++++ > 4 files changed, 362 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml > create mode 100644 drivers/gpu/drm/panel/panel-visionox-rm69299.c > > -- > 2.25.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v10 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel @ 2020-03-28 20:40 ` Sam Ravnborg 0 siblings, 0 replies; 14+ messages in thread From: Sam Ravnborg @ 2020-03-28 20:40 UTC (permalink / raw) To: Harigovindan P Cc: freedreno, devicetree, linux-arm-msm, dri-devel, seanpaul, sean Hi Harigovindan On Fri, Mar 27, 2020 at 01:06:34PM +0530, Harigovindan P wrote: > Adding support for visionox rm69299 panel driver and adding bindings for the same panel. > > Harigovindan P (2): > dt-bindings: display: add visionox rm69299 panel variant > drm/panel: add support for rm69299 visionox panel driver I have only the first patch, which is now applied. Please resend second patch as it is lost somewhere. Sam > > .../display/panel/visionox,rm69299.yaml | 58 ++++ > drivers/gpu/drm/panel/Kconfig | 8 + > drivers/gpu/drm/panel/Makefile | 1 + > .../gpu/drm/panel/panel-visionox-rm69299.c | 295 ++++++++++++++++++ > 4 files changed, 362 insertions(+) > create mode 100644 Documentation/devicetree/bindings/display/panel/visionox,rm69299.yaml > create mode 100644 drivers/gpu/drm/panel/panel-visionox-rm69299.c > > -- > 2.25.1 > > _______________________________________________ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v10 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel 2020-03-28 20:40 ` Sam Ravnborg @ 2020-03-29 17:44 ` Matthias Kaehlcke -1 siblings, 0 replies; 14+ messages in thread From: Matthias Kaehlcke @ 2020-03-29 17:44 UTC (permalink / raw) To: Sam Ravnborg Cc: Harigovindan P, dri-devel, linux-arm-msm, freedreno, devicetree, sean, seanpaul Hi Sam, On Sat, Mar 28, 2020 at 09:40:47PM +0100, Sam Ravnborg wrote: > Hi Harigovindan > > On Fri, Mar 27, 2020 at 01:06:34PM +0530, Harigovindan P wrote: > > Adding support for visionox rm69299 panel driver and adding bindings for the same panel. > > > > Harigovindan P (2): > > dt-bindings: display: add visionox rm69299 panel variant > > drm/panel: add support for rm69299 visionox panel driver > > I have only the first patch, which is now applied. > Please resend second patch as it is lost somewhere. Yes, it seems for v8, v9 and v10 only the bindings were sent, even though the cover letter and subject say it's a series of two patches. To my knowledge the latest version of the driver patch is this: https://patchwork.kernel.org/patch/11439689/ ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v10 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel @ 2020-03-29 17:44 ` Matthias Kaehlcke 0 siblings, 0 replies; 14+ messages in thread From: Matthias Kaehlcke @ 2020-03-29 17:44 UTC (permalink / raw) To: Sam Ravnborg Cc: freedreno, devicetree, linux-arm-msm, Harigovindan P, dri-devel, seanpaul, sean Hi Sam, On Sat, Mar 28, 2020 at 09:40:47PM +0100, Sam Ravnborg wrote: > Hi Harigovindan > > On Fri, Mar 27, 2020 at 01:06:34PM +0530, Harigovindan P wrote: > > Adding support for visionox rm69299 panel driver and adding bindings for the same panel. > > > > Harigovindan P (2): > > dt-bindings: display: add visionox rm69299 panel variant > > drm/panel: add support for rm69299 visionox panel driver > > I have only the first patch, which is now applied. > Please resend second patch as it is lost somewhere. Yes, it seems for v8, v9 and v10 only the bindings were sent, even though the cover letter and subject say it's a series of two patches. To my knowledge the latest version of the driver patch is this: https://patchwork.kernel.org/patch/11439689/ _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v10 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel 2020-03-29 17:44 ` Matthias Kaehlcke @ 2020-03-30 19:25 ` Sam Ravnborg -1 siblings, 0 replies; 14+ messages in thread From: Sam Ravnborg @ 2020-03-30 19:25 UTC (permalink / raw) To: Matthias Kaehlcke Cc: Harigovindan P, dri-devel, linux-arm-msm, freedreno, devicetree, sean, seanpaul Hi Matthias. On Sun, Mar 29, 2020 at 10:44:17AM -0700, Matthias Kaehlcke wrote: > Hi Sam, > > On Sat, Mar 28, 2020 at 09:40:47PM +0100, Sam Ravnborg wrote: > > Hi Harigovindan > > > > On Fri, Mar 27, 2020 at 01:06:34PM +0530, Harigovindan P wrote: > > > Adding support for visionox rm69299 panel driver and adding bindings for the same panel. > > > > > > Harigovindan P (2): > > > dt-bindings: display: add visionox rm69299 panel variant > > > drm/panel: add support for rm69299 visionox panel driver > > > > I have only the first patch, which is now applied. > > Please resend second patch as it is lost somewhere. > > Yes, it seems for v8, v9 and v10 only the bindings were sent, even > though the cover letter and subject say it's a series of two patches. > > To my knowledge the latest version of the driver patch is this: > > https://patchwork.kernel.org/patch/11439689/ I did not go back and check, but I recall there was review feedback that is not yet addressed. I applied the patch here and checkpatch was not happy: total: 14 errors, 22 warnings, 11 checks, 314 lines checked Many of these are trivial indent or spaces used where tabs should be used. These needs to be fixed before the driver will be applied. And there was too much that I just did it while applying. Sam ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v10 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel @ 2020-03-30 19:25 ` Sam Ravnborg 0 siblings, 0 replies; 14+ messages in thread From: Sam Ravnborg @ 2020-03-30 19:25 UTC (permalink / raw) To: Matthias Kaehlcke Cc: freedreno, devicetree, linux-arm-msm, Harigovindan P, dri-devel, seanpaul, sean Hi Matthias. On Sun, Mar 29, 2020 at 10:44:17AM -0700, Matthias Kaehlcke wrote: > Hi Sam, > > On Sat, Mar 28, 2020 at 09:40:47PM +0100, Sam Ravnborg wrote: > > Hi Harigovindan > > > > On Fri, Mar 27, 2020 at 01:06:34PM +0530, Harigovindan P wrote: > > > Adding support for visionox rm69299 panel driver and adding bindings for the same panel. > > > > > > Harigovindan P (2): > > > dt-bindings: display: add visionox rm69299 panel variant > > > drm/panel: add support for rm69299 visionox panel driver > > > > I have only the first patch, which is now applied. > > Please resend second patch as it is lost somewhere. > > Yes, it seems for v8, v9 and v10 only the bindings were sent, even > though the cover letter and subject say it's a series of two patches. > > To my knowledge the latest version of the driver patch is this: > > https://patchwork.kernel.org/patch/11439689/ I did not go back and check, but I recall there was review feedback that is not yet addressed. I applied the patch here and checkpatch was not happy: total: 14 errors, 22 warnings, 11 checks, 314 lines checked Many of these are trivial indent or spaces used where tabs should be used. These needs to be fixed before the driver will be applied. And there was too much that I just did it while applying. Sam _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v10 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel 2020-03-30 19:25 ` Sam Ravnborg @ 2020-03-31 16:39 ` Matthias Kaehlcke -1 siblings, 0 replies; 14+ messages in thread From: Matthias Kaehlcke @ 2020-03-31 16:39 UTC (permalink / raw) To: Sam Ravnborg Cc: Harigovindan P, dri-devel, linux-arm-msm, freedreno, devicetree, sean, seanpaul On Mon, Mar 30, 2020 at 09:25:11PM +0200, Sam Ravnborg wrote: > Hi Matthias. > > On Sun, Mar 29, 2020 at 10:44:17AM -0700, Matthias Kaehlcke wrote: > > Hi Sam, > > > > On Sat, Mar 28, 2020 at 09:40:47PM +0100, Sam Ravnborg wrote: > > > Hi Harigovindan > > > > > > On Fri, Mar 27, 2020 at 01:06:34PM +0530, Harigovindan P wrote: > > > > Adding support for visionox rm69299 panel driver and adding bindings for the same panel. > > > > > > > > Harigovindan P (2): > > > > dt-bindings: display: add visionox rm69299 panel variant > > > > drm/panel: add support for rm69299 visionox panel driver > > > > > > I have only the first patch, which is now applied. > > > Please resend second patch as it is lost somewhere. > > > > Yes, it seems for v8, v9 and v10 only the bindings were sent, even > > though the cover letter and subject say it's a series of two patches. > > > > To my knowledge the latest version of the driver patch is this: > > > > https://patchwork.kernel.org/patch/11439689/ > > I did not go back and check, but I recall there was > review feedback that is not yet addressed. > > I applied the patch here and checkpatch was not happy: > total: 14 errors, 22 warnings, 11 checks, 314 lines checked > > Many of these are trivial indent or spaces used where tabs should be > used. > These needs to be fixed before the driver will be applied. > > And there was too much that I just did it while applying. Oh, I totally missed these, sure this needs to be addressed. Harigovindan, please also double check if there is any outstanding feedback that still needs to be addressed. Thanks Matthias ^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH v10 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel @ 2020-03-31 16:39 ` Matthias Kaehlcke 0 siblings, 0 replies; 14+ messages in thread From: Matthias Kaehlcke @ 2020-03-31 16:39 UTC (permalink / raw) To: Sam Ravnborg Cc: freedreno, devicetree, linux-arm-msm, Harigovindan P, dri-devel, seanpaul, sean On Mon, Mar 30, 2020 at 09:25:11PM +0200, Sam Ravnborg wrote: > Hi Matthias. > > On Sun, Mar 29, 2020 at 10:44:17AM -0700, Matthias Kaehlcke wrote: > > Hi Sam, > > > > On Sat, Mar 28, 2020 at 09:40:47PM +0100, Sam Ravnborg wrote: > > > Hi Harigovindan > > > > > > On Fri, Mar 27, 2020 at 01:06:34PM +0530, Harigovindan P wrote: > > > > Adding support for visionox rm69299 panel driver and adding bindings for the same panel. > > > > > > > > Harigovindan P (2): > > > > dt-bindings: display: add visionox rm69299 panel variant > > > > drm/panel: add support for rm69299 visionox panel driver > > > > > > I have only the first patch, which is now applied. > > > Please resend second patch as it is lost somewhere. > > > > Yes, it seems for v8, v9 and v10 only the bindings were sent, even > > though the cover letter and subject say it's a series of two patches. > > > > To my knowledge the latest version of the driver patch is this: > > > > https://patchwork.kernel.org/patch/11439689/ > > I did not go back and check, but I recall there was > review feedback that is not yet addressed. > > I applied the patch here and checkpatch was not happy: > total: 14 errors, 22 warnings, 11 checks, 314 lines checked > > Many of these are trivial indent or spaces used where tabs should be > used. > These needs to be fixed before the driver will be applied. > > And there was too much that I just did it while applying. Oh, I totally missed these, sure this needs to be addressed. Harigovindan, please also double check if there is any outstanding feedback that still needs to be addressed. Thanks Matthias _______________________________________________ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel ^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2020-03-31 16:39 UTC | newest] Thread overview: 14+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2020-03-27 7:36 [PATCH v10 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel Harigovindan P 2020-03-27 7:36 ` Harigovindan P 2020-03-27 7:36 ` [PATCH v10 1/2] dt-bindings: display: add visionox rm69299 panel variant Harigovindan P 2020-03-27 7:36 ` Harigovindan P 2020-03-28 20:40 ` Sam Ravnborg 2020-03-28 20:40 ` Sam Ravnborg 2020-03-28 20:40 ` [PATCH v10 0/2] Add support for rm69299 Visionox panel driver and add devicetree bindings for visionox panel Sam Ravnborg 2020-03-28 20:40 ` Sam Ravnborg 2020-03-29 17:44 ` Matthias Kaehlcke 2020-03-29 17:44 ` Matthias Kaehlcke 2020-03-30 19:25 ` Sam Ravnborg 2020-03-30 19:25 ` Sam Ravnborg 2020-03-31 16:39 ` Matthias Kaehlcke 2020-03-31 16:39 ` Matthias Kaehlcke
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.