On 03/08, Maíra Canal wrote: > Hi Arthur, > > Would it be possible for you to coordinate with Louis and create a > single series with all the modification? > > I don't see a reason to submit fixes to a series that it is still > on review.
I agree. Moreover, the fix for drm_fixp2int_round() should go in a single patch detached from these multiple work branches, since it is addressing an issue already upstream. Thanks, Melissa > > Best Regards, > - Maíra > > On 3/6/24 17:03, Arthur Grillo wrote: > > These are some patches that add some fixes/features to the series by > > Louis Chauvet[1], it was based on top of the patches from v4. > > > > Patches #2 and #3 should be amended to "[PATCH v4 11/14] drm/vkms: Add > > YUV support". To make patch #3 work, we need patch #1. So, please, add > > it before the patch that #2 and #3 amend to. > > > > Patches #4 to #6 should be amended to "[PATCH v4 14/14] drm/vkms: Create > > KUnit tests for YUV conversions". While doing the additions, I found > > some compilation issues, so I fixed them (patch #4). That's when I > > thought that it would be good to add some documentation on how to run > > them (patch #7), this patch should be added to the series as new patch. > > > > [1]: https://lore.kernel.org/r/20240304-yuv-v4-0-76beac8e9...@bootlin.com > > > > To: Rodrigo Siqueira <rodrigosiqueiram...@gmail.com> > > To: Melissa Wen <melissa....@gmail.com> > > To: Maíra Canal <mairaca...@riseup.net> > > To: Haneen Mohammed <hamohammed...@gmail.com> > > To: Daniel Vetter <dan...@ffwll.ch> > > To: Maarten Lankhorst <maarten.lankho...@linux.intel.com> > > To: Maxime Ripard <mrip...@kernel.org> > > To: Thomas Zimmermann <tzimmerm...@suse.de> > > To: David Airlie <airl...@gmail.com> > > To: arthurgri...@riseup.net > > To: Jonathan Corbet <cor...@lwn.net> > > To: pekka.paala...@haloniitty.fi > > To: Louis Chauvet <louis.chau...@bootlin.com> > > Cc: dri-devel@lists.freedesktop.org > > Cc: linux-ker...@vger.kernel.org > > Cc: jeremie.dautheri...@bootlin.com > > Cc: miquel.ray...@bootlin.com > > Cc: thomas.petazz...@bootlin.com > > Cc: seanp...@google.com > > Cc: marc...@google.com > > Cc: nicolejade...@google.com > > > > Signed-off-by: Arthur Grillo <arthurgri...@riseup.net> > > --- > > Arthur Grillo (7): > > drm: Fix drm_fixp2int_round() making it add 0.5 > > drm/vkms: Add comments > > drm/vkmm: Use drm_fixed api > > drm/vkms: Fix compilation issues > > drm/vkms: Add comments to format tests > > drm/vkms: Change the gray RGB representation > > drm/vkms: Add how to run the Kunit tests > > > > Documentation/gpu/vkms.rst | 11 +++ > > drivers/gpu/drm/vkms/tests/vkms_format_test.c | 81 +++++++++++++++++++-- > > drivers/gpu/drm/vkms/vkms_drv.h | 4 + > > drivers/gpu/drm/vkms/vkms_formats.c | 101 > > +++++++++++++++++++------- > > include/drm/drm_fixed.h | 2 +- > > 5 files changed, 165 insertions(+), 34 deletions(-) > > --- > > base-commit: 9658aba38ae9f3f3068506c9c8e93e85b500fcb4 > > change-id: 20240306-louis-vkms-conv-61362ff12ab8 > > > > Best regards,
