On Thu, Feb 09, 2017 at 05:57:04PM +0100, Daniel Vetter wrote: > On Thu, Feb 09, 2017 at 03:41:41PM +0000, Mihail Atanassov wrote: > > Hi, > > > > I was working on a few patches adding fields to struct malidp_crtc_state and > > found myself writing memcpy multiple times in the ->atomic_duplicate_state > > hook because I wanted to avoid copying the drm_crtc_state twice > > (__drm_atomic_helper_crtc_duplicate_state copies it already). I figured this > > also applies to the other drm_*_state derivatives, so I concocted a macro > > helper to do the copy in one chunk (two if you count the __drm_atomic_helper* > > one). I'd appreciate some comments on whether anyone else might find this > > macro useful. Thanks! > > > > Mihail Atanassov (1): > > drm: Add helper macro for duplicating custom drm_*_state > > I don't have your patch here, did something go wrong with the submission? > Only the cover letter seems to have made it through ... There is another patch that should come through, I can see it in my home account. Liviu > -Daniel > > > > > include/drm/drm_atomic_helper.h | 33 +++++++++++++++++++++++++++++++++ > > 1 file changed, 33 insertions(+) > > > > -- > > Mihail Atanassov > > > > _______________________________________________ > > dri-devel mailing list > > dri-devel@xxxxxxxxxxxxxxxxxxxxx > > https://lists.freedesktop.org/mailman/listinfo/dri-devel > > -- > Daniel Vetter > Software Engineer, Intel Corporation > http://blog.ffwll.ch -- ==================== | I would like to | | fix the world, | | but they're not | | giving me the | \ source code! / --------------- ¯\_(ツ)_/¯ _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel