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 9856DC46CD2 for ; Wed, 24 Jan 2024 08:48:59 +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-Type: Content-Transfer-Encoding:List-Subscribe:List-Help:List-Post:List-Archive: List-Unsubscribe:List-Id:In-Reply-To:From:References:Cc:To:Subject: MIME-Version:Date:Message-ID:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=dpOCKVLRo2D9Rx3YTv24t0h1wm2BkwwazNNwKzO9e/4=; b=os2UZOAG6x04kq aGt+vBfCrAsyFxlC6UGygHj6NFaoFlptstvwKoNGzP8BkgElfkro0KvOgBokSAJxP/1e1XzV/rgFI pMJeZt1ti44kNVt5QKxf0CExCciUNuXiZ5V95NKNtg8ZIIozgcZrIFka7PzaCxzvz1/v6j/FtIXtE 5XeUBCxTU+2EJbslcrF67AlYjYWzOZbvAynr4l7RN1vBhnyw98nSL+faGcIfLkZK16J6EtE3gc1eo OWxMe4A9p5UUiqcjO/uSmPkI5sgs/j5q0vT7yAExRg+2fKSPuFwY3ytArKAmV3zZ5GfstMMuJWVl7 PHFWpV0ciDH10p+ffPgA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1rSYvt-0023ki-28; Wed, 24 Jan 2024 08:48:29 +0000 Received: from madrid.collaboradmins.com ([46.235.227.194]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1rSYvq-0023jY-2g; Wed, 24 Jan 2024 08:48:28 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=collabora.com; s=mail; t=1706086105; bh=ekyS4ZOSSZIbtYGGpgL9knQnsxr3+M0NajqbJuU+F1k=; h=Date:Subject:To:Cc:References:From:In-Reply-To:From; b=BCoTd5gMIKJXbmId+RePuOy78x9TNjz+IVtfVvT8aMoOvt6GFXUMFfiK2evKiYXvb oZyO0AEFs4Dmavi4mJMbhG5F4NI7ioouzVKGC/NVwuEFsJeVkc0nhffNOnErG725n+ VtKyB0hhyArbCln7dl/FSvP+0nKz/P6X/YkzIwnVFCR4mxTUoH1ucl9082Tsyqo2HN ICc7ksvioMRnWFs40WWbkN7ey5nw81o90DtxhGOwuLlF5S8LU17lSrTx0Hkk1mH8XR FvrHj7IpUcE8EFEQjdNOisxGchNMHFqBOSa4cbtHks5G06EP+uMMFJttyKdkxUEagA lchYKYqg7tOzA== Received: from [100.113.186.2] (cola.collaboradmins.com [195.201.22.229]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) (Authenticated sender: kholk11) by madrid.collaboradmins.com (Postfix) with ESMTPSA id 89EB137820B7; Wed, 24 Jan 2024 08:48:24 +0000 (UTC) Message-ID: <4fdbc3d8-3d44-4c2c-aae6-daa0b431e1c9@collabora.com> Date: Wed, 24 Jan 2024 09:48:23 +0100 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 1/2] dt-bindings: usb: mt6360-tcpc: Drop interrupt-names Content-Language: en-US To: Conor Dooley , robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org Cc: chunfeng.yun@mediatek.com, gregkh@linuxfoundation.org, conor+dt@kernel.org, matthias.bgg@gmail.com, linux@roeck-us.net, heikki.krogerus@linux.intel.com, cy_huang@richtek.com, linux-usb@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-mediatek@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20240119094105.98312-1-angelogioacchino.delregno@collabora.com> <20240119-eldest-discharge-e2d3812be0a9@spud> <12b7b339-498b-45c1-bc5e-05e07660aefa@collabora.com> <20240123-procurer-jumbo-ebbec485505d@spud> From: AngeloGioacchino Del Regno In-Reply-To: <20240123-procurer-jumbo-ebbec485505d@spud> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240124_004827_001751_622C8636 X-CRM114-Status: GOOD ( 16.88 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Il 23/01/24 18:14, Conor Dooley ha scritto: > On Mon, Jan 22, 2024 at 11:32:30AM +0100, AngeloGioacchino Del Regno wrote: >> Il 19/01/24 17:32, Conor Dooley ha scritto: >>> On Fri, Jan 19, 2024 at 10:41:04AM +0100, AngeloGioacchino Del Regno wrote: >>>> This IP has only one interrupt, hence interrupt-names is not necessary >>>> to have. >>>> Since there is no user yet, simply remove interrupt-names. >>> >>> I'm a bit confused chief. Patch 2 in this series removes a user of this >>> property from a driver, so can you explain how this statement is true? >>> >>> Maybe I need to drink a few cans of Monster and revisit this patchset? >>> >> >> What I mean with "there is no user" is that there's no device tree with any >> mt6360-tcpc node upstream yet, so there is no meaningful ABI breakage. >> Different story would be if there was a device tree using this already, in >> which case, you can make a required property optional but not remove it. > > Not every devicetree lives within the kernel.. If the driver is using > it, I'm not inclined to agree that it should be removed. I get the point, but as far as I remember, it's not the first time that this kind of change is upstreamed. I'm fine with keeping things as they are but, since my intention is to actually introduce an actual user of this binding upstream, and that actually depends on if this change is accepted or not (as I have to know whether I can omit adding the interrupt-names property or not).... ....may I ask for more feedback/opinions from Rob and/or Krzk? Thanks, Angelo _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel