Re: [PATCH 2/5] omap: control: add functions for DSP boot

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hello Felipe,

On Thu, 11 Nov 2010, Felipe Contreras wrote:

> On Thu, Nov 11, 2010 at 2:00 AM, Paul Walmsley <paul@xxxxxxxxx> wrote:
> >
> > I see that you modified my original patch, but it seems that you also
> > dropped my Signed-off-by:.
> 
> Yes, because you didn't s-o-b my changes. I got that complaint from Catalin.

I have reviewed my paul@xxxxxxxxx inbox. I didn't see any request from you 
to review the changes.  But perhaps it was inadvertently dropped at some 
point in the mail path between you and I.

> > This is a tricky situation, to be sure.
> > Probably the best thing to do in these cases if you don't wish to ask me
> > to review the modified patch,  is to keep my original Signed-off-by: --
> > since I wrote the patch, originally -- and then, immediately before your
> > Signed-off-by:, to add a bracketed note saying what you changed, e.g.,
> >
> > [felipe.contreras@xxxxxxxxx: dropped some changes, etc.]
> >
> > That way the provenance of the patch is clear.
> 
> Ok. I was hoping you would review the patch and put your s-o-b.

I have reviewed your updated patch; you are welcome to re-add my 
Signed-off-by:.


- Paul

[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux