On 11/07/2024 13:32, Varadarajan Narayanan wrote: > Add interconnect-cells to clock provider so that it can be > used as icc provider. > > Add master/slave ids for Qualcomm IPQ5332 Network-On-Chip > interfaces. This will be used by the gcc-ipq5332 driver > for providing interconnect services using the icc-clk > framework. > > Signed-off-by: Varadarajan Narayanan <quic_varada@xxxxxxxxxxx> That's another patchset from Qualcomm where tags get ignored. Maybe it's the same team? You have very good internal guideline, so read it before posting. Expecting us to do the same review we already did, is a waste of community resources. <form letter> This is a friendly reminder during the review process. 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, under or above your Signed-off-by tag. 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. https://elixir.bootlin.com/linux/v6.5-rc3/source/Documentation/process/submitting-patches.rst#L577 If a tag was not added on purpose, please state why and what changed. </form letter> Best regards, Krzysztof