From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751853AbbIQK56 (ORCPT ); Thu, 17 Sep 2015 06:57:58 -0400 Received: from mail-wi0-f178.google.com ([209.85.212.178]:35835 "EHLO mail-wi0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751529AbbIQK55 (ORCPT ); Thu, 17 Sep 2015 06:57:57 -0400 Message-ID: <55FA9CB1.8010402@linaro.org> Date: Thu, 17 Sep 2015 12:57:53 +0200 From: Daniel Lezcano User-Agent: Mozilla/5.0 (X11; Linux i686; rv:31.0) Gecko/20100101 Thunderbird/31.8.0 MIME-Version: 1.0 To: Thomas Gleixner CC: Caesar Wang , Heiko Stuebner , Catalin Marinas , Will Deacon , Caesar Wang , linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 1/3] clocksource: rockchip: Make the driver more readability and compatible References: <1442476272-31723-1-git-send-email-wxt@rock-chips.com> <1442476272-31723-2-git-send-email-wxt@rock-chips.com> <55FA83D5.9010504@linaro.org> <55FA87AA.4040807@gmail.com> <55FA9099.7000903@linaro.org> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 09/17/2015 12:19 PM, Thomas Gleixner wrote: > On Thu, 17 Sep 2015, Daniel Lezcano wrote: >> On 09/17/2015 11:28 AM, Caesar Wang wrote: >>>> I think the NO_IRQ definition is missing for ARM64. >>> >>> Yep, Maybe better to compatible if we don't use the 'NO_IRQ', >> >> Hmm, after digging into drivers/of/irq.c and kernel/irq/irqdomain.c >> >> when there is an error it returns zero. So NO_IRQ and -1 are not correct and >> on the other side zero can be a valid irq. That sounds a little bit fuzzy to >> me. > > IRQ0 is invalid for historical reasons. End of story. Hi Thomas, there is one thing I don't understand. If the IRQ0 is invalid, irq_of_parse_and_map returning zero means an error and from what you said it is ok. But I see the NO_IRQ on ARM is (-1) and the drivers are checking with NO_IRQ the return code of irq_of_parse_and_map. So if there is an error, that won't be detected. For this specific use case above, shall irq_of_parse_and_map returns NO_IRQ or the caller checks against zero ? Beside that, some drivers are internally defining NO_IRQ: drivers/scsi/NCR5380.h drivers/ata/sata_dwc_460ex.c drivers/input/touchscreen/ucb1400_ts.c drivers/mmc/host/of_mmc_spi.c drivers/rtc/rtc-m48t59.c drivers/scsi/NCR5380.h I don't know the historical changes and the subtleties of the irq subsystem (I guess it is considerably complicated by each architecture specific bits and the drivers supported on different architectures) If you have the time, can you give some clarification ? Thanks -- Daniel -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog