On Mon, Nov 25, 2013 at 09:34:27AM +0800, Yijing Wang wrote: > On 2013/11/23 6:39, Tejun Heo wrote: > > On Wed, Nov 20, 2013 at 4:28 AM, Yijing Wang <wangyijing@xxxxxxxxxx> wrote: > >> Since acpi_bus_get_device() returns plain int and not acpi_status, > >> ACPI_FAILURE() should not be used for checking its return value. Fix > >> that. > >> > >> Signed-off-by: Yijing Wang <wangyijing@xxxxxxxxxx> > >> Cc: Tejun Heo <tj@xxxxxxxxxx> > >> Cc: linux-ide@xxxxxxxxxxxxxxx > > > > Applied to libata/for-3.13-fixes. Thanks. > > Hi Tejun, > Sorry for disturb you, this patch has a build warning found by linux-next kbuild test. > > All warnings: > > drivers/ata/libata-zpodd.c: In function 'odd_can_poweroff': > >> drivers/ata/libata-zpodd.c:91:14: warning: unused variable 'status' [-Wunused-variable] > acpi_status status; > > > So I send a new patch "[PATCH v2] ata: Fix acpi_bus_get_device() return value check" to you, > can you help to update it. I already fixed it up last week. Thanks. -- tejun -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html