[PATCH] UBUNTU: SAUCE: acpi: video: fix acpi_backlight=video

Kamal Mostafa kamal at canonical.com
Mon May 3 21:23:41 UTC 2010


On Sat, 2010-05-01 at 15:43 -0400, Chase Douglas wrote:
> On Sat, May 1, 2010 at 3:11 PM, Kamal Mostafa <kamal at canonical.com> wrote:
> > Make "acpi_backlight=video" param enable ACPI_VIDEO_BACKLIGHT_FORCE_VIDEO
> > as intended, instead of incorrectly enabling video output switching.
> >
> > BugLink: http://bugs.launchpad.net/bugs/573120
> >

Thanks for your feedback and ongoing coaching Chase.  :-)  I sincerely
appreciate it!

> Since this is post-release and not a kitten-killer, it should probably
> come directly from the stable tree.  [...]

Agreed.  I'm not inclined to push for this one as an Ubuntu SRU.  And I
already forwarded the patch upstream to LKML and linux-acpi.

Questions:

1. Was it okay that I sent it upstream and to kernel-team at l.u.c
simultaneously, or was I supposed to send to k-t first and wait for some
Ack's before sending upstream?

2. What would have been the proper subject line for this?  I mean, how
do I send a patch to k-t "for the purpose of getting some Ack's" without
asking that it be applied to any Ubuntu tree?  Should I just have left
out "UBUNTU: SAUCE:"?   Or maybe used  "[RFC PATCH]" also?

> 
> Otherwise, this looks good to me. You can ack me for the upstream
> submission if you want, though I don't carry much weight there yet :).

If I hadn't sent it off already, I certainly would have added your Ack.
I get the feeling that I did this "out of order".

 -Kamal

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20100503/85711e89/attachment.sig>


More information about the kernel-team mailing list