Re: [PATCH10 0/3] Intel FPGA Video and Image Processing Suite

[Ong, Hean Loong]

On Tue, 2018-08-14 at 10:48 -0500, Dinh Nguyen wrote:
> 
> On 08/14/2018 03:57 AM, Hean-Loong, Ong wrote:
> > 
> > From: Ong, Hean Loong <hean.loong.ong@xxxxxxxxx>
> > 
> > The FPGA FrameBuffer Soft IP could be seenÂÂas the GPU and the DRM
> > driver patchÂ
> > here is allocating memory for information to be streamed from the
> > ARM/LinuxÂ
> > to the display port. Basically the driver just wraps the
> > information such asÂ
> > the pixels to be drawn by the FPGA FrameBuffer 2.
> > 
> > The piece of hardware in discussion is the SoC FPGA where Linux
> > runs on theÂ
> > ARM chip and the FGPA is driven by its NIOS soft core with its own
> > proprietary firmware.
> This driver doesn't have to ARM specific does it?
> 
> > 
> > 
> > For example the application from the ARM Linux would have to write
> > informationÂ
> > on the /dev/fb0 with the information stored in the SDRAM to be
> > fetched by theÂ
> > FPGA framebuffer IP and displayed on the Display Port Monitor.
> > 
> > 
> > Ong Hean Loong (2):
> > Â ARM:socfpga-defconfig Intel FPGA Video and Image Processing Suite
> > Â ARM:drm ivip Intel FPGA Video and Image Processing Suite
> > 
> > Ong, Hean Loong (1):
> > Â ARM:dt-bindings:display Intel FPGA Video and Image Processing
> > Suite
> > 
> > Â.../devicetree/bindings/display/altr,vip-fb2.txtÂÂÂ|ÂÂÂ63 +++++++
> > Âarch/arm/configs/socfpga_defconfigÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂÂÂ5 +
> > Âdrivers/gpu/drm/KconfigÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂÂÂ2 +
> > Âdrivers/gpu/drm/MakefileÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂÂÂ1 +
> > Âdrivers/gpu/drm/ivip/KconfigÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂÂ14 ++
> > Âdrivers/gpu/drm/ivip/MakefileÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂÂÂ9 +
> > Âdrivers/gpu/drm/ivip/intel_vip_conn.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂÂ95
> > ++++++++++
> > Âdrivers/gpu/drm/ivip/intel_vip_core.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂ161
> > ++++++++++++++++
> > Âdrivers/gpu/drm/ivip/intel_vip_drv.hÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂÂ52 ++++++
> > Âdrivers/gpu/drm/ivip/intel_vip_of.cÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂÂ|ÂÂ193
> > ++++++++++++++++++++
> > Â10 files changed, 595 insertions(+), 0 deletions(-)
> > Âcreate mode 100644
> > Documentation/devicetree/bindings/display/altr,vip-fb2.txt
> > Âcreate mode 100644 drivers/gpu/drm/ivip/Kconfig
> > Âcreate mode 100644 drivers/gpu/drm/ivip/Makefile
> > Âcreate mode 100644 drivers/gpu/drm/ivip/intel_vip_conn.c
> > Âcreate mode 100644 drivers/gpu/drm/ivip/intel_vip_core.c
> > Âcreate mode 100644 drivers/gpu/drm/ivip/intel_vip_drv.h
> > Âcreate mode 100644 drivers/gpu/drm/ivip/intel_vip_of.c
> > 
> Can you clarify your commit header? What is "PATCH10", is it version
> 10?
> If so, it should be "PATCHv10".
> 
> You're also missing a few other maintainers in your CC list, the "DRM
> DRIVERS AND MISC GPU PATCHES" people.
> 
Noted

> Dinh