Re: [PATCH 4/7] stih_cec: use new cec_notifier_find_hdmi_dev helper

2019-04-08 Thread Benjamin Gaignard
Le lun. 8 avr. 2019 à 13:04, Hans Verkuil a écrit : > > The STI CEC driver increased the HDMI device refcount when > it shouldn't. Use the new helper function to ensure that that > doesn't happen and to simplify the driver code. > > Signed-off-by: Hans Verkuil

Re: [PATCH v2 4/4] media: platform: sti: fix possible object reference leak

2019-02-13 Thread Benjamin Gaignard
eral CEC drivers take a reference of the HDMI > device and never release it. So those drivers need to be fixed. > > This patch fixes those two issues. Reviewed-by: Benjamin Gaignard > > Fixes: fc4e009c6c98 ("[media] stih-cec: add CEC notifier support") > Suggested-by

Re: [PATCH 7/8] media: sti/bdisp: don't pass GFP_DMA32 to dma_alloc_attrs

2018-10-18 Thread Benjamin Gaignard
Le mer. 17 oct. 2018 à 09:20, Christoph Hellwig a écrit : > > On Mon, Oct 15, 2018 at 11:12:55AM +0200, Benjamin Gaignard wrote: > > Le sam. 13 oct. 2018 à 17:18, Christoph Hellwig a écrit : > > > > > > The DMA API does its own zone decisions based on the coheren

Re: [PATCH 6/8] drm: sti: don't pass GFP_DMA32 to dma_alloc_wc

2018-10-18 Thread Benjamin Gaignard
Le mer. 17 oct. 2018 à 09:19, Christoph Hellwig a écrit : > > On Tue, Oct 16, 2018 at 02:41:23PM +0200, Benjamin Gaignard wrote: > > Le lun. 15 oct. 2018 à 11:12, Benjamin Gaignard > > a écrit : > > > > > > Le sam. 13 oct. 2018 à 17:17, Christoph Hellwig a

Re: [PATCH 6/8] drm: sti: don't pass GFP_DMA32 to dma_alloc_wc

2018-10-16 Thread Benjamin Gaignard
Le lun. 15 oct. 2018 à 11:12, Benjamin Gaignard a écrit : > > Le sam. 13 oct. 2018 à 17:17, Christoph Hellwig a écrit : > > > > The DMA API does its own zone decisions based on the coherent_dma_mask. > > > > Signed-off-by: Christoph Hellwig > > Reviewed-by:

Re: [PATCH 6/8] drm: sti: don't pass GFP_DMA32 to dma_alloc_wc

2018-10-15 Thread Benjamin Gaignard
Le sam. 13 oct. 2018 à 17:17, Christoph Hellwig a écrit : > > The DMA API does its own zone decisions based on the coherent_dma_mask. > > Signed-off-by: Christoph Hellwig Reviewed-by: Benjamin Gaignard > --- > drivers/gpu/drm/sti/sti_gdp.c | 2 +- > 1 file changed, 1 ins

Re: [PATCH 7/8] media: sti/bdisp: don't pass GFP_DMA32 to dma_alloc_attrs

2018-10-15 Thread Benjamin Gaignard
Le sam. 13 oct. 2018 à 17:18, Christoph Hellwig a écrit : > > The DMA API does its own zone decisions based on the coherent_dma_mask. > > Signed-off-by: Christoph Hellwig Reviewed-by: Benjamin Gaignard > --- > drivers/media/platform/sti/bdisp/bdisp-hw.c | 2 +- > 1 file

[PATCH 2/2] cec: stm32: add suspend/resume functions

2018-03-15 Thread Benjamin Gaignard
From: Benjamin Gaignard If wake up irq is defined in device-tree cec adapter could be used has wake up source. Signed-off-by: Benjamin Gaignard --- drivers/media/platform/stm32/stm32-cec.c | 44 +++- 1 file changed, 43 insertions(+), 1 deletion(-) diff --git a

[PATCH 1/2] cec: stm32: simplify clock management

2018-03-15 Thread Benjamin Gaignard
From: Benjamin Gaignard Since CEC framework enable and disable the adapter when it is needed just follow it orders to enable/disable the clocks. Call stm32_cec_hw_init() when the adapter is enabled and do not let regmap manage registers clock help to simplify clocking scheme. While reworking

[PATCH 0/2] cec: stm32: add wakeup support

2018-03-15 Thread Benjamin Gaignard
CEC hardware block is enable to resume the system if a command is received on data lane. Prior to implement suspend/resume functions a patch simplify clock management int the driver by introducting pm_runtime{suspend/resume} functions. Benjamin Gaignard (2): cec: stm32: simplify clock

[PATCH] media: platform: sti: Adopt SPDX identifier

2017-12-05 Thread Benjamin Gaignard
Add SPDX identifiers to files under sti directory Signed-off-by: Benjamin Gaignard --- drivers/media/platform/sti/bdisp/bdisp-debug.c | 2 +- drivers/media/platform/sti/bdisp/bdisp-filter.h | 2 +- drivers/media/platform/sti/bdisp/bdisp-hw.c | 2 +- drivers

[PATCH] media: platform: stm32: Adopt SPDX identifier

2017-12-05 Thread Benjamin Gaignard
Add SPDX identifiers to files under stm32 directory Signed-off-by: Benjamin Gaignard --- drivers/media/platform/stm32/stm32-cec.c | 5 + drivers/media/platform/stm32/stm32-dcmi.c | 2 +- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/drivers/media/platform/stm32/stm32-cec.c

Re: [PATCH 5/5] stm32-cec: use CEC_CAP_DEFAULTS

2017-08-20 Thread Benjamin Gaignard
2017-08-04 12:41 GMT+02:00 Hans Verkuil : > From: Hans Verkuil > > Use the new CEC_CAP_DEFAULTS define in this driver. > > Signed-off-by: Hans Verkuil Acked-by: Benjamin Gaignard > --- > drivers/media/platform/stm32/stm32-cec.c | 4 +--- > 1 file changed, 1 i

Re: [PATCH 4/5] stih-cec: use CEC_CAP_DEFAULTS

2017-08-20 Thread Benjamin Gaignard
> > Signed-off-by: Hans Verkuil Acked-by: Benjamin Gaignard > --- > drivers/media/platform/sti/cec/stih-cec.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/media/platform/sti/cec/stih-cec.c > b/drivers/media/platform/sti/cec/stih-cec.c

Re: [PATCH 1/5] media/cec.h: add CEC_CAP_DEFAULTS

2017-08-20 Thread Benjamin Gaignard
Rs these four caps to simplify >> drivers. >> >> Signed-off-by: Hans Verkuil > > > Reviewed-by: Sylwester Nawrocki Reviewed-by: Benjamin Gaignard

Re: [PATCH 2/2] dt-bindings: media/s5p-cec.txt, media/stih-cec.txt: refer to cec.txt

2017-06-12 Thread Benjamin Gaignard
2017-06-09 19:54 GMT+02:00 Hans Verkuil : > From: Hans Verkuil > > Now that there is a cec.txt with common CEC bindings, update the two > driver-specific bindings to refer to cec.txt. > > Signed-off-by: Hans Verkuil > Cc: Krzysztof Kozlowski > Cc: Andrzej Hajda >

Re: [PATCH 1/2] dt-bindings: add media/cec.txt

2017-06-12 Thread Benjamin Gaignard
2017-06-09 19:54 GMT+02:00 Hans Verkuil : > From: Hans Verkuil > > Document common HDMI CEC bindings. Add this to the MAINTAINERS file > as well. > > Signed-off-by: Hans Verkuil > Cc: Krzysztof Kozlowski > Cc: Andrzej Hajda > Cc: Benjamin Gaignard >

Re: [PATCH 4/9] stih-cec/vivid/pulse8/rainshadow: use cec_transmit_attempt_done

2017-06-12 Thread Benjamin Gaignard
2017-06-07 16:46 GMT+02:00 Hans Verkuil : > From: Hans Verkuil > > Use the helper function cec_transmit_attempt_done instead of > cec_transmit_done to simplify the code. > > Signed-off-by: Hans Verkuil > Cc: Benjamin Gaignard > --- > drivers/media/platform/sti

[PATCH v7] cec: add STM32 cec driver

2017-05-31 Thread Benjamin Gaignard
;clk_cec); + clk_unprepare(cec->clk_hdmi_cec); + + cec_unregister_adapter(cec->adap); + + return 0; +} + +static const struct of_device_id stm32_cec_of_match[] = { + { .compatible = "st,stm32-cec" }, + { /* end node */ } +}; +MODULE_DEVICE_TABLE(of, stm32_cec_of_match); + +static struct platform_driver stm32_cec_driver = { + .probe = stm32_cec_probe, + .remove = stm32_cec_remove, + .driver = { + .name = CEC_NAME, + .of_match_table = stm32_cec_of_match, + }, +}; + +module_platform_driver(stm32_cec_driver); + +MODULE_AUTHOR("Benjamin Gaignard "); +MODULE_AUTHOR("Yannick Fertre "); +MODULE_DESCRIPTION("STMicroelectronics STM32 Consumer Electronics Control"); +MODULE_LICENSE("GPL v2"); -- 1.9.1

[PATCH v6 1/2] dt-bindings: media: stm32 cec driver

2017-05-31 Thread Benjamin Gaignard
Add bindings documentation for stm32 CEC driver. Signed-off-by: Benjamin Gaignard --- .../devicetree/bindings/media/st,stm32-cec.txt| 19 +++ 1 file changed, 19 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/st,stm32-cec.txt diff --git a

[PATCH v6 2/2] cec: add STM32 cec driver

2017-05-31 Thread Benjamin Gaignard
+ +static int stm32_cec_remove(struct platform_device *pdev) +{ + struct stm32_cec *cec = platform_get_drvdata(pdev); + + clk_unprepare(cec->clk_cec); + clk_unprepare(cec->clk_hdmi_cec); + + cec_unregister_adapter(cec->adap); + + return 0; +} + +static const struct o

[PATCH v6 0/2] cec: STM32 driver

2017-05-31 Thread Benjamin Gaignard
ID : 0x00903e OSD Name : 'TV' Menu Language : fre Power Status : On Total: 10, Succeeded: 10, Failed: 0, Warnings: 0 Benjamin Gaignard (2): dt-bindings: media: stm32 cec driver ce

[PATCH 00/15] make more driver use devm_of_platform_populate()

2017-05-29 Thread Benjamin Gaignard
l.org Benjamin Gaignard (15): iio: adc: stm32: use devm_of_platform_populate() iio: dac: stm32: use devm_of_platform_populate() drm: zte: use devm_of_platform_populate() drm: msm: use devm_of_platform_populate() mfd: stm32-timers: use devm_of_platform_populate mfd: atmel

[PATCH 15/15] media: exynos4-is: use devm_of_platform_populate()

2017-05-29 Thread Benjamin Gaignard
Usage of devm_of_platform_populate() simplify driver code and save somes lines Signed-off-by: Benjamin Gaignard CC: Kyungmin Park CC: Sylwester Nawrocki CC: Mauro Carvalho Chehab CC: Kukjin Kim CC: Krzysztof Kozlowski CC: Javier Martinez Canillas CC: linux-media@vger.kernel.org CC: linux

[PATCH v5 0/2] cec: STM32 driver

2017-05-29 Thread Benjamin Gaignard
7;TV' Menu Language : fre Power Status : On Total: 10, Succeeded: 10, Failed: 0, Warnings: 0 Benjamin Gaignard (2): dt-bindings: media: stm32 cec driver cec: add STM32 cec driver .../devicetree/bindings/media/st,stm32-cec.txt

[PATCH v5 2/2] cec: add STM32 cec driver

2017-05-29 Thread Benjamin Gaignard
t platform_device *pdev) +{ + struct stm32_cec *cec = platform_get_drvdata(pdev); + + clk_unprepare(cec->clk_cec); + clk_unprepare(cec->clk_hdmi_cec); + + cec_unregister_adapter(cec->adap); + + return 0; +} + +static const struct of_device_id stm32_cec_of_match[] =

[PATCH v5 1/2] dt-bindings: media: stm32 cec driver

2017-05-29 Thread Benjamin Gaignard
Add bindings documentation for stm32 CEC driver. Signed-off-by: Benjamin Gaignard --- .../devicetree/bindings/media/st,stm32-cec.txt| 19 +++ 1 file changed, 19 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/st,stm32-cec.txt diff --git a

Re: [PATCH v4 0/2] cec: STM32 driver

2017-05-29 Thread Benjamin Gaignard
2017-05-29 10:54 GMT+02:00 Hans Verkuil : > Hi Benjamin, > > On 05/29/2017 10:34 AM, Benjamin Gaignard wrote: >> >> version 4: >> - rebased on Hans cec-config branch >> - rework bindings commit message >> - add notifier support > > > I really don&#

[PATCH v4 1/2] dt-bindings: media: stm32 cec driver

2017-05-29 Thread Benjamin Gaignard
Add bindings documentation for stm32 CEC driver. Signed-off-by: Benjamin Gaignard --- version 4: - rework commit message - add hdmi-phandle optional property .../devicetree/bindings/media/st,stm32-cec.txt | 22 ++ 1 file changed, 22 insertions(+) create mode 100644

[PATCH v4 2/2] cec: add STM32 cec driver

2017-05-29 Thread Benjamin Gaignard
k_hdmi_cec); + + cec_unregister_adapter(cec->adap); + + if (cec->notifier) + cec_notifier_put(cec->notifier); + + return 0; +} + +static const struct of_device_id stm32_cec_of_match[] = { + { .compatible = "st,stm32-cec" }, + { /* end node */ } +}; +MODULE_DEVICE_TABLE(of, stm32_cec_of_match); + +static struct platform_driver stm32_cec_driver = { + .probe = stm32_cec_probe, + .remove = stm32_cec_remove, + .driver = { + .name = CEC_NAME, + .of_match_table = stm32_cec_of_match, + }, +}; + +module_platform_driver(stm32_cec_driver); + +MODULE_AUTHOR("Benjamin Gaignard "); +MODULE_AUTHOR("Yannick Fertre "); +MODULE_DESCRIPTION("STMicroelectronics STM32 Consumer Electronics Control"); +MODULE_LICENSE("GPL v2"); -- 1.9.1

[PATCH v4 0/2] cec: STM32 driver

2017-05-29 Thread Benjamin Gaignard
Power Status : On Total: 10, Succeeded: 10, Failed: 0, Warnings: 0 Benjamin Gaignard (2): dt-bindings: media: stm32 cec driver cec: add STM32 cec driver .../devicetree/bindings/media/st,stm32-cec.txt | 22 ++ drivers/media/platform/Kconfig

Re: [PATCH v2 1/2] binding for stm32 cec driver

2017-05-23 Thread Benjamin Gaignard
2017-05-23 2:14 GMT+02:00 Rob Herring : > On Tue, May 16, 2017 at 02:56:22PM +0200, Benjamin Gaignard wrote: > > Commit message? is missing, sorry.. > > Preferred subject prefix is "dt-bindings: media: ..." ok > >> Signed-off-by: Benjamin Gaignard >>

[PATCH] cec: stih: fix typos in comments

2017-05-18 Thread Benjamin Gaignard
Minor fixes in comments Signed-off-by: Benjamin Gaignard --- drivers/media/platform/sti/cec/stih-cec.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/sti/cec/stih-cec.c b/drivers/media/platform/sti/cec/stih-cec.c index 39ff551..65ee143 100644

[PATCH] cec: stih: allow to use max CEC logical addresses

2017-05-18 Thread Benjamin Gaignard
Hardware could support up to 16 logical addresses which is more than needed by CEC specifications. Let use CEC_MAX_LOG_ADDRS instead of limited it on one. stih_cec_adap_log_addr() function was alredy written to support multiple addresses requests. Signed-off-by: Benjamin Gaignard --- drivers

[PATCH v3] cec: add STM32 cec driver

2017-05-16 Thread Benjamin Gaignard
ice *pdev) +{ + struct stm32_cec *cec = platform_get_drvdata(pdev); + + clk_unprepare(cec->clk_cec); + clk_unprepare(cec->clk_hdmi_cec); + + cec_unregister_adapter(cec->adap); + + cec_delete_adapter(cec->adap); + + return 0; +} + +static con

Re: [PATCH v2 2/2] cec: add STM32 cec driver

2017-05-16 Thread Benjamin Gaignard
2017-05-16 15:09 GMT+02:00 Hans Verkuil : > Looks good, except for the logical address handling that I think is wrong: > > On 16/05/17 14:56, Benjamin Gaignard wrote: >> This patch add cec driver for STM32 platforms. >> cec hardware block isn't not always used with hdmi

[PATCH v2 0/2] cec: STM32 driver

2017-05-16 Thread Benjamin Gaignard
: TV Vendor ID : 0x00903e OSD Name : 'TV' Menu Language : fre Power Status : On Total: 10, Succeeded: 10, Failed: 0, Warnings: 0 Benjamin Gaignard (2): binding f

[PATCH v2 1/2] binding for stm32 cec driver

2017-05-16 Thread Benjamin Gaignard
Signed-off-by: Benjamin Gaignard --- .../devicetree/bindings/media/st,stm32-cec.txt| 19 +++ 1 file changed, 19 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/st,stm32-cec.txt diff --git a/Documentation/devicetree/bindings/media/st,stm32

[PATCH v2 2/2] cec: add STM32 cec driver

2017-05-16 Thread Benjamin Gaignard
cec_unregister_adapter(cec->adap); + + cec_delete_adapter(cec->adap); + + return 0; +} + +static const struct of_device_id stm32_cec_of_match[] = { + { .compatible = "st,stm32-cec" }, + { /* end node */ } +}; +MODULE_DEVICE_TABLE(of, stm32_cec_of_match); + +static struct platform_driver stm32_cec_driver = { + .probe = stm32_cec_probe, + .remove = stm32_cec_remove, + .driver = { + .name = CEC_NAME, + .of_match_table = stm32_cec_of_match, + }, +}; + +module_platform_driver(stm32_cec_driver); + +MODULE_AUTHOR("Benjamin Gaignard "); +MODULE_AUTHOR("Yannick Fertre "); +MODULE_DESCRIPTION("STMicroelectronics STM32 Consumer Electronics Control"); +MODULE_LICENSE("GPL v2"); -- 1.9.1

Re: [PATCH 0/2] cec: STM32 driver

2017-05-16 Thread Benjamin Gaignard
2017-05-16 11:18 GMT+02:00 Hans Verkuil : > On 16/05/17 11:10, Benjamin Gaignard wrote: >> + Yannick who is the original writer of this driver (sorry) >> >> 2017-05-16 11:01 GMT+02:00 Benjamin Gaignard : >>> This serie of patches add cec driver for STM32 platfo

Re: [PATCH 0/2] cec: STM32 driver

2017-05-16 Thread Benjamin Gaignard
+ Yannick who is the original writer of this driver (sorry) 2017-05-16 11:01 GMT+02:00 Benjamin Gaignard : > This serie of patches add cec driver for STM32 platforms. > > This code doesn't implement cec notifier because STM32 doesn't > provide HDMI yet but it will be

[PATCH 0/2] cec: STM32 driver

2017-05-16 Thread Benjamin Gaignard
OSD Name : 'TV' Menu Language : fre Power Status : On Total: 10, Succeeded: 10, Failed: 0, Warnings: 0 Benjamin Gaignard (2): binding for stm32 cec driver cec: add STM32 cec driver .../devicetree/bindings/me

[PATCH 2/2] cec: add STM32 cec driver

2017-05-16 Thread Benjamin Gaignard
cec_delete_adapter(cec->adap); + + return 0; +} + +static const struct of_device_id stm32_cec_of_match[] = { + { .compatible = "st,stm32-cec" }, + { /* end node */ } +}; +MODULE_DEVICE_TABLE(of, stm32_cec_of_match); + +static struct platform_driver stm32_cec_driver = { + .probe = stm32_cec_probe, + .remove = stm32_cec_remove, + .driver = { + .name = CEC_NAME, + .of_match_table = stm32_cec_of_match, + }, +}; + +module_platform_driver(stm32_cec_driver); + +MODULE_AUTHOR("Benjamin Gaignard "); +MODULE_AUTHOR("Yannick Fertre "); +MODULE_DESCRIPTION("STMicroelectronics STM32 Consumer Electronics Control"); +MODULE_LICENSE("GPL v2"); -- 1.9.1

[PATCH 1/2] binding for stm32 cec driver

2017-05-16 Thread Benjamin Gaignard
Signed-off-by: Benjamin Gaignard --- .../devicetree/bindings/media/st,stm32-cec.txt| 19 +++ 1 file changed, 19 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/st,stm32-cec.txt diff --git a/Documentation/devicetree/bindings/media/st,stm32

Re: [PATCH for v4.12 1/3] cec: Kconfig cleanup

2017-04-18 Thread Benjamin Gaignard
-cec drivers depended on VIDEO_DEV instead of > CEC_CORE, fix that as well. > > Signed-off-by: Hans Verkuil Thanks for this clean up. Acked-by: Benjamin Gaignard > --- > MAINTAINERS | 2 -- > drivers/media/Kconfig| 26 --

Re: [PATCHv5 11/11] arm: sti: update sti-cec for CEC notifier support

2017-03-30 Thread Benjamin Gaignard
+ Patrice for sti DT 2017-03-29 16:15 GMT+02:00 Hans Verkuil : > From: Benjamin Gaignard > > To use CEC notifier sti CEC driver needs to get phandle > of the hdmi device. > > Signed-off-by: Benjamin Gaignard > Signed-off-by: Hans Verkuil > CC: devicet...@vger.kernel.or

Re: [PATCHv5 00/11] video/exynos/sti/cec: add CEC notifier & use in drivers

2017-03-29 Thread Benjamin Gaignard
s support for > it to the CEC framework and finally adds support to the s5p-cec driver, > which now can be moved out of staging. > > Also included is similar code for the STI platform, contributed by > Benjamin Gaignard. > > Tested the exynos code with my Odroid U3 exynos4 devb

Re: [RFC PATCHv2 00/21] Ion clean in preparation for moving out of staging

2017-03-29 Thread Benjamin Gaignard
/staging/android/ion/ion_of.c > delete mode 100644 drivers/staging/android/ion/ion_of.h > delete mode 100644 drivers/staging/android/ion/ion_priv.h > delete mode 100644 drivers/staging/android/ion/ion_test.c > delete mode 100644 drivers/staging/android/ion/tegra/Makefile > delete mode 100644 drivers/staging/android/ion/tegra/tegra_ion.c > delete mode 100644 drivers/staging/android/uapi/ion_test.h > > -- > 2.7.4 > -- Benjamin Gaignard Graphic Study Group Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog

Re: [RFC PATCHv2 02/21] cma: Introduce cma_for_each_area

2017-03-29 Thread Benjamin Gaignard
2017-03-18 1:54 GMT+01:00 Laura Abbott : > > Frameworks (e.g. Ion) may want to iterate over each possible CMA area to > allow for enumeration. Introduce a function to allow a callback. even outside ION rework that could be useful Reviewed-by: Benjamin Gaignard > > Signed-off-b

Re: [PATCH 2/8] [media] staging: st-cec: Use cec_get_drvdata()

2017-03-28 Thread Benjamin Gaignard
off-by: Jose Abreu > Cc: Benjamin Gaignard > --- > drivers/staging/media/st-cec/stih-cec.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/drivers/staging/media/st-cec/stih-cec.c > b/drivers/staging/media/st-cec/stih-cec.c > index 3c25638..52

Re: [RFC PATCH 00/12] Ion cleanup in preparation for moving out of staging

2017-03-14 Thread Benjamin Gaignard
2017-03-13 22:09 GMT+01:00 Laura Abbott : > On 03/12/2017 12:05 PM, Daniel Vetter wrote: >> On Sun, Mar 12, 2017 at 2:34 PM, Benjamin Gaignard >> wrote: >>> 2017-03-09 18:38 GMT+01:00 Laura Abbott : >>>> On 03/09/2017 02:00 AM, Benjamin Gaignard wrote: >

Re: [RFC PATCH 00/12] Ion cleanup in preparation for moving out of staging

2017-03-12 Thread Benjamin Gaignard
2017-03-09 18:38 GMT+01:00 Laura Abbott : > On 03/09/2017 02:00 AM, Benjamin Gaignard wrote: >> 2017-03-06 17:04 GMT+01:00 Daniel Vetter : >>> On Mon, Mar 06, 2017 at 11:58:05AM +0100, Mark Brown wrote: >>>> On Mon, Mar 06, 2017 at 11:40:41AM +0100, Daniel Vetter w

Re: [RFC PATCH 00/12] Ion cleanup in preparation for moving out of staging

2017-03-09 Thread Benjamin Gaignard
2017-03-06 17:04 GMT+01:00 Daniel Vetter : > On Mon, Mar 06, 2017 at 11:58:05AM +0100, Mark Brown wrote: >> On Mon, Mar 06, 2017 at 11:40:41AM +0100, Daniel Vetter wrote: >> >> > No one gave a thing about android in upstream, so Greg KH just dumped it >> > all into staging/android/. We've discussed

Re: [RFC PATCH 00/12] Ion cleanup in preparation for moving out of staging

2017-03-03 Thread Benjamin Gaignard
2017-03-03 11:27 GMT+01:00 Daniel Vetter : > On Fri, Mar 03, 2017 at 11:04:33AM +0100, Daniel Vetter wrote: >> On Thu, Mar 02, 2017 at 01:44:32PM -0800, Laura Abbott wrote: >> > Hi, >> > >> > There's been some recent discussions[1] about Ion-like frameworks. There's >> > apparently interest in just

[PATCH v3 3/3] arm: sti: update sti-cec for HPD notifier support

2017-02-17 Thread Benjamin Gaignard
To use HPD notifier sti CEC driver needs to get phandle of the hdmi device. Signed-off-by: Benjamin Gaignard Signed-off-by: Hans Verkuil CC: devicet...@vger.kernel.org version 3: - change hdmi phandle from "st,hdmi-handle" to "hdmi-handle" --- arch/arm/boot/dts/stih

[PATCH v3 1/3] sti: hdmi: add HPD notifier support

2017-02-17 Thread Benjamin Gaignard
Implement the HPD notifier support to allow CEC drivers to be informed when there is a new EDID and when a connect or disconnect happens. Signed-off-by: Benjamin Gaignard Signed-off-by: Hans Verkuil --- drivers/gpu/drm/sti/Kconfig| 1 + drivers/gpu/drm/sti/sti_hdmi.c | 14

[PATCH v3 0/3] video/sti/cec: add HPD notifier support

2017-02-17 Thread Benjamin Gaignard
instead of HDMI notifier - move stih-cec out of staging - rebase code on top of git://linuxtv.org/hverkuil/media_tree.git exynos4-cec branch - split DT modifications in a separate patch Benjamin Gaignard (3): sti: hdmi: add HPD notifier support stih-cec: add HPD notifier support arm: sti:

[PATCH v3 2/3] stih-cec: add HPD notifier support

2017-02-17 Thread Benjamin Gaignard
m_device *pdev) +{ + struct stih_cec *cec = platform_get_drvdata(pdev); + + cec_unregister_adapter(cec->adap); + hpd_notifier_put(cec->notifier); + + return 0; +} + +static const struct of_device_id stih_cec_match[] = { + { + .compatible = "st,st

Re: [RFC simple allocator v2 1/2] Create Simple Allocator module

2017-02-15 Thread Benjamin Gaignard
2017-02-14 20:59 GMT+01:00 Laurent Pinchart : > Hi Daniel, > > On Tuesday 14 Feb 2017 20:44:44 Daniel Vetter wrote: >> On Tue, Feb 14, 2017 at 8:39 PM, Laurent Pinchart wrote: >> > On Tuesday 14 Feb 2017 20:33:58 Daniel Vetter wrote: >> >> On Mon, Feb 13, 2017

Re: [RFC simple allocator v2 1/2] Create Simple Allocator module

2017-02-15 Thread Benjamin Gaignard
2017-02-14 20:30 GMT+01:00 Laurent Pinchart : > Hi Benjamin, > > Thank you for the patch. I've CC'ed the linux-api mailing list. > > On Monday 13 Feb 2017 15:45:05 Benjamin Gaignard wrote: >> This is the core of simple allocator module. >> It aim to offert

[RFC simple allocator v2 0/2] Simple allocator

2017-02-13 Thread Benjamin Gaignard
ator CMA OK\n"); end: close(fd); } [1] https://github.com/cubanismo/allocator Benjamin Gaignard (2): Create Simple Allocator module add CMA simple allocator module Documentation/simple-allocator.txt | 81 ++ drivers/Kconfig | 2

[RFC simple allocator v2 2/2] add CMA simple allocator module

2017-02-13 Thread Benjamin Gaignard
This patch add simple allocator for CMA regions. version 2: - fix size and page count computation Signed-off-by: Benjamin Gaignard --- drivers/simpleallocator/Kconfig| 7 + drivers/simpleallocator/Makefile | 1 + drivers/simpleallocator/simple-allocator-cma.c

[RFC simple allocator v2 1/2] Create Simple Allocator module

2017-02-13 Thread Benjamin Gaignard
This is the core of simple allocator module. It aim to offert one common ioctl to allocate specific memory. version 2: - rebased on 4.10-rc7 Signed-off-by: Benjamin Gaignard --- Documentation/simple-allocator.txt | 81 +++ drivers/Kconfig

Re: [PATCH] allow buffers allocation with DMABUF memory type

2017-02-13 Thread Benjamin Gaignard
dmabuf in v4l2 > > Regards, > > Hans > > On 04/23/2015 11:22 AM, Benjamin Gaignard wrote: >> Until now the only way to make the driver allocate buffers and >> share them using dma_buf was to use V4L2_MEMORY_MMAP memory type. >> Use of MMAP memory type is a

[RFC simple allocator v1 0/2] Simple allocator

2017-01-20 Thread Benjamin Gaignard
"); } memset(mem, 0xFF, LENGTH); munmap(mem, LENGTH); printf("test simple allocator CMA OK\n"); end: close(fd); } [1] https://github.com/cubanismo/allocator Benjamin Gaignard (2): Create Simple Allocator module add CMA simple allocator

[RFC simple allocator v1 2/2] add CMA simple allocator module

2017-01-20 Thread Benjamin Gaignard
This patch add simple allocator for CMA regions Signed-off-by: Benjamin Gaignard --- drivers/simpleallocator/Kconfig| 7 + drivers/simpleallocator/Makefile | 1 + drivers/simpleallocator/simple-allocator-cma.c | 187 + 3 files changed

[RFC simple allocator v1 1/2] Create Simple Allocator module

2017-01-20 Thread Benjamin Gaignard
This is the core of simple allocator module. It aim to offert one common ioctl to allocate specific memory. Signed-off-by: Benjamin Gaignard --- Documentation/simple-allocator.txt | 81 +++ drivers/Kconfig | 2 + drivers/Makefile

[PATCH v2 3/3] arm: sti: update sti-cec for HPD notifier support

2017-01-03 Thread Benjamin Gaignard
To use HPD notifier sti CEC driver needs to get phandle of the hdmi device. Signed-off-by: Benjamin Gaignard --- arch/arm/boot/dts/stih407-family.dtsi | 12 arch/arm/boot/dts/stih410.dtsi| 13 + 2 files changed, 13 insertions(+), 12 deletions(-) diff --git a

[PATCH v2 2/3] stih-cec: add HPD notifier support

2017-01-03 Thread Benjamin Gaignard
struct stih_cec *cec = platform_get_drvdata(pdev); + + cec_unregister_adapter(cec->adap); + hpd_notifier_put(cec->notifier); + + return 0; +} + +static const struct of_device_id stih_cec_match[] = { + { + .compatible = "st,stih-cec", +

[PATCH v2 1/3] sti: hdmi: add HPD notifier support

2017-01-03 Thread Benjamin Gaignard
Implement the HPD notifier support to allow CEC drivers to be informed when there is a new EDID and when a connect or disconnect happens. Signed-off-by: Benjamin Gaignard --- version 2: - use HPD notifier instead of HDMI notifier --- drivers/gpu/drm/sti/Kconfig| 1 + drivers/gpu/drm/sti

[PATCH v2 0/3] video/sti/cec: add HPD notifier support

2017-01-03 Thread Benjamin Gaignard
tree.git exynos4-cec branch - split DT modifications in a separate patch Regards, Benjamin Benjamin Gaignard (3): sti: hdmi: add HPD notifier support stih-cec: add HPD notifier support arm: sti: update sti-cec for HPD notifier support .../devicetree/bindings/media/stih-cec.txt | 2

[PATCH 0/2] video/sti/cec: add HDMI notifier support

2016-12-14 Thread Benjamin Gaignard
c after applying those patches ? Regards, Benjamin Benjamin Gaignard (2): sti: hdmi: add HDMI notifier support stih-cec: add hdmi-notifier support .../devicetree/bindings/media/stih-cec.txt | 2 ++ arch/arm/boot/dts/stih407-family.dtsi | 12 - arch/arm/boot/dts/stih410

[PATCH 1/2] sti: hdmi: add HDMI notifier support

2016-12-14 Thread Benjamin Gaignard
Implement the HDMI notifier support to allow CEC drivers to be informed when there is a new EDID and when a connect or disconnect happens. Signed-off-by: Benjamin Gaignard --- drivers/gpu/drm/sti/Kconfig| 1 + drivers/gpu/drm/sti/sti_hdmi.c | 15 +++ drivers/gpu/drm/sti

[PATCH 2/2] stih-cec: add hdmi-notifier support

2016-12-14 Thread Benjamin Gaignard
stih407-family.dtsi accordingly. Signed-off-by: Benjamin Gaignard --- .../devicetree/bindings/media/stih-cec.txt | 2 ++ arch/arm/boot/dts/stih407-family.dtsi | 12 - arch/arm/boot/dts/stih410.dtsi | 15 ++- drivers/staging/media/st-cec

Re: [PATCH] staging: st-cec: add parentheses around complex macros

2016-11-04 Thread Benjamin Gaignard
Thanks, Acked-by: Benjamin Gaignard 2016-11-04 8:58 GMT+01:00 Maninder Singh : > This patch fixes the following checkpatch.pl error: > ERROR: Macros with complex values should be enclosed in parentheses > > Signed-off-by: Maninder Singh > --- > drivers/staging/media/st-cec

Re: [PATCH 5/5] [media] st-cec: Fix module autoload

2016-10-18 Thread Benjamin Gaignard
Thanks, Acked-by: Benjamin Gaignard 2016-10-17 17:44 GMT+02:00 Javier Martinez Canillas : > If the driver is built as a module, autoload won't work because the module > alias information is not filled. So user-space can't match the registered > device with the corresponding

Re: [PATCH v10 0/3] Secure Memory Allocation Framework

2016-10-07 Thread Benjamin Gaignard
o not impose any specific constraints. How "liballoc-ion.so" > backend figures out how to map constraints/usage to a heap is a bit > hand-wavey at the moment. > > BR, > -R > > On Wed, Oct 5, 2016 at 9:40 AM, Benjamin Gaignard > wrote: >> because with ion it is up

Re: [PATCH v10 0/3] Secure Memory Allocation Framework

2016-10-06 Thread Benjamin Gaignard
that if you can export more device information to userland we can get access to them in kernel too. 2016-10-05 15:43 GMT+02:00 Daniel Vetter : > On Wed, Oct 05, 2016 at 03:40:14PM +0200, Benjamin Gaignard wrote: >> because with ion it is up to userland to decide which heap to use >>

Re: [PATCH v10 0/3] Secure Memory Allocation Framework

2016-10-05 Thread Benjamin Gaignard
smaf. Benjamin 2016-10-05 15:19 GMT+02:00 Daniel Vetter : > On Tue, Oct 04, 2016 at 01:47:21PM +0200, Benjamin Gaignard wrote: >> version 10 changes: >> - rebased on kernel 4.8 tag >> - minor typo fix >> >> version 9 changes: >> - rebased on 4.8-rc5 >>

[PATCH v10 2/3] SMAF: add CMA allocator

2016-10-04 Thread Benjamin Gaignard
be compatible with ARM 64bits architecture Signed-off-by: Benjamin Gaignard --- drivers/smaf/Kconfig| 6 ++ drivers/smaf/Makefile | 1 + drivers/smaf/smaf-cma.c | 186 3 files changed, 193 insertions(+) create mode 100644 drivers/smaf

[PATCH v10 1/3] create SMAF module

2016-10-04 Thread Benjamin Gaignard
check if the devices are compatible or not with it allocation method. Securing module (smaf-secure.h): The way of how securing memory it is done is platform specific. Secure module is responsible of grant/revoke memory access. Signed-off-by: Benjamin Gaignard --- drivers/Kconfig

[PATCH v10 3/3] SMAF: add test secure module

2016-10-04 Thread Benjamin Gaignard
This module is allow testing secure calls of SMAF. Signed-off-by: Benjamin Gaignard --- drivers/smaf/Kconfig | 6 +++ drivers/smaf/Makefile | 1 + drivers/smaf/smaf-testsecure.c | 90 ++ 3 files changed, 97 insertions(+) create mode

[PATCH v10 0/3] Secure Memory Allocation Framework

2016-10-04 Thread Benjamin Gaignard
ted by Sumit Semwal work about constraint aware allocator. Benjamin Gaignard (3): create SMAF module SMAF: add CMA allocator SMAF: add test secure module drivers/Kconfig| 2 + drivers/Makefile | 1 + drivers/smaf/Kconfig | 17 + drivers/smaf/Makefile

Re: [PATCH -next] staging: media: stih-cec: remove unused including

2016-09-28 Thread Benjamin Gaignard
Acked-by: Benjamin Gaignard 2016-09-28 17:13 GMT+02:00 Wei Yongjun : > From: Wei Yongjun > > Remove including that don't need it. > > Signed-off-by: Wei Yongjun > --- > drivers/staging/media/st-cec/stih-cec.c | 1 - > 1 file changed, 1 deletion(-) > > di

[PATCH v3 2/4] add stih-cec driver

2016-09-15 Thread Benjamin Gaignard
This patch implement CEC driver for stih4xx platform. Driver compliance has been test with cec-ctl and cec-compliance tools. Signed-off-by: Benjamin Gaignard --- drivers/staging/media/Kconfig | 2 + drivers/staging/media/Makefile | 1 + drivers/staging/media/st-cec

[PATCH v3 3/4] add stih-cec driver into DT

2016-09-15 Thread Benjamin Gaignard
Signed-off-by: Benjamin Gaignard --- arch/arm/boot/dts/stih407-family.dtsi | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi index d294e82..9f88086 100644 --- a/arch/arm/boot/dts/stih407-family.dtsi

[PATCH v3 0/4] STIH CEC driver

2016-09-15 Thread Benjamin Gaignard
PA: 1.0.0.0, LA mask: 0x Event: State Change: PA: 1.0.0.0, LA mask: 0x4000 Event: State Change: PA: 1.0.0.0, LA mask: 0x Event: State Change: PA: 1.0.0.0, LA mask: 0x4000 Event: State Change: PA: 1.0.0.0, LA mask: 0x Event: State Change: PA: 1.0.0.0, LA mask: 0x0008 Benjamin Gaignard (4):

[PATCH v3 4/4] add maintainer for stih-cec driver

2016-09-15 Thread Benjamin Gaignard
Signed-off-by: Benjamin Gaignard --- MAINTAINERS | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 0bbe4b1..f541117 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -10281,6 +10281,13 @@ S: Maintained F: Documentation/devicetree/bindings/serial/ F

[PATCH v3 1/4] bindings for stih-cec driver

2016-09-15 Thread Benjamin Gaignard
Add bindings documentation for stih-cec driver. Signed-off-by: Benjamin Gaignard --- .../devicetree/bindings/media/stih-cec.txt | 25 ++ 1 file changed, 25 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/stih-cec.txt diff --git a

Re: [PATCH v2 2/4] add stih-cec driver

2016-09-14 Thread Benjamin Gaignard
Fixed and tested, thanks. I will wait for any others comments before send a v3. Regards, Benjamin 2016-09-14 14:58 GMT+02:00 Hans Verkuil : > Hi Benjamin, > > Just one comment: > > On 09/14/2016 01:24 PM, Benjamin Gaignard wrote: >> This patch implement CEC driver

[PATCH v2 4/4] add maintainer for stih-cec driver

2016-09-14 Thread Benjamin Gaignard
Signed-off-by: Benjamin Gaignard --- MAINTAINERS | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 0bbe4b1..f541117 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -10281,6 +10281,13 @@ S: Maintained F: Documentation/devicetree/bindings/serial/ F

[PATCH v2 3/4] add stih-cec driver into DT

2016-09-14 Thread Benjamin Gaignard
Signed-off-by: Benjamin Gaignard --- arch/arm/boot/dts/stih407-family.dtsi | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/boot/dts/stih407-family.dtsi b/arch/arm/boot/dts/stih407-family.dtsi index d294e82..9f88086 100644 --- a/arch/arm/boot/dts/stih407-family.dtsi

[PATCH v2 1/4] bindings for stih-cec driver

2016-09-14 Thread Benjamin Gaignard
Add bindings documentation for stih-cec driver. Signed-off-by: Benjamin Gaignard --- .../devicetree/bindings/media/stih-cec.txt | 25 ++ 1 file changed, 25 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/stih-cec.txt diff --git a

[PATCH v2 2/4] add stih-cec driver

2016-09-14 Thread Benjamin Gaignard
This patch implement CEC driver for stih4xx platform. Driver compliance has been test with cec-ctl and cec-compliance tools. Signed-off-by: Benjamin Gaignard --- drivers/staging/media/Kconfig | 2 + drivers/staging/media/Makefile | 1 + drivers/staging/media/st-cec

[PATCH v2 0/4] STIH CEC driver

2016-09-14 Thread Benjamin Gaignard
nt: State Change: PA: 1.0.0.0, LA mask: 0x4000 Event: State Change: PA: 1.0.0.0, LA mask: 0x Event: State Change: PA: 1.0.0.0, LA mask: 0x0008 Benjamin Gaignard (4): bindings for stih-cec driver add stih-cec driver add stih-cec driver into DT add maintainer for stih-cec driver ...

[PATCH 4/4] add maintainer for stih-cec driver

2016-09-14 Thread Benjamin Gaignard
Signed-off-by: Benjamin Gaignard --- MAINTAINERS | 7 +++ 1 file changed, 7 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 0bbe4b1..f541117 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -10281,6 +10281,13 @@ S: Maintained F: Documentation/devicetree/bindings/serial/ F

[PATCH 2/4] add stih-cec driver

2016-09-14 Thread Benjamin Gaignard
This patch implement CEC driver for stih4xx platform. Driver compliance has been test with cec-ctl and cec-compliance tools. Signed-off-by: Benjamin Gaignard --- drivers/staging/media/Kconfig | 2 + drivers/staging/media/Makefile | 1 + drivers/staging/media/st-cec

[PATCH 3/4] add stih-cec driver into DT

2016-09-14 Thread Benjamin Gaignard
Signed-off-by: Benjamin Gaignard --- arch/arm/boot/dts/stih410.dtsi | 12 1 file changed, 12 insertions(+) diff --git a/arch/arm/boot/dts/stih410.dtsi b/arch/arm/boot/dts/stih410.dtsi index 18ed1ad..440c4bd 100644 --- a/arch/arm/boot/dts/stih410.dtsi +++ b/arch/arm/boot/dts/stih410

[PATCH 1/4] bindings for stih-cec driver

2016-09-14 Thread Benjamin Gaignard
Add bindings documentation for stih-cec driver. Signed-off-by: Benjamin Gaignard --- .../devicetree/bindings/media/stih-cec.txt | 25 ++ 1 file changed, 25 insertions(+) create mode 100644 Documentation/devicetree/bindings/media/stih-cec.txt diff --git a

[PATCH 0/4] STIH CEC driver

2016-09-14 Thread Benjamin Gaignard
nt: State Change: PA: 1.0.0.0, LA mask: 0x0008 Benjamin Gaignard (4): bindings for stih-cec driver add stih-cec driver add stih-cec driver into DT add maintainer for stih-cec driver .../devicetree/bindings/media/stih-cec.txt | 25 ++ MAINTAINERS|

  1   2   >