On Thu, 2018-05-17 at 12:29 +0300, Mika Westerberg wrote:
> There is no need for them and it makes the code look uglier than it
> should. Remove them.
>
Reviewed-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> Signed-off-by: Mika Westerberg <mika.westerberg@xxxxxxxxxxxxxxx>
> Reviewed-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>
> ---
> drivers/pci/hotplug/acpiphp_glue.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/pci/hotplug/acpiphp_glue.c
> b/drivers/pci/hotplug/acpiphp_glue.c
> index db256bb2bd6a..07d511d62296 100644
> --- a/drivers/pci/hotplug/acpiphp_glue.c
> +++ b/drivers/pci/hotplug/acpiphp_glue.c
> @@ -522,7 +522,7 @@ static void enable_slot(struct acpiphp_slot *slot)
> if (!dev) {
> /* Do not set SLOT_ENABLED flag if some funcs
> are not added. */
> - slot->flags &= (~SLOT_ENABLED);
> + slot->flags &= ~SLOT_ENABLED;
> continue;
> }
> }
> @@ -551,7 +551,7 @@ static void disable_slot(struct acpiphp_slot
> *slot)
> list_for_each_entry(func, &slot->funcs, sibling)
> acpi_bus_trim(func_to_acpi_device(func));
>
> - slot->flags &= (~SLOT_ENABLED);
> + slot->flags &= ~SLOT_ENABLED;
> }
>
> static bool slot_no_hotplug(struct acpiphp_slot *slot)
--
Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
Intel Finland Oy
--
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
