Re: [RFC PATCH 0/5] Move IPUv3 core out of staging, add CSI support
Please fix the following static checker complaints before moving out of staging: drivers/staging/imx-drm/ipu-v3/ipu-dmfc.c:164 ipu_dmfc_setup_channel() warn: variable dereferenced before check 'dmfc' (see line 157) drivers/staging/imx-drm/imx-ldb.c:109 imx_ldb_connector_get_modes() error: potential null dereference 'mode'. (drm_mode_create returns null) drivers/staging/imx-drm/imx-ldb.c:340 imx_ldb_get_clk() error: format string overflow. buf_size: 16 length: 18 probably 18 is theory and not real life, but 16 is based on theory as well. drivers/staging/imx-drm/parallel-display.c:64 imx_pd_connector_get_modes() error: potential null dereference 'mode'. (drm_mode_create returns null) drivers/staging/imx-drm/parallel-display.c:73 imx_pd_connector_get_modes() error: potential null dereference 'mode'. (drm_mode_create returns null) drivers/staging/imx-drm/ipuv3-plane.c: In function ‘ipu_plane_set_base’: drivers/staging/imx-drm/ipuv3-plane.c:75:2: warning: format ‘%x’ expects argument of type ‘unsigned int’, but argument 4 has type ‘dma_addr_t’ [-Wformat] cast it to u64 and use %llx drivers/staging/imx-drm/imx-drm-core.c:89:21: warning: symbol 'imx_drm_find_crtc' was not declared. Should it be static? regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [RFC PATCH 0/5] Move IPUv3 core out of staging, add CSI support
On Wed, Feb 26, 2014 at 11:39:03AM +0300, Dan Carpenter wrote: Please fix the following static checker complaints before moving out of staging: drivers/staging/imx-drm/ipu-v3/ipu-dmfc.c:164 ipu_dmfc_setup_channel() warn: variable dereferenced before check 'dmfc' (see line 157) Note that what's being talked about being moved out is only the above, not the files in drivers/staging/imx-drm. DRM people have not yet reviewed imx-drm itself, which is a necessary step - but nevertheless, thanks for running a static checker on this stuff. -- FTTC broadband for 0.8mile line: now at 9.7Mbps down 460kbps up... slowly improving, and getting towards what was expected from it. ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
Re: [RFC PATCH 0/5] Move IPUv3 core out of staging, add CSI support
Am Freitag, den 20.12.2013, 11:12 -0800 schrieb Greg Kroah-Hartman: On Fri, Dec 20, 2013 at 06:52:41PM +0100, Philipp Zabel wrote: Hi, this is mostly about the first patch, which moves the IPUv3 core code (drivers/staging/imx-drm/ipu-v3) to drivers/gpu. host1x, which serves a similar purpose, already sits there. The other four patches add the necessary code for CSI and SMFC handling, which is used by the V4L2 CSI capture driver. Currently this is based on Russell's patch [PATCH 62/64] imx-drm: pass an IPU ID to crtc and core I am aware that there are now quite a few other patches in the pipeline that touch drivers/staging/imx-drm/ipu-v3/*, so I am happy to rebase this (or them) as needed. I'd like to move the core code out of staging so that we can start submitting V4L2 code for video capture and scaling / colorspace conversion in parallel. I'd recommend doing the move (if the gpu/drm maintainer agrees) after 3.14-rc1 as then all of my pending patches would be applied and we would be synced up with everything. Ok. Once this core is moved, what is keeping the rest in staging and why isn't it moving out as well? Russell put a lot of work into fixing imx-drm issues, and I'd like to make the change to a common device tree binding before this leaves staging. regards Philipp ___ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
[RFC PATCH 0/5] Move IPUv3 core out of staging, add CSI support
Hi,
this is mostly about the first patch, which moves the IPUv3 core code
(drivers/staging/imx-drm/ipu-v3) to drivers/gpu. host1x, which
serves a similar purpose, already sits there.
The other four patches add the necessary code for CSI and SMFC handling,
which is used by the V4L2 CSI capture driver.
Currently this is based on Russell's patch
[PATCH 62/64] imx-drm: pass an IPU ID to crtc and core
I am aware that there are now quite a few other patches in the pipeline
that touch drivers/staging/imx-drm/ipu-v3/*, so I am happy to rebase this
(or them) as needed. I'd like to move the core code out of staging so that
we can start submitting V4L2 code for video capture and scaling / colorspace
conversion in parallel.
Philipp Zabel (5):
gpu: ipu-v3: Move i.MX IPUv3 core driver out of staging
gpu: ipu-v3: Add SMFC code
gpu: ipu-v3: Add ipu_idmac_get_current_buffer function
gpu: ipu-v3: Add CSI and SMFC module enable wrappers
gpu: ipu-v3: Register the CSI modules
drivers/gpu/Makefile | 1 +
drivers/gpu/ipu-v3/Kconfig | 7 ++
drivers/{staging/imx-drm = gpu}/ipu-v3/Makefile | 4 +-
.../{staging/imx-drm = gpu}/ipu-v3/ipu-common.c | 81 --
drivers/{staging/imx-drm = gpu}/ipu-v3/ipu-dc.c | 3 +-
drivers/{staging/imx-drm = gpu}/ipu-v3/ipu-di.c | 2 +-
drivers/{staging/imx-drm = gpu}/ipu-v3/ipu-dmfc.c | 2 +-
drivers/{staging/imx-drm = gpu}/ipu-v3/ipu-dp.c | 2 +-
drivers/{staging/imx-drm = gpu}/ipu-v3/ipu-prv.h | 8 +-
drivers/gpu/ipu-v3/ipu-smfc.c | 97 ++
drivers/staging/imx-drm/Kconfig| 11 +--
drivers/staging/imx-drm/Makefile | 1 -
drivers/staging/imx-drm/imx-drm.h | 4 -
drivers/staging/imx-drm/imx-tve.c | 1 +
drivers/staging/imx-drm/ipuv3-crtc.c | 2 +-
drivers/staging/imx-drm/ipuv3-plane.c | 2 +-
drivers/video/Kconfig | 1 +
.../imx-drm/ipu-v3 = include/video}/imx-ipu-v3.h | 18
18 files changed, 217 insertions(+), 30 deletions(-)
create mode 100644 drivers/gpu/ipu-v3/Kconfig
rename drivers/{staging/imx-drm = gpu}/ipu-v3/Makefile (51%)
rename drivers/{staging/imx-drm = gpu}/ipu-v3/ipu-common.c (94%)
rename drivers/{staging/imx-drm = gpu}/ipu-v3/ipu-dc.c (99%)
rename drivers/{staging/imx-drm = gpu}/ipu-v3/ipu-di.c (99%)
rename drivers/{staging/imx-drm = gpu}/ipu-v3/ipu-dmfc.c (99%)
rename drivers/{staging/imx-drm = gpu}/ipu-v3/ipu-dp.c (99%)
rename drivers/{staging/imx-drm = gpu}/ipu-v3/ipu-prv.h (96%)
create mode 100644 drivers/gpu/ipu-v3/ipu-smfc.c
rename {drivers/staging/imx-drm/ipu-v3 = include/video}/imx-ipu-v3.h (94%)
regards
Philipp
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
