[Intel-gfx] linux-next: manual merge of the mali-dp tree with the drm-misc tree
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.. --- 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(_state->mode, - , ); - + mc = to_malidp_crtc_state(crtc_state); + - ret = drm_atomic_helper_check_plane_state(state, crtc_state, , + ret = drm_atomic_helper_check_plane_state(state, crtc_state, 0, INT_MAX, true, true); if (ret) return ret; pgpH3ThJbdc86.pgp Description: OpenPGP digital signature ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] linux-next: manual merge of the mali-dp tree with the drm-misc tree
On Sat, Nov 05, 2016 at 03:55:03AM +1100, Stephen Rothwell wrote: > Hi Liviu, > > On Fri, 4 Nov 2016 15:48:02 + Liviu Dudauwrote: > > > > 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
Re: [Intel-gfx] linux-next: manual merge of the mali-dp tree with the drm-misc tree
Hi Liviu, On Fri, 4 Nov 2016 15:48:02 + Liviu Dudauwrote: > > 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 ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] linux-next: manual merge of the mali-dp tree with the drm-misc tree
On Fri, Nov 04, 2016 at 04:38:54PM +1100, Stephen Rothwell wrote: > Hi Liviu, > > On Thu, 3 Nov 2016 17:19:58 + Liviu Dudauwrote: > > > > 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! / --- ¯\_(ツ)_/¯ ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] linux-next: manual merge of the mali-dp tree with the drm-misc tree
Hi Liviu, On Thu, 3 Nov 2016 17:19:58 + Liviu Dudauwrote: > > 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 ___ Intel-gfx mailing list Intel-gfx@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/intel-gfx
Re: [Intel-gfx] linux-next: manual merge of the mali-dp tree with the drm-misc tree
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.. > --- 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(>base, > + _de_plane_helper_funcs); > + plane->hwdev = malidp->dev; > + plane->layer = >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(>base, > + DRM_ROTATE_0, > + flags); > } > > - drm_plane_helper_add(>base, > - _de_plane_helper_funcs); > - plane->hwdev = malidp->dev; > - plane->layer = >layers[i]; > -if (drm->mode_config.rotation_property) > -drm_object_attach_property(>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
[Intel-gfx] linux-next: manual merge of the mali-dp tree with the drm-misc tree
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.. --- 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(>base, +_de_plane_helper_funcs); + plane->hwdev = malidp->dev; + plane->layer = >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(>base, + DRM_ROTATE_0, + flags); } - drm_plane_helper_add(>base, -_de_plane_helper_funcs); - plane->hwdev = malidp->dev; - plane->layer = >layers[i]; - if (drm->mode_config.rotation_property) - drm_object_attach_property(>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