Re: [PATCH v3 11/22] PCI, drm: Kill pci_root_buses in alpha hose setting

2013-01-27 Thread Yijing Wang
On 2013/1/28 3:23, Yinghai Lu wrote: > Replace that with hotplug-safe version. > > Signed-off-by: Yinghai Lu > Cc: David Airlie > Cc: dri-devel@lists.freedesktop.org > --- > drivers/gpu/drm/drm_fops.c | 10 +++--- > 1 file changed, 7 insertions(+), 3 deletions(-) > > diff --git a/drivers

[PATCH 4/6] PCI/powerpc: Use list_for_each_entry() for bus traversal

2014-02-13 Thread Yijing Wang
Replace list_for_each() + pci_bus_b() with the simpler list_for_each_entry(). Signed-off-by: Yijing Wang --- arch/powerpc/kernel/pci_64.c |4 +--- arch/powerpc/platforms/pseries/pci_dlpar.c |6 +++--- 2 files changed, 4 insertions(+), 6 deletions(-) diff --git a/arch

[PATCH 5/6] PCI/pcmcia: Use list_for_each_entry() for bus traversal

2014-02-13 Thread Yijing Wang
Replace list_for_each() + pci_bus_b() with the simpler list_for_each_entry(). Signed-off-by: Yijing Wang --- drivers/pcmcia/yenta_socket.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/pcmcia/yenta_socket.c b/drivers/pcmcia/yenta_socket.c index 8485761

[PATCH 3/6] PCI/drm: Use list_for_each_entry() for bus traversal

2014-02-13 Thread Yijing Wang
Replace list_for_each() + pci_bus_b() with the simpler list_for_each_entry(). Signed-off-by: Yijing Wang --- drivers/gpu/drm/drm_fops.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/drivers/gpu/drm/drm_fops.c b/drivers/gpu/drm/drm_fops.c index 7f2af9a..70d2987

[PATCH 1/6] PCI,acpiphp: Use list_for_each_entry() for bus traversal

2014-02-13 Thread Yijing Wang
Replace list_for_each() + pci_bus_b() with the simpler list_for_each_entry(). Signed-off-by: Yijing Wang --- drivers/pci/hotplug/acpiphp_glue.c |6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/pci/hotplug/acpiphp_glue.c b/drivers/pci/hotplug/acpiphp_glue.c

[PATCH 6/6] PCI: Remove pci_bus_b() and use list_entry() directly

2014-02-13 Thread Yijing Wang
Replace pci_bus_b() with list_entry(), so we can remove pci_bus_b(). Signed-off-by: Yijing Wang --- drivers/pci/pci.c|6 +++--- drivers/pci/search.c | 10 +- include/linux/pci.h |1 - 3 files changed, 8 insertions(+), 9 deletions(-) diff --git a/drivers/pci/pci.c b

[PATCH 2/6] PCI/arm: Use list_for_each_entry() for bus traversal

2014-02-13 Thread Yijing Wang
Replace list_for_each() + pci_bus_b() with the simpler list_for_each_entry(). Signed-off-by: Yijing Wang --- arch/arm/kernel/bios32.c |7 ++- 1 files changed, 2 insertions(+), 5 deletions(-) diff --git a/arch/arm/kernel/bios32.c b/arch/arm/kernel/bios32.c index 317da88..0a77858 100644

[PATCH 1/6] PCI, acpiphp: Use list_for_each_entry() for bus traversal

2014-02-14 Thread Yijing Wang
On 2014/2/14 7:54, Rafael J. Wysocki wrote: > On Thursday, February 13, 2014 09:13:58 PM Yijing Wang wrote: >> Replace list_for_each() + pci_bus_b() with the simpler >> list_for_each_entry(). >> >> Signed-off-by: Yijing Wang > > Looks reasonable to me. &g

[PATCH 1/6] PCI, acpiphp: Use list_for_each_entry() for bus traversal

2014-02-15 Thread Yijing Wang
>>> Does it conflict with anything currently in linux-next (the linux-next >>> branch >>> of linux-pm.git in particular)? >> >> Hi Rafael, >>I applied this to your linux-next branch successfully . No conflicts >> found. > > Good. :-) > > Please feel free to add my ACK to it. Thanks very mu

[PATCH 1/6] PCI, acpiphp: Use list_for_each_entry() for bus traversal

2014-02-15 Thread Yijing Wang
On 2014/2/15 2:23, Bjorn Helgaas wrote: > On Thu, Feb 13, 2014 at 09:13:58PM +0800, Yijing Wang wrote: >> Replace list_for_each() + pci_bus_b() with the simpler >> list_for_each_entry(). >> >> Signed-off-by: Yijing Wang > > I applied all six of these (please

[PATCH v3 11/22] PCI, drm: Kill pci_root_buses in alpha hose setting

2013-01-28 Thread Yijing Wang
On 2013/1/28 3:23, Yinghai Lu wrote: > Replace that with hotplug-safe version. > > Signed-off-by: Yinghai Lu > Cc: David Airlie > Cc: dri-devel at lists.freedesktop.org > --- > drivers/gpu/drm/drm_fops.c | 10 +++--- > 1 file changed, 7 insertions(+), 3 deletions(-) > > diff --git a/driv

[PATCH v2] ACPI: Fix acpi_evaluate_object() return value check

2014-01-17 Thread Yijing Wang
>> diff --git a/drivers/gpu/drm/i915/intel_acpi.c >> b/drivers/gpu/drm/i915/intel_acpi.c >> index dfff090..7ea00e5 100644 >> --- a/drivers/gpu/drm/i915/intel_acpi.c >> +++ b/drivers/gpu/drm/i915/intel_acpi.c >> @@ -35,7 +35,7 @@ static int intel_dsm(acpi_handle handle, int func) >> union acpi

[PATCH v2] ACPI: Fix acpi_evaluate_object() return value check

2014-01-17 Thread Yijing Wang
Fix acpi_evaluate_object() return value check, shoud acpi_status not int. Signed-off-by: Yijing Wang --- v1->v2: Add CC to the related subsystem MAINTAINERS. --- drivers/gpu/drm/i915/intel_acpi.c | 13 +++-- drivers/gpu/drm/nouveau/core/subdev/mxm/base.c |

[PATCH v3] ACPI: Fix acpi_evaluate_object() return value check

2014-01-17 Thread Yijing Wang
Fix acpi_evaluate_object() return value check, shoud acpi_status not int. Signed-off-by: Yijing Wang --- v2->v3: Fix compile error pointed out by Hanjun. v1->v2: Add CC to related subsystem MAINTAINERS --- drivers/gpu/drm/i915/intel_acpi.c | 24 ++-- d

[PATCH v4] ACPI: Fix acpi_evaluate_object() return value check

2014-01-21 Thread Yijing Wang
Since acpi_evaluate_object() returns acpi_status and not plain int, ACPI_FAILURE() should be used for checking its return value. Reviewed-by: Jani Nikula Signed-off-by: Yijing Wang --- v3->v4: Fix spell error, add Jani Nikula reviewed-by. v2->v3: Fix compile error pointed out by Hanjun.

[PATCH v5] ACPI: Fix acpi_evaluate_object() return value check

2014-01-23 Thread Yijing Wang
Since acpi_evaluate_object() returns acpi_status and not plain int, ACPI_FAILURE() should be used for checking its return value. Also add some detailed debug info when acpi_evaluate_object() failed. Reviewed-by: Jani Nikula Acked-by: Bjorn Helgaas Signed-off-by: Yijing Wang --- v4->v5:

[PATCH v4] ACPI: Fix acpi_evaluate_object() return value check

2014-01-23 Thread Yijing Wang
On 2014/1/23 5:37, Bjorn Helgaas wrote: > On Mon, Jan 20, 2014 at 7:46 PM, Yijing Wang wrote: >> Since acpi_evaluate_object() returns acpi_status and not plain int, >> ACPI_FAILURE() should be used for checking its return value. >> >> Reviewed-by: Jani Nikula &g