On Mon 04 Oct 00:22 PDT 2021, Manivannan Sadhasivam wrote: > On Mon, Oct 04, 2021 at 12:35:31PM +0530, Viresh Kumar wrote: > > On 04-10-21, 10:13, Manivannan Sadhasivam wrote: > > > Convert Qualcomm cpufreq devicetree binding to YAML. > > > > > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx> > > > Signed-off-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@xxxxxxxxxxxxxx> > > > Reviewed-by: Rob Herring <robh@xxxxxxxxxx> > > > > I am not sure if Rob ever gave this. > > > > I'm not fooling you :) > https://patchwork.kernel.org/project/linux-pm/patch/20210701105730.322718-5-angelogioacchino.delregno@xxxxxxxxxxxxxx/#24312445 > > > > Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@xxxxxxxxxx> > > > > Why double signed off ? > > > > Ah, it came while I applied the patch from Angelo's series. If you want > I can send a new version removing one or you can do that while applying. > If you wrote the patch, then Angelo handled it, then you handled it again the double S-o-b captures that nicely. Looking it from the other angle, if you remove the first S-o-b, then you forgot to signed it off when you authored the original patch and if you skip the last S-o-b then you didn't adequately sign off the final result. Regards, Bjorn > Let me know. > > Thanks, > Mani > > > -- > > viresh