Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1525354

Re: [PATCHv0 1/1] fbdev: add Intel FPGA FRAME BUFFER driver

Show all headers | View raw

On Wed, Nov 16, 2016 at 05:07:58PM +0800, Ong, Hean Loong wrote:
> From: Ong Hean Loong <hean.loong.ong@intel.com>
> 
> 	This patch enables the display port IP driver for
> 	Intel Arria 10 SOCFPGA Golden Hardware
> 	Reference Design (GHRD).
> 
> 	The driver requires enabling the options such as
> 	Coheherent Memory Allocation,
> 	Intel FPGA Frame Buffer, Frame Buffer Conasole
> 
> Signed-off-by: Ong Hean Loong <hean.loong.ong@intel.com>
> ---
>  .../devicetree/bindings/video/intelfpgavipfb.txt   |   22 ++

bindings/display/

It is preferred that bindings are a separate patch.

>  MAINTAINERS                                        |    6 +
>  drivers/video/fbdev/Kconfig                        |   15 +
>  drivers/video/fbdev/Makefile                       |    1 +
>  drivers/video/fbdev/intelfpgavipfb.c               |  302 ++++++++++++++++++++

AIUI, we're not taking new FB drivers. This should be a DRM driver 
instead.

>  5 files changed, 346 insertions(+), 0 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/video/intelfpgavipfb.txt
>  create mode 100644 drivers/video/fbdev/intelfpgavipfb.c
> 
> diff --git a/Documentation/devicetree/bindings/video/intelfpgavipfb.txt b/Documentation/devicetree/bindings/video/intelfpgavipfb.txt
> new file mode 100644
> index 0000000..8928c99
> --- /dev/null
> +++ b/Documentation/devicetree/bindings/video/intelfpgavipfb.txt
> @@ -0,0 +1,22 @@
> +Intel FPGA Video and Image Processing(VIP) Frame Buffer bindings
> +
> +Required properties:
> +- compatible: "intel,vip-frame-buffer2"

Should be something like 'intel,arria10-vip'. Describe the h/w block, 
frame-buffer is a Linux term.

> +- reg: Physical base address and length of the framebuffer controller's
> +  registers.
> +- max-width: The width of the framebuffer in pixels.
> +- max-height: The height of the framebuffer in pixels.
> +- bits-per-color: only "8" is currently supported

These are not h/w properties.

> +- mem-word-width = the bus width of the avalon master port on the frame reader

Needs a vendor prefix.

> +
> +Example:
> +
> +alt_vip_vfr_0: vip@0xff260000 {
> +	compatible = "intel,vip-frame-buffer2";
> +	reg = <0xff260000 0x00000080>;
> +	max-width = <1024>;
> +	max-height = <768>;
> +	bits-per-color = <8>;
> +	mem-word-width = <128>;
> +};
> +

Back to linux.kernel | Previous | Next — Previous in thread | Next in thread | Find similar | Unroll thread

Thread

[PATCHv0 1/1] fbdev: add Intel FPGA FRAME BUFFER driver "Ong, Hean Loong" <hean.loong.ong@intel.com> - 2016-11-16 10:10 +0100
  Re: [PATCHv0 1/1] fbdev: add Intel FPGA FRAME BUFFER driver Rob Herring <robh@kernel.org> - 2016-11-18 15:10 +0100
    Re: [PATCHv0 1/1] fbdev: add Intel FPGA FRAME BUFFER driver One Thousand Gnomes <gnomes@lxorguk.ukuu.org.uk> - 2016-11-18 15:20 +0100
      Re: [PATCHv0 1/1] fbdev: add Intel FPGA FRAME BUFFER driver Rob Herring <robh@kernel.org> - 2016-11-18 20:00 +0100
        Re: [PATCHv0 1/1] fbdev: add Intel FPGA FRAME BUFFER driver "Ong, Hean Loong" <hean.loong.ong@intel.com> - 2016-11-21 03:50 +0100
  Re: [PATCHv0 1/1] fbdev: add Intel FPGA FRAME BUFFER driver Tomi Valkeinen <tomi.valkeinen@ti.com> - 2016-11-23 08:40 +0100

csiph-web