drm/ingenic: Switch IPU plane to type OVERLAY
It should have been an OVERLAY from the beginning. The documentation
stipulates that there should be an unique PRIMARY plane per CRTC.
Fixes: fc1acf317b ("drm/ingenic: Add support for the IPU")
Cc: <stable@vger.kernel.org> # 5.8+
Signed-off-by: Paul Cercueil <paul@crapouillou.net>
Acked-by: Simon Ser <contact@emersion.fr>
Link: https://patchwork.freedesktop.org/patch/msgid/20210329175046.214629-2-paul@crapouillou.net
			
			
This commit is contained in:
		
							parent
							
								
									d17685260b
								
							
						
					
					
						commit
						68b433fe69
					
				| @ -419,7 +419,7 @@ static void ingenic_drm_plane_enable(struct ingenic_drm *priv, | ||||
| 	unsigned int en_bit; | ||||
| 
 | ||||
| 	if (priv->soc_info->has_osd) { | ||||
| 		if (plane->type == DRM_PLANE_TYPE_PRIMARY) | ||||
| 		if (plane != &priv->f0) | ||||
| 			en_bit = JZ_LCD_OSDC_F1EN; | ||||
| 		else | ||||
| 			en_bit = JZ_LCD_OSDC_F0EN; | ||||
| @ -434,7 +434,7 @@ void ingenic_drm_plane_disable(struct device *dev, struct drm_plane *plane) | ||||
| 	unsigned int en_bit; | ||||
| 
 | ||||
| 	if (priv->soc_info->has_osd) { | ||||
| 		if (plane->type == DRM_PLANE_TYPE_PRIMARY) | ||||
| 		if (plane != &priv->f0) | ||||
| 			en_bit = JZ_LCD_OSDC_F1EN; | ||||
| 		else | ||||
| 			en_bit = JZ_LCD_OSDC_F0EN; | ||||
| @ -461,8 +461,7 @@ void ingenic_drm_plane_config(struct device *dev, | ||||
| 
 | ||||
| 	ingenic_drm_plane_enable(priv, plane); | ||||
| 
 | ||||
| 	if (priv->soc_info->has_osd && | ||||
| 	    plane->type == DRM_PLANE_TYPE_PRIMARY) { | ||||
| 	if (priv->soc_info->has_osd && plane != &priv->f0) { | ||||
| 		switch (fourcc) { | ||||
| 		case DRM_FORMAT_XRGB1555: | ||||
| 			ctrl |= JZ_LCD_OSDCTRL_RGB555; | ||||
| @ -510,7 +509,7 @@ void ingenic_drm_plane_config(struct device *dev, | ||||
| 	} | ||||
| 
 | ||||
| 	if (priv->soc_info->has_osd) { | ||||
| 		if (plane->type == DRM_PLANE_TYPE_PRIMARY) { | ||||
| 		if (plane != &priv->f0) { | ||||
| 			xy_reg = JZ_REG_LCD_XYP1; | ||||
| 			size_reg = JZ_REG_LCD_SIZE1; | ||||
| 		} else { | ||||
| @ -561,7 +560,7 @@ static void ingenic_drm_plane_atomic_update(struct drm_plane *plane, | ||||
| 		height = newstate->src_h >> 16; | ||||
| 		cpp = newstate->fb->format->cpp[0]; | ||||
| 
 | ||||
| 		if (!priv->soc_info->has_osd || plane->type == DRM_PLANE_TYPE_OVERLAY) | ||||
| 		if (!priv->soc_info->has_osd || plane == &priv->f0) | ||||
| 			hwdesc = &priv->dma_hwdescs->hwdesc_f0; | ||||
| 		else | ||||
| 			hwdesc = &priv->dma_hwdescs->hwdesc_f1; | ||||
|  | ||||
| @ -767,7 +767,7 @@ static int ingenic_ipu_bind(struct device *dev, struct device *master, void *d) | ||||
| 
 | ||||
| 	err = drm_universal_plane_init(drm, plane, 1, &ingenic_ipu_plane_funcs, | ||||
| 				       soc_info->formats, soc_info->num_formats, | ||||
| 				       NULL, DRM_PLANE_TYPE_PRIMARY, NULL); | ||||
| 				       NULL, DRM_PLANE_TYPE_OVERLAY, NULL); | ||||
| 	if (err) { | ||||
| 		dev_err(dev, "Failed to init plane: %i\n", err); | ||||
| 		return err; | ||||
|  | ||||
		Loading…
	
		Reference in New Issue
	
	Block a user