On 17/03/2025 20:21, Kaustabh Chakraborty wrote: > Document the compatible string for Exynos7870's UART driver. The > devicetree property samsung,uart-fifosize must be mandatory, as the > driver enquires about the FIFO sizes. This feature makes it compatible > with Exynos8895's UART. > > Signed-off-by: Kaustabh Chakraborty <kauschluss@xxxxxxxxxxx> > --- So you just ignored the tags and ask us to do the work twice? So all reviews are wasted and we need to duplicate the effort? My time on upstream is very limited, thus I find it very disrespectful. <form letter> It looks like you received a tag and forgot to add it. If you do not know the process, here is a short explanation: Please add Acked-by/Reviewed-by/Tested-by tags when posting new versions of patchset, under or above your Signed-off-by tag, unless patch changed significantly (e.g. new properties added to the DT bindings). Tag is "received", when provided in a message replied to you on the mailing list. Tools like b4 can help here. However, there's no need to repost patches *only* to add the tags. The upstream maintainer will do that for tags received on the version they apply. Please read: https://elixir.bootlin.com/linux/v6.12-rc3/source/Documentation/process/submitting-patches.rst#L577 If a tag was not added on purpose, please state why and what changed. </form letter>