From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753877AbbIGG6k (ORCPT ); Mon, 7 Sep 2015 02:58:40 -0400 Received: from mail-lb0-f181.google.com ([209.85.217.181]:35745 "EHLO mail-lb0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753707AbbIGG6i (ORCPT ); Mon, 7 Sep 2015 02:58:38 -0400 Subject: Re: [PATCH 0/5] ACPI probing infrastructure To: Marc Zyngier , "Rafael J. Wysocki" , Len Brown , Hanjun Guo , Thomas Gleixner , Jason Cooper , Lorenzo Pieralisi , Sudeep Holla , Will Deacon , Catalin Marinas References: <1441386412-8139-1-git-send-email-marc.zyngier@arm.com> Cc: linaro-acpi@lists.linaro.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org From: Tomasz Nowicki Message-ID: <55ED3599.5090508@semihalf.com> Date: Mon, 7 Sep 2015 08:58:33 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Firefox/38.0 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: <1441386412-8139-1-git-send-email-marc.zyngier@arm.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Marc, On 04.09.2015 19:06, Marc Zyngier wrote: > IRQ controllers and timers are the two types of device the kernel > requires before being able to use the device driver model. > > ACPI so far lacks a proper probing infrastructure similar to the one > we have with DT, where we're able to declare IRQ chips and > clocksources inside the driver code, and let the core code pick it up > and call us back on a match. This leads to all kind of really ugly > hacks all over the arm64 code and even in the ACPI layer. > > It turns out that providing such a probing infrastructure is rather > easy, and provides a much deserved cleanup in both the arch code, the > GIC driver, and the architected timer driver. > > I'm sure there is some more code to be deleted, and one can only > wonder why this wasn't done before the arm64 code was initially merged > (the diffstat says it all...). > > Patches are against v4.2, and a branch is available at > > git://git.kernel.org/pub/scm/linux/kernel/git/maz/arm-platforms.git acpi/device-probing > > Marc Zyngier (5): > acpi: Add basic device probing infrastructure > irqchip/acpi: Add probing infrastructure for ACPI-based irqchips > irqchip/gic: Convert the GIC driver to ACPI probing > clocksource/acpi: Add probing infrastructure for ACPI-based > clocksources > clocksource/arm_arch_timer: Convert to ACPI probing > > arch/arm64/include/asm/acpi.h | 1 - > arch/arm64/include/asm/irq.h | 13 ------- > arch/arm64/kernel/acpi.c | 25 ------------- > arch/arm64/kernel/time.c | 6 ---- > drivers/acpi/scan.c | 41 +++++++++++++++++++++ > drivers/clocksource/arm_arch_timer.c | 10 +----- > drivers/clocksource/clksrc-of.c | 4 +++ > drivers/irqchip/irq-gic.c | 69 ++++++++++++++++++------------------ > drivers/irqchip/irqchip.c | 5 ++- > include/asm-generic/vmlinux.lds.h | 11 ++++++ > include/linux/acpi.h | 56 +++++++++++++++++++++++++++++ > include/linux/acpi_irq.h | 10 ------ > include/linux/clocksource.h | 6 ---- > include/linux/irqchip.h | 16 +++++++++ > include/linux/irqchip/arm-gic-acpi.h | 31 ---------------- > 15 files changed, 166 insertions(+), 138 deletions(-) > delete mode 100644 include/linux/acpi_irq.h > delete mode 100644 include/linux/irqchip/arm-gic-acpi.h > Thanks for cleaning this up! For the whole series: Acked-by: Tomasz Nowicki