media: platform: stm32: Add check for clk_enable()Add check for the return value of clk_enable() to gurantee the success.Fixes: 002e8f0d5927 ("media: stm32-dma2d: STM32 DMA2D driver")Cc: stable@
media: platform: stm32: Add check for clk_enable()Add check for the return value of clk_enable() to gurantee the success.Fixes: 002e8f0d5927 ("media: stm32-dma2d: STM32 DMA2D driver")Cc: [email protected]Signed-off-by: Jiasheng Jiang <[email protected]>Signed-off-by: Hans Verkuil <[email protected]>
show more ...
media: platform: drop vb2_ops_wait_prepare/finishSince commit 88785982a19d ("media: vb2: use lock if wait_prepare/finishare NULL") it is no longer needed to set the wait_prepare/finishvb2_ops cal
media: platform: drop vb2_ops_wait_prepare/finishSince commit 88785982a19d ("media: vb2: use lock if wait_prepare/finishare NULL") it is no longer needed to set the wait_prepare/finishvb2_ops callbacks as long as the lock field in vb2_queue is set.Since the vb2_ops_wait_prepare/finish callbacks already rely on that field,we can safely drop these callbacks.This simplifies the code and this is a step towards the goal of deletingthese callbacks.Signed-off-by: Hans Verkuil <[email protected]>Reviewed-by: Neil Armstrong <[email protected]> # for meson-ge2dAcked-by: Andrzej Pietrasiewicz <[email protected]>
media: Switch back to struct platform_driver::remove()After commit 0edb555a65d1 ("platform: Make platform_driver::remove()return void") .remove() is (again) the right callback to implement forpla
media: Switch back to struct platform_driver::remove()After commit 0edb555a65d1 ("platform: Make platform_driver::remove()return void") .remove() is (again) the right callback to implement forplatform drivers.Convert all platform drivers below drivers/media to use .remove(), withthe eventual goal to drop struct platform_driver::remove_new(). As.remove() and .remove_new() have the same prototypes, conversion is doneby just changing the structure member name in the driver initializer.Signed-off-by: Uwe Kleine-König <[email protected]>Signed-off-by: Hans Verkuil <[email protected]>
media: dma2d: Convert to platform remove callback returning voidThe .remove() callback for a platform driver returns an int which makesmany driver authors wrongly assume it's possible to do error
media: dma2d: Convert to platform remove callback returning voidThe .remove() callback for a platform driver returns an int which makesmany driver authors wrongly assume it's possible to do error handling byreturning an error code. However the value returned is (mostly) ignoredand this typically results in resource leaks. To improve here there is aquest to make the remove callback return void. In the first step of thisquest all drivers are converted to .remove_new() which already returnsvoid.Trivially convert this driver from always returning zero in the removecallback to the void returning variant.Signed-off-by: Uwe Kleine-König <[email protected]>Signed-off-by: Hans Verkuil <[email protected]>
media: stm32: dma2d: remove unused fb_bufDrop the unused struct v4l2_framebuffer fb_buf in struct dma2d_ctx.Signed-off-by: Hans Verkuil <[email protected]>CC: Hugues Fruchet <hugues.fruch
media: stm32: dma2d: remove unused fb_bufDrop the unused struct v4l2_framebuffer fb_buf in struct dma2d_ctx.Signed-off-by: Hans Verkuil <[email protected]>CC: Hugues Fruchet <[email protected]>Signed-off-by: Mauro Carvalho Chehab <[email protected]>
media: platform: stm32: use devm_platform_get_and_ioremap_resource()Convert platform_get_resource(), devm_ioremap_resource() to a singlecall to devm_platform_get_and_ioremap_resource(), as this is
media: platform: stm32: use devm_platform_get_and_ioremap_resource()Convert platform_get_resource(), devm_ioremap_resource() to a singlecall to devm_platform_get_and_ioremap_resource(), as this is exactlywhat this function does.Signed-off-by: Ye Xingchen <[email protected]>Signed-off-by: Hans Verkuil <[email protected]>Signed-off-by: Mauro Carvalho Chehab <[email protected]>
media: platform: place stm32/ and sti/ under st/ dirAs the end goal is to have platform drivers split by vendor,move both stm32/ and sti/ for them to be inside st/ directory.Acked-by: Hugues Fru
media: platform: place stm32/ and sti/ under st/ dirAs the end goal is to have platform drivers split by vendor,move both stm32/ and sti/ for them to be inside st/ directory.Acked-by: Hugues Fruchet <[email protected]>Signed-off-by: Mauro Carvalho Chehab <[email protected]>