From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH resend 2/4] backlight: Add backlight device (un)registration notification Date: Tue, 27 May 2014 10:20:33 +0100 Message-ID: <20140527092033.GF5875@lee--X1> References: <1400679596-19663-1-git-send-email-hdegoede@redhat.com> <20140522090215.GO6679@lee--X1> <000301cf788f$1a59e870$4f0db950$%han@samsung.com> <2139692.XxRO5tdkik@vostro.rjw.lan> <538323A3.1050209@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Received: from mail-ig0-f173.google.com ([209.85.213.173]:33871 "EHLO mail-ig0-f173.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751615AbaE0JU7 (ORCPT ); Tue, 27 May 2014 05:20:59 -0400 Received: by mail-ig0-f173.google.com with SMTP id hn18so814936igb.12 for ; Tue, 27 May 2014 02:20:58 -0700 (PDT) Content-Disposition: inline In-Reply-To: <538323A3.1050209@redhat.com> Sender: linux-acpi-owner@vger.kernel.org List-Id: linux-acpi@vger.kernel.org To: Hans de Goede Cc: "Rafael J. Wysocki" , Jingoo Han , 'Aaron Lu' , 'Bryan Wu' , 'Jean-Christophe Plagniol-Villard' , 'Tomi Valkeinen' , 'Ben Skeggs' , 'David Airlie' , 'Zhang Rui' , 'Len Brown' , linux-acpi@vger.kernel.org, linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org > On 05/26/2014 01:03 PM, Rafael J. Wysocki wrote: > > On Monday, May 26, 2014 12:03:43 PM Jingoo Han wrote: > >> On Thursday, May 22, 2014 6:02 PM, Lee Jones wrote: > >>> On Thursday, May 22, 2014 5:45 PM, Hans de Goede wrote: > >>>> On Thursday, May 22, 2014 8:31 AM, Rafael J. Wysocki wrote: > >>>>> On Wednesday, May 21, 2014 10:40 PM, Hans de Goede wrote: > >>>>>> Some firmware drivers, ie acpi-video want to get themselves ou= t of the > >>>>>> way (in some cases) when their also is a raw backlight device = available. > >>>>>> > >>>>>> Due to module loading ordering being unknown, acpi-video canno= t be certain > >>>>>> that the backlight_device_registered(BACKLIGHT_RAW) it does fo= r this is > >>>>>> the final verdict wrt there being a BACKLIGHT_RAW device. > >>>>>> > >>>>>> By adding notification acpi-video can listen for backlight dev= ices showing > >>>>>> up after it has loaded, and unregister its backlight device if= desired. > >>>>>> > >>>>>> Signed-off-by: Hans de Goede > >>>>> > >>>>> Backlight maintainer's ACK is requisite here. > >>>> > >>>> Agreed, which is why I send this set to all 3 the backlight main= tainers > >>>> directly on both postings. > >>>> > >>>> What may be helpful for them is to hear from you if you're ok wi= th the > >>>> acpi-video bits which are actually going to use this, since thos= e will > >>>> be the only user of the new backlight api (for now). > >>> > >>> I'm happy to apply any Backlight patches which have either Bryan = or > >>> Jingoo's Ack, as they are the reviewers for the BL subsystem. > >> > >> Acked-by: Jingoo Han > >> > >> Lee Jones, > >> Would you merge this patch into your backlight git tree? > >=20 > > Hans, does this series depend on things that I've applied already? = If so, > > I'd very much prefer to take this series too as a whole. >=20 > The 3th patch in this series: > " acpi-video: Unregister the backlight device if a raw one shows up l= ater" > depends on my "acpi-video: Add an acpi_video_unregister_backlight fun= ction" > patch, which you've applied to your linux-next branch already. >=20 > As well as on the 2nd patch in this series: > "backlight: Add backlight device (un)registration notification" >=20 > So I agree that it is a good idea to take the whole series through yo= ur tree. I'm fine with that. Rafael, could you apply the set onto an immutable branch and send me a signed pull-request please? --=20 Lee Jones Linaro STMicroelectronics Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog -- To unsubscribe from this list: send the line "unsubscribe linux-acpi" i= n the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html