media: bcm2835-unicam: Drop WARN on uing direct cache alias
authorDave Stevenson <dave.stevenson@raspberrypi.com>
Thu, 27 Aug 2020 15:30:26 +0000 (16:30 +0100)
committerDom Cobley <popcornmix@gmail.com>
Mon, 19 Feb 2024 11:33:03 +0000 (11:33 +0000)
Pi 0&1 pass all ARM accesses through the VPU L2 cache, therefore
the dma-ranges property sets the cache alias bits to other
than the direct alias, hence this WARN was firing.

It was overprotective coding, so assume that everything is OK
with the dma-ranges, and remove the WARN.

Signed-off-by: Dave Stevenson <dave.stevenson@raspberrypi.com>
drivers/media/platform/bcm2835/bcm2835-unicam.c

index 1e9c9f5..85e98ea 100644 (file)
@@ -711,13 +711,6 @@ static void unicam_wr_dma_addr(struct unicam_device *dev, dma_addr_t dmaaddr,
 {
        dma_addr_t endaddr = dmaaddr + buffer_size;
 
-       /*
-        * dmaaddr and endaddr should be a 32-bit address with the top two bits
-        * set to 0x3 to signify uncached access through the Videocore memory
-        * controller.
-        */
-       WARN_ON((dmaaddr >> 30) != 0x3 || (endaddr >> 30) != 0x3);
-
        if (pad_id == IMAGE_PAD) {
                reg_write(dev, UNICAM_IBSA0, dmaaddr);
                reg_write(dev, UNICAM_IBEA0, endaddr);