On Mon, Oct 04, 2021 at 10:01:43AM -0700, Bjorn Andersson wrote: > 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. > Makes sense! I'll leave it as it is. Thanks, Mani > Regards, > Bjorn > > > Let me know. > > > > Thanks, > > Mani > > > > > -- > > > viresh