Re: [PATCH v3 2/9] PM / Domains: Add generic OF-based PM domain look-up

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

 



On Monday, September 08, 2014 10:53:09 PM Geert Uytterhoeven wrote:
> Hi Rafael,
> 
> On Mon, Sep 8, 2014 at 11:08 PM, Rafael J. Wysocki <rjw@xxxxxxxxxxxxx> wrote:
> > BTW, I get bounces from t.figa@xxxxxxxxxxx, so I won't apply the patch with
> > that as the "From" field.  And the s-o-b from a bouncing address is worthless
> > too.
> 
> In the mean time, Tomasz has moved:
> http://www.spinics.net/lists/arm-kernel/msg357102.html
> 
> Still, the From/SoB indicate attribution to Samsung, so I think it should be
> kept.

The "From:" field should point to a submitter's address that is actually active,
so that (s)he can handle problem reports *sent* *to* *that* address.  I'm not
going to apply anything with a bouncing "From:".

The s-o-b is more of a gray area, but again it is supposed to be possible
to verify which it won't be if it bounces.

Also note that this is not Tomasz who is submitting this patch today, but Ulf
who doesn't even work for Samsung as far as I know.

So I'd suggest using "Tomasz Figa <tomasz.figa@xxxxxxxxx>" as a "From:", remove
his s-o-b entirely (and using the Ulf's one only) and add something like that
to the changelog:

"This patch was originally submitted by Tomasz Figa when he was employed
 by Samsung.

 Link: <URL-of-the-original-submission>"

This way Samsung will get the recognition and there won't be bouncing addresses
in tags etc.

Rafael

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]

  Powered by Linux