linux-next: manual merge of the mali-dp tree with the drm-misc tree

linux-next: manual merge of the mali-dp tree with the drm-misc tree

[Stephen Rothwell]

[-- Attachment #1: Type: text/plain, Size: 1448 bytes --]

Hi Liviu,

Today's linux-next merge of the mali-dp tree got a conflict in:

  drivers/gpu/drm/arm/malidp_planes.c

between commit:

  81af63a4af82 ("drm: Don't pass clip to drm_atomic_helper_check_plane_state()")

from the drm-misc tree and commit:

  4968211e7b8f ("drm: mali-dp: Uninitialized variable in malidp_se_check_scaling()")

from the mali-dp tree.

I fixed it up (see below) and can carry the fix as necessary. This
is now fixed as far as linux-next is concerned, but any non trivial
conflicts should be mentioned to your upstream maintainer when your tree
is submitted for merging.  You may also want to consider cooperating
with the maintainer of the conflicting tree to minimise any particularly
complex conflicts.

-- 
Cheers,
Stephen Rothwell

diff --cc drivers/gpu/drm/arm/malidp_planes.c
index ee32361c87ac,b2d11df6b5e1..000000000000
--- a/drivers/gpu/drm/arm/malidp_planes.c
+++ b/drivers/gpu/drm/arm/malidp_planes.c
@@@ -147,7 -147,13 +146,9 @@@ static int malidp_se_check_scaling(stru
  	if (!crtc_state)
  		return -EINVAL;
  
 -	if (crtc_state->enable)
 -		drm_mode_get_hv_timing(&crtc_state->mode,
 -				       &clip.x2, &clip.y2);
 -
+ 	mc = to_malidp_crtc_state(crtc_state);
+ 
 -	ret = drm_atomic_helper_check_plane_state(state, crtc_state, &clip,
 +	ret = drm_atomic_helper_check_plane_state(state, crtc_state,
  						  0, INT_MAX, true, true);
  	if (ret)
  		return ret;

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

linux-next: manual merge of the mali-dp tree with the drm-misc tree

[Stephen Rothwell]

Hi Liviu,

Today's linux-next merge of the mali-dp tree got a conflict in:

  drivers/gpu/drm/arm/malidp_planes.c

between commit:

  ea0e1ce20f73 ("drm/arm: Use per-plane rotation property")

from the drm-misc tree and commit:

  9ebb89762c30 ("drm: mali-dp: Refactor plane initialisation")

from the mali-dp tree.

I fixed it up (see below) and can carry the fix as necessary. This
is now fixed as far as linux-next is concerned, but any non trivial
conflicts should be mentioned to your upstream maintainer when your tree
is submitted for merging.  You may also want to consider cooperating
with the maintainer of the conflicting tree to minimise any particularly
complex conflicts.

-- 
Cheers,
Stephen Rothwell

diff --cc drivers/gpu/drm/arm/malidp_planes.c
index abaca03b9d36,9020c0d8399c..000000000000
--- a/drivers/gpu/drm/arm/malidp_planes.c
+++ b/drivers/gpu/drm/arm/malidp_planes.c
@@@ -254,23 -284,33 +284,30 @@@ int malidp_de_planes_init(struct drm_de
  		if (ret < 0)
  			goto cleanup;
  
+ 		drm_plane_helper_add(&plane->base,
+ 				     &malidp_de_plane_helper_funcs);
+ 		plane->hwdev = malidp->dev;
+ 		plane->layer = &map->layers[i];
+ 
+ 		/* Skip the features which the SMART layer doesn't have */
+ 		if (id == DE_SMART)
+ 			continue;
+ 
 -		if (!drm->mode_config.rotation_property) {
 +		/* SMART layer can't be rotated */
 +		if (id != DE_SMART) {
  			unsigned long flags = DRM_ROTATE_0 |
  					      DRM_ROTATE_90 |
  					      DRM_ROTATE_180 |
  					      DRM_ROTATE_270 |
  					      DRM_REFLECT_X |
  					      DRM_REFLECT_Y;
 -			drm->mode_config.rotation_property =
 -				drm_mode_create_rotation_property(drm, flags);
 +			drm_plane_create_rotation_property(&plane->base,
 +							   DRM_ROTATE_0,
 +							   flags);
  		}
  
- 		drm_plane_helper_add(&plane->base,
- 				     &malidp_de_plane_helper_funcs);
- 		plane->hwdev = malidp->dev;
- 		plane->layer = &map->layers[i];
 -		if (drm->mode_config.rotation_property)
 -			drm_object_attach_property(&plane->base.base,
 -						   drm->mode_config.rotation_property,
 -						   DRM_ROTATE_0);
 -
+ 		malidp_hw_write(malidp->dev, MALIDP_ALPHA_LUT,
+ 				plane->layer->base + MALIDP_LAYER_COMPOSE);
  	}
  
  	kfree(formats);
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: linux-next: manual merge of the mali-dp tree with the drm-misc tree

[Liviu Dudau]

On Tue, Oct 25, 2016 at 11:20:44AM +1100, Stephen Rothwell wrote:
> Hi Liviu,

Hi Stephen,

> 
> Today's linux-next merge of the mali-dp tree got a conflict in:
> 
>   drivers/gpu/drm/arm/malidp_planes.c
> 
> between commit:
> 
>   ea0e1ce20f73 ("drm/arm: Use per-plane rotation property")
> 
> from the drm-misc tree and commit:
> 
>   9ebb89762c30 ("drm: mali-dp: Refactor plane initialisation")
> 
> from the mali-dp tree.
> 
> I fixed it up (see below) and can carry the fix as necessary. This
> is now fixed as far as linux-next is concerned, but any non trivial
> conflicts should be mentioned to your upstream maintainer when your tree
> is submitted for merging.  You may also want to consider cooperating
> with the maintainer of the conflicting tree to minimise any particularly
> complex conflicts.

Sorry for delay in answering, I was on holiday.

I have revamped the mali-dp tree and rebased it on the newer
version of drm-next (which includes the drm-misc change) and pushed the
updated patch in my tree.

Best regards,
Liviu


> 
> -- 
> Cheers,
> Stephen Rothwell
> 
> diff --cc drivers/gpu/drm/arm/malidp_planes.c
> index abaca03b9d36,9020c0d8399c..000000000000
> --- a/drivers/gpu/drm/arm/malidp_planes.c
> +++ b/drivers/gpu/drm/arm/malidp_planes.c
> @@@ -254,23 -284,33 +284,30 @@@ int malidp_de_planes_init(struct drm_de
>   		if (ret < 0)
>   			goto cleanup;
>   
> + 		drm_plane_helper_add(&plane->base,
> + 				     &malidp_de_plane_helper_funcs);
> + 		plane->hwdev = malidp->dev;
> + 		plane->layer = &map->layers[i];
> + 
> + 		/* Skip the features which the SMART layer doesn't have */
> + 		if (id == DE_SMART)
> + 			continue;
> + 
>  -		if (!drm->mode_config.rotation_property) {
>  +		/* SMART layer can't be rotated */
>  +		if (id != DE_SMART) {
>   			unsigned long flags = DRM_ROTATE_0 |
>   					      DRM_ROTATE_90 |
>   					      DRM_ROTATE_180 |
>   					      DRM_ROTATE_270 |
>   					      DRM_REFLECT_X |
>   					      DRM_REFLECT_Y;
>  -			drm->mode_config.rotation_property =
>  -				drm_mode_create_rotation_property(drm, flags);
>  +			drm_plane_create_rotation_property(&plane->base,
>  +							   DRM_ROTATE_0,
>  +							   flags);
>   		}
>   
> - 		drm_plane_helper_add(&plane->base,
> - 				     &malidp_de_plane_helper_funcs);
> - 		plane->hwdev = malidp->dev;
> - 		plane->layer = &map->layers[i];
>  -		if (drm->mode_config.rotation_property)
>  -			drm_object_attach_property(&plane->base.base,
>  -						   drm->mode_config.rotation_property,
>  -						   DRM_ROTATE_0);
>  -
> + 		malidp_hw_write(malidp->dev, MALIDP_ALPHA_LUT,
> + 				plane->layer->base + MALIDP_LAYER_COMPOSE);
>   	}
>   
>   	kfree(formats);
> 

-- 
====================
| I would like to |
| fix the world,  |
| but they're not |
| giving me the   |
 \ source code!  /
  ---------------
    ¯\_(ツ)_/¯
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx

Re: linux-next: manual merge of the mali-dp tree with the drm-misc tree

[Stephen Rothwell]

Hi Liviu,

On Thu, 3 Nov 2016 17:19:58 +0000 Liviu Dudau <Liviu.Dudau@arm.com> wrote:
>
> I have revamped the mali-dp tree and rebased it on the newer
> version of drm-next (which includes the drm-misc change) and pushed the
> updated patch in my tree.

Thanks for that.  However, several of the commits in your tree now have
no Signed-off-by from you as the committer :-(

-- 
Cheers,
Stephen Rothwell
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: linux-next: manual merge of the mali-dp tree with the drm-misc tree

[Liviu Dudau]

On Fri, Nov 04, 2016 at 04:38:54PM +1100, Stephen Rothwell wrote:
> Hi Liviu,
> 
> On Thu, 3 Nov 2016 17:19:58 +0000 Liviu Dudau <Liviu.Dudau@arm.com> wrote:
> >
> > I have revamped the mali-dp tree and rebased it on the newer
> > version of drm-next (which includes the drm-misc change) and pushed the
> > updated patch in my tree.
> 
> Thanks for that.  However, several of the commits in your tree now have
> no Signed-off-by from you as the committer :-(

Brian Starkey is a co-maintainer for the Mali DP tree, so his Signed-off-by
alone should be good. Baoyou's patch is in my tree to stop him repeatedly
send me the same patch over and over again :) But yes, I will add my
Signed-off-by for that one.

Many thanks,
Liviu

> 
> -- 
> Cheers,
> Stephen Rothwell

-- 
====================
| I would like to |
| fix the world,  |
| but they're not |
| giving me the   |
 \ source code!  /
  ---------------
    ¯\_(ツ)_/¯
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Re: linux-next: manual merge of the mali-dp tree with the drm-misc tree

[Stephen Rothwell]

Hi Liviu,

On Fri, 4 Nov 2016 15:48:02 +0000 Liviu Dudau <Liviu.Dudau@arm.com> wrote:
>
> Brian Starkey is a co-maintainer for the Mali DP tree, so his Signed-off-by
> alone should be good. Baoyou's patch is in my tree to stop him repeatedly
> send me the same patch over and over again :) But yes, I will add my
> Signed-off-by for that one.

Sorry, but this is not sufficient.  Please read section 11 of
Documentation/SubmittingPatches (or
Documentation/process/submitting-patches.rst where it has been moved
recently).  If you are in the path of a patch to Linus, you must add a
Signed-off-by line, and as the person who committed those patches to
the tree, you are in the path.
-- 
Cheers,
Stephen Rothwell

Re: linux-next: manual merge of the mali-dp tree with the drm-misc tree

[Liviu Dudau]

On Sat, Nov 05, 2016 at 03:55:03AM +1100, Stephen Rothwell wrote:
> Hi Liviu,
> 
> On Fri, 4 Nov 2016 15:48:02 +0000 Liviu Dudau <Liviu.Dudau@arm.com> wrote:
> >
> > Brian Starkey is a co-maintainer for the Mali DP tree, so his Signed-off-by
> > alone should be good. Baoyou's patch is in my tree to stop him repeatedly
> > send me the same patch over and over again :) But yes, I will add my
> > Signed-off-by for that one.
> 
> Sorry, but this is not sufficient.  Please read section 11 of
> Documentation/SubmittingPatches (or
> Documentation/process/submitting-patches.rst where it has been moved
> recently).  If you are in the path of a patch to Linus, you must add a
> Signed-off-by line, and as the person who committed those patches to
> the tree, you are in the path.

Thanks for correcting me. I will add my Signed-off-bys to the relevant
patches.

Best regards,
Liviu

> -- 
> Cheers,
> Stephen Rothwell

-- 
====================
| I would like to |
| fix the world,  |
| but they're not |
| giving me the   |
 \ source code!  /
  ---------------
    ¯\_(ツ)_/¯
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/intel-gfx