[PATCH v2] media: bcm2835-unicam: Fix log status runtime access

Jai Luthra jai.luthra at ideasonboard.com
Sat Jun 6 20:42:44 PDT 2026


Quoting Eugen Hristev (2026-05-22 20:58:50)
> When requesting log status, the block might be powered off, but registers
> are being read.
> Avoid reading the registers if the device is not resumed, thus also avoid
> powering up the device just for log status.
> 
> Fixes: 392cd78d495f ("media: bcm2835-unicam: Add support for CCP2/CSI2 camera interface")
> Signed-off-by: Eugen Hristev <ehristev at kernel.org>

Reviewed-by: Jai Luthra <jai.luthra at ideasonboard.com>

> ---
> Changes in v2:
> - changed to use pm_runtime_get_if_active()
> - add corresponding put()
> - Link to v1: https://patch.msgid.link/20260521-bcmpipm-v1-1-3eba88d88045@kernel.org
> 
> To: Raspberry Pi Kernel Maintenance <kernel-list at raspberrypi.com>
> To: Mauro Carvalho Chehab <mchehab at kernel.org>
> To: Florian Fainelli <florian.fainelli at broadcom.com>
> To: Ray Jui <rjui at broadcom.com>
> To: Scott Branden <sbranden at broadcom.com>
> To: Broadcom internal kernel review list <bcm-kernel-feedback-list at broadcom.com>
> To: Sakari Ailus <sakari.ailus at linux.intel.com>
> To: Jean-Michel Hautbois <jeanmichel.hautbois at yoseli.org>
> To: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
> To: Hans Verkuil <hverkuil at kernel.org>
> To: Naushir Patuck <naush at raspberrypi.com>
> Cc: Dave Stevenson <dave.stevenson at raspberrypi.com>
> Cc: linux-media at vger.kernel.org
> Cc: linux-rpi-kernel at lists.infradead.org
> Cc: linux-arm-kernel at lists.infradead.org
> Cc: linux-kernel at vger.kernel.org

[snip]

Thanks,
    Jai



More information about the linux-arm-kernel mailing list