On 7/19/2021 3:48 PM, Rob Herring wrote: > On Mon, 19 Jul 2021 11:20:00 -0700, Alex Helms wrote: >> Renesas 8T49N241 has 4 outputs, 1 integral and 3 fractional dividers. >> The 8T49N241 accepts up to two differential or single-ended input clocks >> and a fundamental-mode crystal input. The internal PLL can lock to either >> of the input reference clocks or to the crystal to behave as a frequency >> synthesizer. >> >> Signed-off-by: Alex Helms <alexander.helms.jy@xxxxxxxxxxx> >> --- >> .../bindings/clock/renesas,8t49n241.yaml | 190 ++++++++++++++++++ >> MAINTAINERS | 6 + >> 2 files changed, 196 insertions(+) >> create mode 100644 Documentation/devicetree/bindings/clock/renesas,8t49n241.yaml >> > > > Please add Acked-by/Reviewed-by tags when posting new versions. However, > there's no need to repost patches *only* to add the tags. The upstream > maintainer will do that for acks received on the version they apply. > > If a tag was not added on purpose, please state why and what changed. > Thank you for the info. I'm new to the kernel process and appreciate your advice. I felt uncomfortable adding your Reviewed-By tag but since there were no changes to the dt portion of the patch, in retrospect I guess I should have added it. I'll keep this in mind for the future. Is there anything I need to do for this patch?