| Message ID | 20200416115047.233720-4-paul.kocialkowski@bootlin.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | media: rockchip: rga: PX30 support and YUV2YUV fix | expand |
On 16/04/2020 13:50, Paul Kocialkowski wrote: > The PX30 SoC has a RGA block, so add the associated compatible to > support it. > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> > --- > drivers/media/platform/rockchip/rga/rga.c | 4 +++- > 1 file changed, 3 insertions(+), 1 deletion(-) > > diff --git a/drivers/media/platform/rockchip/rga/rga.c b/drivers/media/platform/rockchip/rga/rga.c > index e9ff12b6b5bb..0ebeb9b4c14a 100644 > --- a/drivers/media/platform/rockchip/rga/rga.c > +++ b/drivers/media/platform/rockchip/rga/rga.c > @@ -880,7 +880,6 @@ static int rga_probe(struct platform_device *pdev) > rga->cmdbuf_virt = dma_alloc_attrs(rga->dev, RGA_CMDBUF_SIZE, > &rga->cmdbuf_phy, GFP_KERNEL, > DMA_ATTR_WRITE_COMBINE); > - Spurious change? Regards, Hans > rga->src_mmu_pages = > (unsigned int *)__get_free_pages(GFP_KERNEL | __GFP_ZERO, 3); > rga->dst_mmu_pages = > @@ -955,6 +954,9 @@ static const struct dev_pm_ops rga_pm = { > }; > > static const struct of_device_id rockchip_rga_match[] = { > + { > + .compatible = "rockchip,px30-rga", > + }, > { > .compatible = "rockchip,rk3288-rga", > }, >
Hi, On Thu 16 Apr 20, 13:58, Hans Verkuil wrote: > On 16/04/2020 13:50, Paul Kocialkowski wrote: > > The PX30 SoC has a RGA block, so add the associated compatible to > > support it. > > > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> > > --- > > drivers/media/platform/rockchip/rga/rga.c | 4 +++- > > 1 file changed, 3 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/media/platform/rockchip/rga/rga.c b/drivers/media/platform/rockchip/rga/rga.c > > index e9ff12b6b5bb..0ebeb9b4c14a 100644 > > --- a/drivers/media/platform/rockchip/rga/rga.c > > +++ b/drivers/media/platform/rockchip/rga/rga.c > > @@ -880,7 +880,6 @@ static int rga_probe(struct platform_device *pdev) > > rga->cmdbuf_virt = dma_alloc_attrs(rga->dev, RGA_CMDBUF_SIZE, > > &rga->cmdbuf_phy, GFP_KERNEL, > > DMA_ATTR_WRITE_COMBINE); > > - > > Spurious change? Ah sorry about that. Will fix in v2. Cheers, Paul > Regards, > > Hans > > > rga->src_mmu_pages = > > (unsigned int *)__get_free_pages(GFP_KERNEL | __GFP_ZERO, 3); > > rga->dst_mmu_pages = > > @@ -955,6 +954,9 @@ static const struct dev_pm_ops rga_pm = { > > }; > > > > static const struct of_device_id rockchip_rga_match[] = { > > + { > > + .compatible = "rockchip,px30-rga", > > + }, > > { > > .compatible = "rockchip,rk3288-rga", > > }, > > >
diff --git a/drivers/media/platform/rockchip/rga/rga.c b/drivers/media/platform/rockchip/rga/rga.c index e9ff12b6b5bb..0ebeb9b4c14a 100644 --- a/drivers/media/platform/rockchip/rga/rga.c +++ b/drivers/media/platform/rockchip/rga/rga.c @@ -880,7 +880,6 @@ static int rga_probe(struct platform_device *pdev) rga->cmdbuf_virt = dma_alloc_attrs(rga->dev, RGA_CMDBUF_SIZE, &rga->cmdbuf_phy, GFP_KERNEL, DMA_ATTR_WRITE_COMBINE); - rga->src_mmu_pages = (unsigned int *)__get_free_pages(GFP_KERNEL | __GFP_ZERO, 3); rga->dst_mmu_pages = @@ -955,6 +954,9 @@ static const struct dev_pm_ops rga_pm = { }; static const struct of_device_id rockchip_rga_match[] = { + { + .compatible = "rockchip,px30-rga", + }, { .compatible = "rockchip,rk3288-rga", },
The PX30 SoC has a RGA block, so add the associated compatible to support it. Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> --- drivers/media/platform/rockchip/rga/rga.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-)