On 23/02/2025 14:39, Lorenzo Bianconi wrote: >> On Sat, Feb 22, 2025 at 11:43:44AM +0100, Lorenzo Bianconi wrote: >>> Introduce the mediatek,pbus-csr property for the pbus-csr syscon node >>> available on EN7581 SoC. The airoha pbus-csr block provides a configuration >>> interface for the PBUS controller used to detect if a given address is >>> accessible on PCIe controller. >>> >>> Signed-off-by: Lorenzo Bianconi <lorenzo@xxxxxxxxxx> >>> --- >>> .../devicetree/bindings/pci/mediatek-pcie-gen3.yaml | 17 +++++++++++++++++ >>> 1 file changed, 17 insertions(+) >>> >> >> You got review tag, so if you decided to skip it, this should be >> mentioned why. >> >> >> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > > Since I modified the patch with respect to the previous version, I was thinking I know > you want to review it again before applying the Reviewed-by tag. Added it now. That was not my comment. I commented that you must explicitly say that you dropped someone's tag. And docs clearly ask for that: "Usually removal of someone's Tested-by or Reviewed-by tags should be mentioned in the patch changelog (after the '---' separator)." Best regards, Krzysztof
