Re: [PATCH 02/30] include: drm: drm_atomic: Artificially use 'crtc' to avoid 'not used' warning
On Tue, 17 Nov 2020, Daniel Vetter wrote: > On Tue, Nov 17, 2020 at 10:34:04AM +, Lee Jones wrote: > > Daniel, > > > > For some reason, you're not appearing in the recipents list when I > > reply to all. You're not in the Mail-Followup-To header. Any idea > > why this might be? > > No idea either, could be my mutt not setting the reply headers like it > should. I'm generally confused with how email works ... Yes, email is a mystery! > > Anyway, please see below: > > > > On Tue, 17 Nov 2020, Daniel Vetter wrote: > > > On Fri, Nov 13, 2020 at 10:01:57PM +, Lee Jones wrote: > > > > On Fri, 13 Nov 2020, 21:31 Daniel Vetter, wrote: > > > > > > > > > On Fri, Nov 13, 2020 at 9:53 PM Lee Jones > > > > > wrote: > > > > > > > > > > > > > > > > > > > > > > > > On Fri, 13 Nov 2020, 20:50 Daniel Vetter, wrote: > > > > > >> > > > > > >> On Thu, Nov 12, 2020 at 09:25:16PM +0100, Sam Ravnborg wrote: > > > > > >> > Hi Lee, > > > > > >> > > > > > > >> > On Thu, Nov 12, 2020 at 07:00:11PM +, Lee Jones wrote: > > > > > >> > > The precedent has already been set by other macros in the same > > > > > >> > > file. > > > > > >> > > > > > > > >> > > Fixes the following W=1 kernel build warning(s): > > > > > >> > > > > > > > >> > > drivers/gpu/drm/vkms/vkms_drv.c:55:19: warning: variable > > > > > >> > > ‘crtc’ > > > > > set but not used [-Wunused-but-set-variable] > > > > > >> > > 55 | struct drm_crtc *crtc; > > > > > >> > > | ^~~~ > > > > > >> > > > > > > > >> > > Cc: Maarten Lankhorst > > > > > >> > > Cc: Maxime Ripard > > > > > >> > > Cc: Thomas Zimmermann > > > > > >> > > Cc: David Airlie > > > > > >> > > Cc: Daniel Vetter > > > > > >> > > Cc: Rob Clark > > > > > >> > > Cc: dri-devel@lists.freedesktop.org > > > > > >> > > Signed-off-by: Lee Jones > > > > > >> > > > > > > >> > Also applied to drm-misc-next. > > > > > >> > This was the last patch from this batch I will process. > > > > > >> > The others are left for the maintainers to pick up. > > > > > >> > > > > > >> btw for patches that maintainers don't pick up, the usual process > > > > > >> is > > > > > that > > > > > >> we give them 2 weeks, then just mass apply. Now you're producing a > > > > > >> lot > > > > > of > > > > > >> patches, too much for me to keep track, so please just ping me > > > > > >> with a > > > > > >> resend for those that expired and I'll go through and pick them > > > > > >> all up. > > > > > > > > > > > > > > > > > > That's great Daniel. Thanks for your support. > > > > > > > > > > > > I can do one better than that. > > > > > > > > > > > > Would a pull-request suit you? > > > > > > > > > > We have a few trees going on, and your patches are landing through all > > > > > kinds of them. So this might be more coordination pain. If you can > > > > > exclude patches for the separately and usually fairly well maintained > > > > > drivers out of the pull it should work (drm/amd, drm/radeon, drm/i915, > > > > > drm/nouveau, drm/msm and drm/omapdrm probably the usual ones). > > > > > > > > > > Or you just send the next pile and we'll see. > > > > > > > > > > Also I guess I can't really interest you in commit rights so this > > > > > patch bombs get off my back again? :-) > > > > > > > > > > > > > What does that mean? Merge my own patches? > > > > > > > > Not sure how that works with your group maintenance setup. > > > > > > > > Is it just a `git push`? > > > > > > It's a bunch of scripting and setup, might not be worth it for just one > > > of. Plus we still take pull requests from submaintainers so it's all just > > > if you feel like it. Some docs if you're curious: > > > > > > https://drm.pages.freedesktop.org/maintainer-tools/getting-started.html > > > > As *fun* as that looks, I think I'll give it a miss, thanks. :) > > > > So for the final merge of the non-Radeon set [0] to date (Alex just > > pulled in all of the outstanding Radeon patches), we have a few > > options: > > > > 1. Take them directly from the most recent set [0] > > 2. I can rebase and resubmit again in a few days > > 3. I can submit a PR > > > > Whatever works for you. > > > > [0] "[PATCH v2 00/42] Rid W=1 warnings from GPU (non-Radeon)" > > I planned to go through that today and apply it all. Wanted to ping a few > maintainers first whether they pick anything up, but aside from Alex > they're non-committal so I think I just vacuum them all up so it's done. Okay, perfect. Thanks a bunch. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH 02/30] include: drm: drm_atomic: Artificially use 'crtc' to avoid 'not used' warning
On Tue, Nov 17, 2020 at 10:34:04AM +, Lee Jones wrote: > Daniel, > > For some reason, you're not appearing in the recipents list when I > reply to all. You're not in the Mail-Followup-To header. Any idea > why this might be? No idea either, could be my mutt not setting the reply headers like it should. I'm generally confused with how email works ... > > Anyway, please see below: > > On Tue, 17 Nov 2020, Daniel Vetter wrote: > > On Fri, Nov 13, 2020 at 10:01:57PM +, Lee Jones wrote: > > > On Fri, 13 Nov 2020, 21:31 Daniel Vetter, wrote: > > > > > > > On Fri, Nov 13, 2020 at 9:53 PM Lee Jones wrote: > > > > > > > > > > > > > > > > > > > > On Fri, 13 Nov 2020, 20:50 Daniel Vetter, wrote: > > > > >> > > > > >> On Thu, Nov 12, 2020 at 09:25:16PM +0100, Sam Ravnborg wrote: > > > > >> > Hi Lee, > > > > >> > > > > > >> > On Thu, Nov 12, 2020 at 07:00:11PM +, Lee Jones wrote: > > > > >> > > The precedent has already been set by other macros in the same > > > > >> > > file. > > > > >> > > > > > > >> > > Fixes the following W=1 kernel build warning(s): > > > > >> > > > > > > >> > > drivers/gpu/drm/vkms/vkms_drv.c:55:19: warning: variable ‘crtc’ > > > > set but not used [-Wunused-but-set-variable] > > > > >> > > 55 | struct drm_crtc *crtc; > > > > >> > > | ^~~~ > > > > >> > > > > > > >> > > Cc: Maarten Lankhorst > > > > >> > > Cc: Maxime Ripard > > > > >> > > Cc: Thomas Zimmermann > > > > >> > > Cc: David Airlie > > > > >> > > Cc: Daniel Vetter > > > > >> > > Cc: Rob Clark > > > > >> > > Cc: dri-devel@lists.freedesktop.org > > > > >> > > Signed-off-by: Lee Jones > > > > >> > > > > > >> > Also applied to drm-misc-next. > > > > >> > This was the last patch from this batch I will process. > > > > >> > The others are left for the maintainers to pick up. > > > > >> > > > > >> btw for patches that maintainers don't pick up, the usual process is > > > > that > > > > >> we give them 2 weeks, then just mass apply. Now you're producing a > > > > >> lot > > > > of > > > > >> patches, too much for me to keep track, so please just ping me with a > > > > >> resend for those that expired and I'll go through and pick them all > > > > >> up. > > > > > > > > > > > > > > > That's great Daniel. Thanks for your support. > > > > > > > > > > I can do one better than that. > > > > > > > > > > Would a pull-request suit you? > > > > > > > > We have a few trees going on, and your patches are landing through all > > > > kinds of them. So this might be more coordination pain. If you can > > > > exclude patches for the separately and usually fairly well maintained > > > > drivers out of the pull it should work (drm/amd, drm/radeon, drm/i915, > > > > drm/nouveau, drm/msm and drm/omapdrm probably the usual ones). > > > > > > > > Or you just send the next pile and we'll see. > > > > > > > > Also I guess I can't really interest you in commit rights so this > > > > patch bombs get off my back again? :-) > > > > > > > > > > What does that mean? Merge my own patches? > > > > > > Not sure how that works with your group maintenance setup. > > > > > > Is it just a `git push`? > > > > It's a bunch of scripting and setup, might not be worth it for just one > > of. Plus we still take pull requests from submaintainers so it's all just > > if you feel like it. Some docs if you're curious: > > > > https://drm.pages.freedesktop.org/maintainer-tools/getting-started.html > > As *fun* as that looks, I think I'll give it a miss, thanks. :) > > So for the final merge of the non-Radeon set [0] to date (Alex just > pulled in all of the outstanding Radeon patches), we have a few > options: > > 1. Take them directly from the most recent set [0] > 2. I can rebase and resubmit again in a few days > 3. I can submit a PR > > Whatever works for you. > > [0] "[PATCH v2 00/42] Rid W=1 warnings from GPU (non-Radeon)" I planned to go through that today and apply it all. Wanted to ping a few maintainers first whether they pick anything up, but aside from Alex they're non-committal so I think I just vacuum them all up so it's done. -Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH 02/30] include: drm: drm_atomic: Artificially use 'crtc' to avoid 'not used' warning
Daniel, For some reason, you're not appearing in the recipents list when I reply to all. You're not in the Mail-Followup-To header. Any idea why this might be? Anyway, please see below: On Tue, 17 Nov 2020, Daniel Vetter wrote: > On Fri, Nov 13, 2020 at 10:01:57PM +, Lee Jones wrote: > > On Fri, 13 Nov 2020, 21:31 Daniel Vetter, wrote: > > > > > On Fri, Nov 13, 2020 at 9:53 PM Lee Jones wrote: > > > > > > > > > > > > > > > > On Fri, 13 Nov 2020, 20:50 Daniel Vetter, wrote: > > > >> > > > >> On Thu, Nov 12, 2020 at 09:25:16PM +0100, Sam Ravnborg wrote: > > > >> > Hi Lee, > > > >> > > > > >> > On Thu, Nov 12, 2020 at 07:00:11PM +, Lee Jones wrote: > > > >> > > The precedent has already been set by other macros in the same > > > >> > > file. > > > >> > > > > > >> > > Fixes the following W=1 kernel build warning(s): > > > >> > > > > > >> > > drivers/gpu/drm/vkms/vkms_drv.c:55:19: warning: variable ‘crtc’ > > > set but not used [-Wunused-but-set-variable] > > > >> > > 55 | struct drm_crtc *crtc; > > > >> > > | ^~~~ > > > >> > > > > > >> > > Cc: Maarten Lankhorst > > > >> > > Cc: Maxime Ripard > > > >> > > Cc: Thomas Zimmermann > > > >> > > Cc: David Airlie > > > >> > > Cc: Daniel Vetter > > > >> > > Cc: Rob Clark > > > >> > > Cc: dri-devel@lists.freedesktop.org > > > >> > > Signed-off-by: Lee Jones > > > >> > > > > >> > Also applied to drm-misc-next. > > > >> > This was the last patch from this batch I will process. > > > >> > The others are left for the maintainers to pick up. > > > >> > > > >> btw for patches that maintainers don't pick up, the usual process is > > > that > > > >> we give them 2 weeks, then just mass apply. Now you're producing a lot > > > of > > > >> patches, too much for me to keep track, so please just ping me with a > > > >> resend for those that expired and I'll go through and pick them all up. > > > > > > > > > > > > That's great Daniel. Thanks for your support. > > > > > > > > I can do one better than that. > > > > > > > > Would a pull-request suit you? > > > > > > We have a few trees going on, and your patches are landing through all > > > kinds of them. So this might be more coordination pain. If you can > > > exclude patches for the separately and usually fairly well maintained > > > drivers out of the pull it should work (drm/amd, drm/radeon, drm/i915, > > > drm/nouveau, drm/msm and drm/omapdrm probably the usual ones). > > > > > > Or you just send the next pile and we'll see. > > > > > > Also I guess I can't really interest you in commit rights so this > > > patch bombs get off my back again? :-) > > > > > > > What does that mean? Merge my own patches? > > > > Not sure how that works with your group maintenance setup. > > > > Is it just a `git push`? > > It's a bunch of scripting and setup, might not be worth it for just one > of. Plus we still take pull requests from submaintainers so it's all just > if you feel like it. Some docs if you're curious: > > https://drm.pages.freedesktop.org/maintainer-tools/getting-started.html As *fun* as that looks, I think I'll give it a miss, thanks. :) So for the final merge of the non-Radeon set [0] to date (Alex just pulled in all of the outstanding Radeon patches), we have a few options: 1. Take them directly from the most recent set [0] 2. I can rebase and resubmit again in a few days 3. I can submit a PR Whatever works for you. [0] "[PATCH v2 00/42] Rid W=1 warnings from GPU (non-Radeon)" -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH 02/30] include: drm: drm_atomic: Artificially use 'crtc' to avoid 'not used' warning
On Fri, Nov 13, 2020 at 10:01:57PM +, Lee Jones wrote: > On Fri, 13 Nov 2020, 21:31 Daniel Vetter, wrote: > > > On Fri, Nov 13, 2020 at 9:53 PM Lee Jones wrote: > > > > > > > > > > > > On Fri, 13 Nov 2020, 20:50 Daniel Vetter, wrote: > > >> > > >> On Thu, Nov 12, 2020 at 09:25:16PM +0100, Sam Ravnborg wrote: > > >> > Hi Lee, > > >> > > > >> > On Thu, Nov 12, 2020 at 07:00:11PM +, Lee Jones wrote: > > >> > > The precedent has already been set by other macros in the same file. > > >> > > > > >> > > Fixes the following W=1 kernel build warning(s): > > >> > > > > >> > > drivers/gpu/drm/vkms/vkms_drv.c:55:19: warning: variable ‘crtc’ > > set but not used [-Wunused-but-set-variable] > > >> > > 55 | struct drm_crtc *crtc; > > >> > > | ^~~~ > > >> > > > > >> > > Cc: Maarten Lankhorst > > >> > > Cc: Maxime Ripard > > >> > > Cc: Thomas Zimmermann > > >> > > Cc: David Airlie > > >> > > Cc: Daniel Vetter > > >> > > Cc: Rob Clark > > >> > > Cc: dri-devel@lists.freedesktop.org > > >> > > Signed-off-by: Lee Jones > > >> > > > >> > Also applied to drm-misc-next. > > >> > This was the last patch from this batch I will process. > > >> > The others are left for the maintainers to pick up. > > >> > > >> btw for patches that maintainers don't pick up, the usual process is > > that > > >> we give them 2 weeks, then just mass apply. Now you're producing a lot > > of > > >> patches, too much for me to keep track, so please just ping me with a > > >> resend for those that expired and I'll go through and pick them all up. > > > > > > > > > That's great Daniel. Thanks for your support. > > > > > > I can do one better than that. > > > > > > Would a pull-request suit you? > > > > We have a few trees going on, and your patches are landing through all > > kinds of them. So this might be more coordination pain. If you can > > exclude patches for the separately and usually fairly well maintained > > drivers out of the pull it should work (drm/amd, drm/radeon, drm/i915, > > drm/nouveau, drm/msm and drm/omapdrm probably the usual ones). > > > > Or you just send the next pile and we'll see. > > > > Also I guess I can't really interest you in commit rights so this > > patch bombs get off my back again? :-) > > > > What does that mean? Merge my own patches? > > Not sure how that works with your group maintenance setup. > > Is it just a `git push`? It's a bunch of scripting and setup, might not be worth it for just one of. Plus we still take pull requests from submaintainers so it's all just if you feel like it. Some docs if you're curious: https://drm.pages.freedesktop.org/maintainer-tools/getting-started.html Cheers, Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH 02/30] include: drm: drm_atomic: Artificially use 'crtc' to avoid 'not used' warning
On Fri, 13 Nov 2020, Daniel Vetter wrote: > On Thu, Nov 12, 2020 at 09:25:16PM +0100, Sam Ravnborg wrote: > > Hi Lee, > > > > On Thu, Nov 12, 2020 at 07:00:11PM +, Lee Jones wrote: > > > The precedent has already been set by other macros in the same file. > > > > > > Fixes the following W=1 kernel build warning(s): > > > > > > drivers/gpu/drm/vkms/vkms_drv.c:55:19: warning: variable ‘crtc’ set but > > > not used [-Wunused-but-set-variable] > > > 55 | struct drm_crtc *crtc; > > > | ^~~~ > > > > > > Cc: Maarten Lankhorst > > > Cc: Maxime Ripard > > > Cc: Thomas Zimmermann > > > Cc: David Airlie > > > Cc: Daniel Vetter > > > Cc: Rob Clark > > > Cc: dri-devel@lists.freedesktop.org > > > Signed-off-by: Lee Jones > > > > Also applied to drm-misc-next. > > This was the last patch from this batch I will process. > > The others are left for the maintainers to pick up. > > btw for patches that maintainers don't pick up, the usual process is that > we give them 2 weeks, then just mass apply. Now you're producing a lot of > patches, too much for me to keep track, so please just ping me with a > resend for those that expired and I'll go through and pick them all up. Okay, I have split the remaining patches into 2 new sets. One for Radeon, the other for everything else. Hopefully that makes things easier/more obvious. -- Lee Jones [李琼斯] Senior Technical Lead - Developer Services Linaro.org │ Open source software for Arm SoCs Follow Linaro: Facebook | Twitter | Blog ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH 02/30] include: drm: drm_atomic: Artificially use 'crtc' to avoid 'not used' warning
On Fri, 13 Nov 2020, 21:31 Daniel Vetter, wrote: > On Fri, Nov 13, 2020 at 9:53 PM Lee Jones wrote: > > > > > > > > On Fri, 13 Nov 2020, 20:50 Daniel Vetter, wrote: > >> > >> On Thu, Nov 12, 2020 at 09:25:16PM +0100, Sam Ravnborg wrote: > >> > Hi Lee, > >> > > >> > On Thu, Nov 12, 2020 at 07:00:11PM +, Lee Jones wrote: > >> > > The precedent has already been set by other macros in the same file. > >> > > > >> > > Fixes the following W=1 kernel build warning(s): > >> > > > >> > > drivers/gpu/drm/vkms/vkms_drv.c:55:19: warning: variable ‘crtc’ > set but not used [-Wunused-but-set-variable] > >> > > 55 | struct drm_crtc *crtc; > >> > > | ^~~~ > >> > > > >> > > Cc: Maarten Lankhorst > >> > > Cc: Maxime Ripard > >> > > Cc: Thomas Zimmermann > >> > > Cc: David Airlie > >> > > Cc: Daniel Vetter > >> > > Cc: Rob Clark > >> > > Cc: dri-devel@lists.freedesktop.org > >> > > Signed-off-by: Lee Jones > >> > > >> > Also applied to drm-misc-next. > >> > This was the last patch from this batch I will process. > >> > The others are left for the maintainers to pick up. > >> > >> btw for patches that maintainers don't pick up, the usual process is > that > >> we give them 2 weeks, then just mass apply. Now you're producing a lot > of > >> patches, too much for me to keep track, so please just ping me with a > >> resend for those that expired and I'll go through and pick them all up. > > > > > > That's great Daniel. Thanks for your support. > > > > I can do one better than that. > > > > Would a pull-request suit you? > > We have a few trees going on, and your patches are landing through all > kinds of them. So this might be more coordination pain. If you can > exclude patches for the separately and usually fairly well maintained > drivers out of the pull it should work (drm/amd, drm/radeon, drm/i915, > drm/nouveau, drm/msm and drm/omapdrm probably the usual ones). > > Or you just send the next pile and we'll see. > > Also I guess I can't really interest you in commit rights so this > patch bombs get off my back again? :-) > What does that mean? Merge my own patches? Not sure how that works with your group maintenance setup. Is it just a `git push`? > ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH 02/30] include: drm: drm_atomic: Artificially use 'crtc' to avoid 'not used' warning
On Fri, Nov 13, 2020 at 9:53 PM Lee Jones wrote: > > > > On Fri, 13 Nov 2020, 20:50 Daniel Vetter, wrote: >> >> On Thu, Nov 12, 2020 at 09:25:16PM +0100, Sam Ravnborg wrote: >> > Hi Lee, >> > >> > On Thu, Nov 12, 2020 at 07:00:11PM +, Lee Jones wrote: >> > > The precedent has already been set by other macros in the same file. >> > > >> > > Fixes the following W=1 kernel build warning(s): >> > > >> > > drivers/gpu/drm/vkms/vkms_drv.c:55:19: warning: variable ‘crtc’ set but >> > > not used [-Wunused-but-set-variable] >> > > 55 | struct drm_crtc *crtc; >> > > | ^~~~ >> > > >> > > Cc: Maarten Lankhorst >> > > Cc: Maxime Ripard >> > > Cc: Thomas Zimmermann >> > > Cc: David Airlie >> > > Cc: Daniel Vetter >> > > Cc: Rob Clark >> > > Cc: dri-devel@lists.freedesktop.org >> > > Signed-off-by: Lee Jones >> > >> > Also applied to drm-misc-next. >> > This was the last patch from this batch I will process. >> > The others are left for the maintainers to pick up. >> >> btw for patches that maintainers don't pick up, the usual process is that >> we give them 2 weeks, then just mass apply. Now you're producing a lot of >> patches, too much for me to keep track, so please just ping me with a >> resend for those that expired and I'll go through and pick them all up. > > > That's great Daniel. Thanks for your support. > > I can do one better than that. > > Would a pull-request suit you? We have a few trees going on, and your patches are landing through all kinds of them. So this might be more coordination pain. If you can exclude patches for the separately and usually fairly well maintained drivers out of the pull it should work (drm/amd, drm/radeon, drm/i915, drm/nouveau, drm/msm and drm/omapdrm probably the usual ones). Or you just send the next pile and we'll see. Also I guess I can't really interest you in commit rights so this patch bombs get off my back again? :-) Cheers, Daniel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH 02/30] include: drm: drm_atomic: Artificially use 'crtc' to avoid 'not used' warning
On Fri, 13 Nov 2020, 20:50 Daniel Vetter, wrote: > On Thu, Nov 12, 2020 at 09:25:16PM +0100, Sam Ravnborg wrote: > > Hi Lee, > > > > On Thu, Nov 12, 2020 at 07:00:11PM +, Lee Jones wrote: > > > The precedent has already been set by other macros in the same file. > > > > > > Fixes the following W=1 kernel build warning(s): > > > > > > drivers/gpu/drm/vkms/vkms_drv.c:55:19: warning: variable ‘crtc’ set > but not used [-Wunused-but-set-variable] > > > 55 | struct drm_crtc *crtc; > > > | ^~~~ > > > > > > Cc: Maarten Lankhorst > > > Cc: Maxime Ripard > > > Cc: Thomas Zimmermann > > > Cc: David Airlie > > > Cc: Daniel Vetter > > > Cc: Rob Clark > > > Cc: dri-devel@lists.freedesktop.org > > > Signed-off-by: Lee Jones > > > > Also applied to drm-misc-next. > > This was the last patch from this batch I will process. > > The others are left for the maintainers to pick up. > > btw for patches that maintainers don't pick up, the usual process is that > we give them 2 weeks, then just mass apply. Now you're producing a lot of > patches, too much for me to keep track, so please just ping me with a > resend for those that expired and I'll go through and pick them all up. > That's great Daniel. Thanks for your support. I can do one better than that. Would a pull-request suit you? > ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH 02/30] include: drm: drm_atomic: Artificially use 'crtc' to avoid 'not used' warning
On Thu, Nov 12, 2020 at 09:25:16PM +0100, Sam Ravnborg wrote: > Hi Lee, > > On Thu, Nov 12, 2020 at 07:00:11PM +, Lee Jones wrote: > > The precedent has already been set by other macros in the same file. > > > > Fixes the following W=1 kernel build warning(s): > > > > drivers/gpu/drm/vkms/vkms_drv.c:55:19: warning: variable ‘crtc’ set but > > not used [-Wunused-but-set-variable] > > 55 | struct drm_crtc *crtc; > > | ^~~~ > > > > Cc: Maarten Lankhorst > > Cc: Maxime Ripard > > Cc: Thomas Zimmermann > > Cc: David Airlie > > Cc: Daniel Vetter > > Cc: Rob Clark > > Cc: dri-devel@lists.freedesktop.org > > Signed-off-by: Lee Jones > > Also applied to drm-misc-next. > This was the last patch from this batch I will process. > The others are left for the maintainers to pick up. btw for patches that maintainers don't pick up, the usual process is that we give them 2 weeks, then just mass apply. Now you're producing a lot of patches, too much for me to keep track, so please just ping me with a resend for those that expired and I'll go through and pick them all up. Cheers, Daniel > > Sam > ___ > dri-devel mailing list > dri-devel@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/dri-devel -- Daniel Vetter Software Engineer, Intel Corporation http://blog.ffwll.ch ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
Re: [PATCH 02/30] include: drm: drm_atomic: Artificially use 'crtc' to avoid 'not used' warning
Hi Lee, On Thu, Nov 12, 2020 at 07:00:11PM +, Lee Jones wrote: > The precedent has already been set by other macros in the same file. > > Fixes the following W=1 kernel build warning(s): > > drivers/gpu/drm/vkms/vkms_drv.c:55:19: warning: variable ‘crtc’ set but not > used [-Wunused-but-set-variable] > 55 | struct drm_crtc *crtc; > | ^~~~ > > Cc: Maarten Lankhorst > Cc: Maxime Ripard > Cc: Thomas Zimmermann > Cc: David Airlie > Cc: Daniel Vetter > Cc: Rob Clark > Cc: dri-devel@lists.freedesktop.org > Signed-off-by: Lee Jones Also applied to drm-misc-next. This was the last patch from this batch I will process. The others are left for the maintainers to pick up. Sam ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
[PATCH 02/30] include: drm: drm_atomic: Artificially use 'crtc' to avoid 'not used' warning
The precedent has already been set by other macros in the same file. Fixes the following W=1 kernel build warning(s): drivers/gpu/drm/vkms/vkms_drv.c:55:19: warning: variable ‘crtc’ set but not used [-Wunused-but-set-variable] 55 | struct drm_crtc *crtc; | ^~~~ Cc: Maarten Lankhorst Cc: Maxime Ripard Cc: Thomas Zimmermann Cc: David Airlie Cc: Daniel Vetter Cc: Rob Clark Cc: dri-devel@lists.freedesktop.org Signed-off-by: Lee Jones --- include/drm/drm_atomic.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/drm/drm_atomic.h b/include/drm/drm_atomic.h index 413fd0ca56a8f..dc5e0fbecc04e 100644 --- a/include/drm/drm_atomic.h +++ b/include/drm/drm_atomic.h @@ -802,6 +802,7 @@ void drm_state_dump(struct drm_device *dev, struct drm_printer *p); (__i)++) \ for_each_if ((__state)->crtcs[__i].ptr && \ ((crtc) = (__state)->crtcs[__i].ptr, \ +(void)(crtc) /* Only to avoid unused-but-set-variable warning */, \ (old_crtc_state) = (__state)->crtcs[__i].old_state, 1)) /** -- 2.25.1 ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel