From: Rob Herring <robherring2@gmail.com> To: Eric Anholt <eric@anholt.net> Cc: dri-devel <dri-devel@lists.freedesktop.org>, "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>, Stephen Warren <swarren@wwwdotorg.org>, Lee Jones <lee@kernel.org>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, linux-rpi-kernel@lists.infradead.org, "linux-arm-kernel@lists.infradead.org" <linux-arm-kernel@lists.infradead.org> Subject: Re: [PATCH 1/7] drm/vc4: Add devicetree bindings for VC4. Date: Mon, 24 Aug 2015 08:56:14 -0500 [thread overview] Message-ID: <CAL_JsqKhfQS46NwbTq0PNDpnNuztnRkcK+WDhFMaayE_8dDt3Q@mail.gmail.com> (raw) In-Reply-To: <1439427380-2436-2-git-send-email-eric@anholt.net> On Wed, Aug 12, 2015 at 7:56 PM, Eric Anholt <eric@anholt.net> wrote: > Signed-off-by: Eric Anholt <eric@anholt.net> > --- > .../devicetree/bindings/gpu/brcm,bcm-vc4.txt | 83 ++++++++++++++++++++++ > 1 file changed, 83 insertions(+) > create mode 100644 Documentation/devicetree/bindings/gpu/brcm,bcm-vc4.txt > > diff --git a/Documentation/devicetree/bindings/gpu/brcm,bcm-vc4.txt b/Documentation/devicetree/bindings/gpu/brcm,bcm-vc4.txt > new file mode 100644 > index 0000000..2b13e61 > --- /dev/null > +++ b/Documentation/devicetree/bindings/gpu/brcm,bcm-vc4.txt > @@ -0,0 +1,83 @@ > +Broadcom VC4 GPU > + > +The VC4 device present on the Raspberry Pi includes a display system > +with HDMI output and the HVS scaler for compositing display planes. > + > +Required properties for VC4: > +- compatible: Should be "brcm,vc4" > +- crtcs: List of references to pixelvalve scanout engines > +- hvss: List of references to HVS video scalers > +- encoders: List of references to output encoders (HDMI, SDTV) Creating these links is what the OF graph binding is for. Please use it. Plus this is a DRMism in the binding. > + > +Required properties for Pixel Valve: > +- compatible: Should be "brcm,vc4-pixelvalve" There's only one version of IP and all chips have the same bugs? You should have chip names in the compatible strings. > +- reg: Physical base address and length of the PV's registers > +- interrupts: The interrupt number > + See bindings/interrupt-controller/brcm,bcm2835-armctrl-ic.txt > + > +Required properties for HVS: > +- compatible: Should be "brcm,vc4-hvs" > +- reg: Physical base address and length of the HVS's registers > +- interrupts: The interrupt number > + See bindings/interrupt-controller/brcm,bcm2835-armctrl-ic.txt > + > +Required properties for HDMI > +- compatible: Should be "brcm,vc4-hdmi" > +- reg: Physical base address and length of the two register ranges > + ("HDMI" and "HD") > +- interrupts: The interrupt numbers > + See bindings/interrupt-controller/brcm,bcm2835-armctrl-ic.txt > +- ddc: phandle of the I2C controller used for DDC EDID probing > +- crtc: phandle to the pixelvalve CRTC the HDMI encoder is attached to Same comment about OF graph. > + > +Optional properties for HDMI: > +- hpd-gpio: The GPIO pin for HDMI hotplug detect (if it doesn't appear > + as an interrupt/status bit in the HDMI controller > + itself). See bindings/pinctrl/brcm,bcm2835-gpio.txt Use the hdmi-connector binding. This doesn't belong here. The clue is your comment in parenthesis. Rob
WARNING: multiple messages have this Message-ID (diff)
From: robherring2@gmail.com (Rob Herring) To: linux-arm-kernel@lists.infradead.org Subject: [PATCH 1/7] drm/vc4: Add devicetree bindings for VC4. Date: Mon, 24 Aug 2015 08:56:14 -0500 [thread overview] Message-ID: <CAL_JsqKhfQS46NwbTq0PNDpnNuztnRkcK+WDhFMaayE_8dDt3Q@mail.gmail.com> (raw) In-Reply-To: <1439427380-2436-2-git-send-email-eric@anholt.net> On Wed, Aug 12, 2015 at 7:56 PM, Eric Anholt <eric@anholt.net> wrote: > Signed-off-by: Eric Anholt <eric@anholt.net> > --- > .../devicetree/bindings/gpu/brcm,bcm-vc4.txt | 83 ++++++++++++++++++++++ > 1 file changed, 83 insertions(+) > create mode 100644 Documentation/devicetree/bindings/gpu/brcm,bcm-vc4.txt > > diff --git a/Documentation/devicetree/bindings/gpu/brcm,bcm-vc4.txt b/Documentation/devicetree/bindings/gpu/brcm,bcm-vc4.txt > new file mode 100644 > index 0000000..2b13e61 > --- /dev/null > +++ b/Documentation/devicetree/bindings/gpu/brcm,bcm-vc4.txt > @@ -0,0 +1,83 @@ > +Broadcom VC4 GPU > + > +The VC4 device present on the Raspberry Pi includes a display system > +with HDMI output and the HVS scaler for compositing display planes. > + > +Required properties for VC4: > +- compatible: Should be "brcm,vc4" > +- crtcs: List of references to pixelvalve scanout engines > +- hvss: List of references to HVS video scalers > +- encoders: List of references to output encoders (HDMI, SDTV) Creating these links is what the OF graph binding is for. Please use it. Plus this is a DRMism in the binding. > + > +Required properties for Pixel Valve: > +- compatible: Should be "brcm,vc4-pixelvalve" There's only one version of IP and all chips have the same bugs? You should have chip names in the compatible strings. > +- reg: Physical base address and length of the PV's registers > +- interrupts: The interrupt number > + See bindings/interrupt-controller/brcm,bcm2835-armctrl-ic.txt > + > +Required properties for HVS: > +- compatible: Should be "brcm,vc4-hvs" > +- reg: Physical base address and length of the HVS's registers > +- interrupts: The interrupt number > + See bindings/interrupt-controller/brcm,bcm2835-armctrl-ic.txt > + > +Required properties for HDMI > +- compatible: Should be "brcm,vc4-hdmi" > +- reg: Physical base address and length of the two register ranges > + ("HDMI" and "HD") > +- interrupts: The interrupt numbers > + See bindings/interrupt-controller/brcm,bcm2835-armctrl-ic.txt > +- ddc: phandle of the I2C controller used for DDC EDID probing > +- crtc: phandle to the pixelvalve CRTC the HDMI encoder is attached to Same comment about OF graph. > + > +Optional properties for HDMI: > +- hpd-gpio: The GPIO pin for HDMI hotplug detect (if it doesn't appear > + as an interrupt/status bit in the HDMI controller > + itself). See bindings/pinctrl/brcm,bcm2835-gpio.txt Use the hdmi-connector binding. This doesn't belong here. The clue is your comment in parenthesis. Rob
next prev parent reply other threads:[~2015-08-24 13:56 UTC|newest] Thread overview: 101+ messages / expand[flat|nested] mbox.gz Atom feed top 2015-08-13 0:56 Raspberry Pi KMS-only driver Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-13 0:56 ` [PATCH 1/7] drm/vc4: Add devicetree bindings for VC4 Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-15 4:38 ` Stephen Warren 2015-08-15 4:38 ` Stephen Warren 2015-08-17 18:30 ` Eric Anholt 2015-08-17 18:30 ` Eric Anholt 2015-08-17 18:30 ` Eric Anholt 2015-08-24 13:47 ` Rob Herring 2015-08-24 13:47 ` Rob Herring 2015-08-24 13:47 ` Rob Herring 2015-08-25 20:42 ` Rob Clark 2015-08-25 20:42 ` Rob Clark 2015-08-25 20:42 ` Rob Clark 2015-08-25 23:22 ` Rob Herring 2015-08-25 23:22 ` Rob Herring 2015-08-25 23:22 ` Rob Herring 2015-08-26 11:52 ` Daniel Vetter 2015-08-26 11:52 ` Daniel Vetter 2015-08-26 11:52 ` Daniel Vetter 2015-08-26 12:09 ` Thierry Reding 2015-08-26 12:09 ` Thierry Reding 2015-08-26 12:09 ` Thierry Reding 2015-08-26 14:30 ` Rob Herring 2015-08-26 14:30 ` Rob Herring 2015-08-26 14:30 ` Rob Herring 2015-08-26 20:59 ` Dave Airlie 2015-08-26 20:59 ` Dave Airlie 2015-08-26 20:59 ` Dave Airlie 2015-08-27 0:35 ` Rob Herring 2015-08-27 0:35 ` Rob Herring 2015-08-27 0:35 ` Rob Herring 2015-08-26 11:51 ` Thierry Reding 2015-08-26 11:51 ` Thierry Reding 2015-08-26 11:51 ` Thierry Reding 2015-08-24 13:56 ` Rob Herring [this message] 2015-08-24 13:56 ` Rob Herring 2015-08-24 13:56 ` Rob Herring 2015-08-13 0:56 ` [PATCH 2/7] MAINTAINERS: Add myself for the new VC4 (RPi GPU) graphics driver Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-15 4:39 ` Stephen Warren 2015-08-15 4:39 ` Stephen Warren 2015-08-15 4:39 ` Stephen Warren 2015-08-17 18:47 ` Eric Anholt 2015-08-17 18:47 ` Eric Anholt 2015-08-17 18:47 ` Eric Anholt 2015-08-13 0:56 ` [PATCH 3/7] drm/vc4: Add KMS support for Raspberry Pi Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-13 7:51 ` Daniel Vetter 2015-08-13 7:51 ` Daniel Vetter 2015-08-13 7:51 ` Daniel Vetter 2015-08-13 20:44 ` Eric Anholt 2015-08-13 20:44 ` Eric Anholt 2015-08-13 20:44 ` Eric Anholt 2015-08-13 21:17 ` Daniel Vetter 2015-08-13 21:17 ` Daniel Vetter 2015-08-13 21:17 ` Daniel Vetter 2015-08-18 20:56 ` Eric Anholt 2015-08-18 20:56 ` Eric Anholt 2015-08-18 20:56 ` Eric Anholt 2015-08-13 21:29 ` Russell King - ARM Linux 2015-08-13 21:29 ` Russell King - ARM Linux 2015-08-13 21:29 ` Russell King - ARM Linux 2015-08-13 23:03 ` Eric Anholt 2015-08-13 23:03 ` Eric Anholt 2015-08-13 23:03 ` Eric Anholt 2015-08-13 11:45 ` Emil Velikov 2015-08-13 11:45 ` Emil Velikov 2015-08-13 11:45 ` Emil Velikov 2015-08-15 4:45 ` Stephen Warren 2015-08-15 4:45 ` Stephen Warren 2015-08-15 4:45 ` Stephen Warren 2015-08-17 17:56 ` Eric Anholt 2015-08-17 17:56 ` Eric Anholt 2015-08-17 17:56 ` Eric Anholt 2015-08-13 0:56 ` [PATCH 4/7] drm/vc4: Use the fbdev_cma helpers Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-13 0:56 ` [PATCH 5/7] drm/vc4: Allow vblank to be disabled Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-13 0:56 ` [PATCH 6/7] ARM: bcm2835: Add the DDC I2C controller to the device tree Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-15 4:51 ` Stephen Warren 2015-08-15 4:51 ` Stephen Warren 2015-08-15 4:51 ` Stephen Warren 2015-08-17 18:35 ` Eric Anholt 2015-08-17 18:35 ` Eric Anholt 2015-08-17 18:35 ` Eric Anholt 2015-08-13 0:56 ` [PATCH 7/7] ARM: bcm2835: Add VC4 " Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-13 0:56 ` Eric Anholt 2015-08-15 4:54 ` Stephen Warren 2015-08-15 4:54 ` Stephen Warren 2015-08-15 4:54 ` Stephen Warren
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=CAL_JsqKhfQS46NwbTq0PNDpnNuztnRkcK+WDhFMaayE_8dDt3Q@mail.gmail.com \ --to=robherring2@gmail.com \ --cc=devicetree@vger.kernel.org \ --cc=dri-devel@lists.freedesktop.org \ --cc=eric@anholt.net \ --cc=lee@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-rpi-kernel@lists.infradead.org \ --cc=swarren@wwwdotorg.org \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
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.