From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 7944DC433F5 for ; Wed, 19 Jan 2022 11:42:49 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date: Message-ID:From:References:CC:To:Subject:Reply-To:Content-ID: Content-Description:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc :Resent-Message-ID:List-Owner; bh=t+ns+tujv0cM1k1IEzIe5Z4QO4VctHLCz7DWS0Pojqs=; b=ud9/aQwa+a+kXbzoQxvszeEHZ3 MirHCnUbQgWbw/gj/7FpQ9eXRmxbVgiKbdhZDxnvg+1539J83Un3UycxVLgwEFdy55jSp3TOsnq6K 4angGjoNzd+xXu9aEzQUeg0vIgiOUGhdXda9ZZjIOrGDCT8fU21CvP//5lyHvQImsfNrQGdRq3HCw uWlkVXzxA1OgW7vOzTHsPaIZB5OnZ3sn7WV1KPMpPMnZJTAusKtcX0GukZ7WpVb/7EaT5KZLGlIAA SnBVU+bfKTaHIGng0oNV05dyzyTHuuPzQ19p0QIhMApOrCxsw9vEqqTREnEL4voEb6E8uXDoQqG/s 5/Ipr85A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1nA9M6-005DCt-Sz; Wed, 19 Jan 2022 11:42:22 +0000 Received: from mxout04.lancloud.ru ([45.84.86.114]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1nA9M1-005DBc-PI; Wed, 19 Jan 2022 11:42:19 +0000 Received: from LanCloud DKIM-Filter: OpenDKIM Filter v2.11.0 mxout04.lancloud.ru 385F120D1C64 Received: from LanCloud Received: from LanCloud Received: from LanCloud Subject: Re: [PATCH 1/2] platform: make platform_get_irq_optional() optional (summary) To: =?UTF-8?Q?Uwe_Kleine-K=c3=b6nig?= CC: Greg Kroah-Hartman , Andrew Lunn , Ulf Hansson , Vignesh Raghavendra , , "Rafael J. Wysocki" , , Linus Walleij , Amit Kucheria , , Andy Shevchenko , Joakim Zhang , Guenter Roeck , Thierry Reding , , , Miquel Raynal , , , Lee Jones , , "Khuong Dinh" , Florian Fainelli , Matthias Schiffer , Kamal Dasu , "Jiri Slaby" , Bartosz Golaszewski , "Daniel Lezcano" , Kishon Vijay Abraham I , , , Jakub Kicinski , Zhang Rui , "Jaroslav Kysela" , , Hans de Goede , Robert Richter , Saravanan Sekar , Corey Minyard , , Mauro Carvalho Chehab , "John Garry" , Peter Korsgaard , "William Breathitt Gray" , Mark Gross , , Alex Williamson , Mark Brown , Borislav Petkov , "Sebastian Reichel" , Matthias Brugger , "Takashi Iwai" , , Benson Leung , , , Tony Luck , Mun Yew Tham , Eric Auger , , Yoshihiro Shimoda , Cornelia Huck , , Liam Girdwood , , , "Vinod Koul" , James Morse , Zha Qipeng , Pengutronix Kernel Team , Richard Weinberger , =?UTF-8?Q?Niklas_S=c3=b6derlund?= , , "Brian Norris" , "David S. Miller" References: <20220110195449.12448-1-s.shtylyov@omp.ru> <20220110195449.12448-2-s.shtylyov@omp.ru> <20220115183643.6zxalxqxrhkfgdfq@pengutronix.de> <20220118091819.zzxpffrxbckoxiys@pengutronix.de> <20220119113314.tpqfdgi6nurmzfun@pengutronix.de> From: Sergey Shtylyov Organization: Open Mobile Platform Message-ID: Date: Wed, 19 Jan 2022 14:42:09 +0300 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.10.1 MIME-Version: 1.0 In-Reply-To: <20220119113314.tpqfdgi6nurmzfun@pengutronix.de> Content-Language: en-US X-Originating-IP: [192.168.11.198] X-ClientProxiedBy: LFEXT02.lancloud.ru (fd00:f066::142) To LFEX1907.lancloud.ru (fd00:f066::207) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220119_034218_207935_79A26F05 X-CRM114-Status: GOOD ( 16.05 ) X-BeenThere: linux-mtd@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: quoted-printable Sender: "linux-mtd" Errors-To: linux-mtd-bounces+linux-mtd=archiver.kernel.org@lists.infradead.org On 1/19/22 2:33 PM, Uwe Kleine-K=F6nig wrote: [...] >>>>> A possible compromise: We can have both. We rename >>>>> platform_get_irq_optional() to platform_get_irq_silent() (or >>>>> platform_get_irq_silently() if this is preferred) and once all users = are >>>>> are changed (which can be done mechanically), we reintroduce a >>>>> platform_get_irq_optional() with Sergey's suggested semantic (i.e. >>>>> return 0 on not-found, no error message printking). >>>> >>>> Please do not do that as anyone trying to forward-port an old driver >>>> will miss the abi change of functionality and get confused. Make >>>> build-breaking changes, if the way a function currently works is >>>> changed in order to give people a chance. >>> >>> Fine for me. I assume this is a Nack for Sergey's patch? >> >> Which patch do you mean? I'm starting to get really muddled... :-( > = > I'm talking about "[PATCH 1/2] platform: make > platform_get_irq_optional() optional" I thought GregKH was talking about your renaming patch... :-/ > because "trying to forward-port an > old driver will miss the abi" applies to it. Mhm... why not tell me right from the start? Jr even tell that to Andy instead of merging his patch, so I wouldn't get sucked into this work? = I wouldn't bother with v2 and it would have saved a lot of time spent on email... :-( Do we also remember that "the stable API is a nonsense" thing? :-) > Best regards > Uwe MBR, Sergey ______________________________________________________ Linux MTD discussion mailing list http://lists.infradead.org/mailman/listinfo/linux-mtd/