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 X-Spam-Level: X-Spam-Status: No, score=-5.0 required=3.0 tests=BAYES_00,DKIM_ADSP_CUSTOM_MED, DKIM_INVALID,DKIM_SIGNED,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9F144C4363C for ; Fri, 2 Oct 2020 23:59:36 +0000 (UTC) Received: from whitealder.osuosl.org (smtp1.osuosl.org [140.211.166.138]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 260E6206B7 for ; Fri, 2 Oct 2020 23:59:35 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="uV51v3n2" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 260E6206B7 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=gmail.com Authentication-Results: mail.kernel.org; spf=pass smtp.mailfrom=iommu-bounces@lists.linux-foundation.org Received: from localhost (localhost [127.0.0.1]) by whitealder.osuosl.org (Postfix) with ESMTP id A47B486D10; Fri, 2 Oct 2020 23:59:35 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from whitealder.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id hcjkmI8+i7Fe; Fri, 2 Oct 2020 23:59:34 +0000 (UTC) Received: from lists.linuxfoundation.org (lf-lists.osuosl.org [140.211.9.56]) by whitealder.osuosl.org (Postfix) with ESMTP id BBFB586D46; Fri, 2 Oct 2020 23:59:34 +0000 (UTC) Received: from lf-lists.osuosl.org (localhost [127.0.0.1]) by lists.linuxfoundation.org (Postfix) with ESMTP id A9176C0889; Fri, 2 Oct 2020 23:59:34 +0000 (UTC) Received: from silver.osuosl.org (smtp3.osuosl.org [140.211.166.136]) by lists.linuxfoundation.org (Postfix) with ESMTP id EF905C0051 for ; Fri, 2 Oct 2020 23:59:33 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by silver.osuosl.org (Postfix) with ESMTP id C4D1B20406 for ; Fri, 2 Oct 2020 23:59:33 +0000 (UTC) X-Virus-Scanned: amavisd-new at osuosl.org Received: from silver.osuosl.org ([127.0.0.1]) by localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id 1iyxHfo6uZCc for ; Fri, 2 Oct 2020 23:59:32 +0000 (UTC) X-Greylist: domain auto-whitelisted by SQLgrey-1.7.6 Received: from mail-pg1-f193.google.com (mail-pg1-f193.google.com [209.85.215.193]) by silver.osuosl.org (Postfix) with ESMTPS id 7F30220382 for ; Fri, 2 Oct 2020 23:59:32 +0000 (UTC) Received: by mail-pg1-f193.google.com with SMTP id q18so1877138pgk.7 for ; Fri, 02 Oct 2020 16:59:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to :user-agent; bh=cEwMT8zhmCOTK4dn4wJHnutFKvkcPru3sg5M5Ou9uiQ=; b=uV51v3n2ao+EwdBPLJkXHu/VhIwvBHMFZOSMkL+r9h5h+g274KejT08OjfQL7We0m9 Uw0QBu/jr9QMu1eOOv462dhyLHHg060jPWfyiq19WRnOCRZ/ELwmDGoxWzEalm9r/6SE KbIVX8T2SV1FxezmpWLedTryoK9HL2aWhme5O6GWoPKTFZRuwPJlcl+oloVUM7NoLW6a r4CDmPofaYOgrBuVER3otbXC/aAag6zaooih893H/nmfQvy11fAmgeCD/sxu10/JZbo7 Ucnsc/8peSOS+WbzdqeA65q6aWJIl+2EV5yTr+wb4s/vm/0N8Y5UfBpaI62D9GXsRi4H tF4Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=cEwMT8zhmCOTK4dn4wJHnutFKvkcPru3sg5M5Ou9uiQ=; b=euBnw9lx4LevuDVK7AZkXm7rprivGz7NEciVtKZoimMJjdY9nuPmDWXHIfN3FrCTIm UaXHMdxBTQDJFp1/3Cinl9ptElu2RoOJmk7OKiKVts5dQal7mnNuFHNLWcOW5q+zjCDR ziUyBILsA4nisonCENJNnjDXTrPn6A6+1OD8KiWClk0PGOkvWid5oV75EAW811W/e2M9 h2b2zkhbs5BbdnkROt7aLmhxHr1JGkHjFcQ1pYpuDDDXt1VOw3QB8jqg4SmJIzSXA4fg o0yX1piF/oQBdaPoHUHBiM4gwfClTV4IGjLscyrJiiNCNd1mFgnzhnYSzfO9vCNLvmUp oPhA== X-Gm-Message-State: AOAM531iFSiKTUshUaMKkP7TLMHAqWLUt5PRFnV90wcfNbaQueVlrq0n kMkp9ZvJ8m1GqMycWfluu/Q= X-Google-Smtp-Source: ABdhPJwAQRO8CGixT56i5K6YcLkU40fsKQ1AdlA/xQtYW8SDhWeD6964MtOOPLb5Vz4J0S+BkHIf9A== X-Received: by 2002:a05:6a00:174a:b029:152:2bb6:4877 with SMTP id j10-20020a056a00174ab02901522bb64877mr4848736pfc.29.1601683171943; Fri, 02 Oct 2020 16:59:31 -0700 (PDT) Received: from Asurada-Nvidia (thunderhill.nvidia.com. [216.228.112.22]) by smtp.gmail.com with ESMTPSA id i62sm3405092pfe.140.2020.10.02.16.59.31 (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 02 Oct 2020 16:59:31 -0700 (PDT) Date: Fri, 2 Oct 2020 16:53:30 -0700 From: Nicolin Chen To: Dmitry Osipenko Subject: Re: [PATCH v4 1/3] iommu/tegra-smmu: Use fwspec in tegra_smmu_(de)attach_dev Message-ID: <20201002235329.GA11409@Asurada-Nvidia> References: <20201002060807.32138-1-nicoleotsuka@gmail.com> <20201002060807.32138-2-nicoleotsuka@gmail.com> <20201002194508.GD29706@Asurada-Nvidia> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.4 (2018-02-28) Cc: linux-kernel@vger.kernel.org, iommu@lists.linux-foundation.org, jonathanh@nvidia.com, thierry.reding@gmail.com, linux-tegra@vger.kernel.org X-BeenThere: iommu@lists.linux-foundation.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Development issues for Linux IOMMU support List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: iommu-bounces@lists.linux-foundation.org Sender: "iommu" T24gRnJpLCBPY3QgMDIsIDIwMjAgYXQgMTE6MTI6MThQTSArMDMwMCwgRG1pdHJ5IE9zaXBlbmtv IHdyb3RlOgo+IDAyLjEwLjIwMjAgMjI6NDUsIE5pY29saW4gQ2hlbiDQv9C40YjQtdGCOgo+ID4g T24gRnJpLCBPY3QgMDIsIDIwMjAgYXQgMDU6NDE6NTBQTSArMDMwMCwgRG1pdHJ5IE9zaXBlbmtv IHdyb3RlOgo+ID4+IDAyLjEwLjIwMjAgMDk6MDgsIE5pY29saW4gQ2hlbiDQv9C40YjQtdGCOgo+ ID4+PiAgc3RhdGljIGludCB0ZWdyYV9zbW11X2F0dGFjaF9kZXYoc3RydWN0IGlvbW11X2RvbWFp biAqZG9tYWluLAo+ID4+PiAgCQkJCSBzdHJ1Y3QgZGV2aWNlICpkZXYpCj4gPj4+ICB7Cj4gPj4+ ICsJc3RydWN0IGlvbW11X2Z3c3BlYyAqZndzcGVjID0gZGV2X2lvbW11X2Z3c3BlY19nZXQoZGV2 KTsKPiA+Pj4gIAlzdHJ1Y3QgdGVncmFfc21tdSAqc21tdSA9IGRldl9pb21tdV9wcml2X2dldChk ZXYpOwo+ID4+PiAgCXN0cnVjdCB0ZWdyYV9zbW11X2FzICphcyA9IHRvX3NtbXVfYXMoZG9tYWlu KTsKPiA+Pj4gLQlzdHJ1Y3QgZGV2aWNlX25vZGUgKm5wID0gZGV2LT5vZl9ub2RlOwo+ID4+PiAt CXN0cnVjdCBvZl9waGFuZGxlX2FyZ3MgYXJnczsKPiA+Pj4gIAl1bnNpZ25lZCBpbnQgaW5kZXgg PSAwOwo+ID4+PiAgCWludCBlcnIgPSAwOwo+ID4+PiAgCj4gPj4+IC0Jd2hpbGUgKCFvZl9wYXJz ZV9waGFuZGxlX3dpdGhfYXJncyhucCwgImlvbW11cyIsICIjaW9tbXUtY2VsbHMiLCBpbmRleCwK PiA+Pj4gLQkJCQkJICAgJmFyZ3MpKSB7Cj4gPj4+IC0JCXVuc2lnbmVkIGludCBzd2dyb3VwID0g YXJncy5hcmdzWzBdOwo+ID4+PiAtCj4gPj4+IC0JCWlmIChhcmdzLm5wICE9IHNtbXUtPmRldi0+ b2Zfbm9kZSkgewo+ID4+PiAtCQkJb2Zfbm9kZV9wdXQoYXJncy5ucCk7Cj4gPj4+IC0JCQljb250 aW51ZTsKPiA+Pj4gLQkJfQo+ID4+PiAtCj4gPj4+IC0JCW9mX25vZGVfcHV0KGFyZ3MubnApOwo+ ID4+PiArCWlmICghZndzcGVjKQo+ID4+PiArCQlyZXR1cm4gLUVOT0VOVDsKPiA+Pgo+ID4+IENv dWxkIHRoZSAhZndzcGVjIGV2ZXIgYmUgdHJ1ZSBoZXJlIGFzIHdlbGw/Cj4gPiAKPiA+IFRoZXJl IGFyZSBtdWx0aXBsZSBjYWxsZXJzIG9mIHRoaXMgZnVuY3Rpb24uIEl0J3MgcmVhbGx5IG5vdCB0 aGF0Cj4gPiBzdHJhaWdodGZvcndhcmQgdG8gdHJhY2sgZXZlcnkgb25lIG9mIHRoZW0uIFNvIEkn ZCByYXRoZXIgaGF2ZSBpdAo+ID4gaGVyZSBhcyBvdGhlciBpb21tdSBkcml2ZXJzIGRvLiBXZSBh cmUgaHVtYW4gYmVpbmdzLCBzbyB3ZSBjb3VsZAo+ID4gaGF2ZSBtaXNzZWQgc29tZXRoaW5nIHNv bWV3aGVyZSwgZXNwZWNpYWxseSBjYWxsZXJzIGFyZSBub3QgZnJvbQo+ID4gdGVncmEtKiBkcml2 ZXJzLgo+ID4gCj4gCj4gSSdtIGxvb2tpbmcgYXQgdGhlIElPTU1VIGNvcmUgYW5kIGl0IHJlcXVp cmVzIGRldmljZSB0byBiZSBpbiBJT01NVQo+IGdyb3VwIGJlZm9yZSBhdHRhY2hfZGV2KCkgY291 bGQgYmUgY2FsbGVkLgo+IAo+IFRoZSBncm91cCBjYW4ndCBiZSBhc3NpZ25lZCB0byBkZXZpY2Ug d2l0aG91dCB0aGUgZndzcGVjLCBzZWUKPiB0ZWdyYV9zbW11X2RldmljZV9ncm91cCgpLgo+Cj4g U2VlbXMgbWFqb3JpdHkgb2YgSU9NTVUgZHJpdmVycyBhcmUgY2hlY2tpbmcgZGV2X2lvbW11X3By aXZfZ2V0KCkgZm9yCj4gTlVMTCBpbiBhdHRhY2hfZGV2KCksIHNvbWUgbm90IGNoZWNraW5nIGFu eXRoaW5nLCBzb21lIGNoZWNrIGJvdGggYW5kCj4gb25seSBhcm0tc21tdSBjaGVja3MgdGhlIGZ3 c3BlYy4KCkFzIEkgc2FpZCBhIGNvdXBsZSBvZiBkYXlzIGFnbywgSSBkb24ndCBsaWtlIHRvIGFz c3VtZSB0aGF0IHRoZQpjYWxsZXJzIHdvbid0IGNoYW5nZS4gQW5kIHRoaXMgdGltZSwgaXQncyBm cm9tIG9wZW4gY29kZS4gU28gSQpkb24ndCB3YW50IHRvIGFzc3VtZSB0aGF0IHRoZXJlIHdvbid0 IGJlIGEgY2hhbmdlLgoKSWYgeW91IGFyZSBjb25maWRlbnQgdGhhdCB0aGVyZSBpcyBubyBuZWVk IHRvIGFkZCBzdWNoIGEgY2hlY2ssCnBsZWFzZSBzZW5kIHBhdGNoZXMgdG8gcmVtb3ZlIHRob3Nl IGNoZWNrcyBpbiB0aG9zZSBkcml2ZXJzIHRvCnNlZSBpZiBvdGhlcnMgd291bGQgYWdyZWUuIEkg d291bGQgYmUgd2lsbGluZyB0byByZW1vdmUgaXQgYWZ0ZXIKdGhhdC4gT3RoZXJ3aXNlLCBJJ2Qg bGlrZSB0byBrZWVwIHRoaXMuCgpUaGFua3MgZm9yIHRoZSByZXZpZXcuCl9fX19fX19fX19fX19f X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmlvbW11IG1haWxpbmcgbGlzdAppb21t dUBsaXN0cy5saW51eC1mb3VuZGF0aW9uLm9yZwpodHRwczovL2xpc3RzLmxpbnV4Zm91bmRhdGlv bi5vcmcvbWFpbG1hbi9saXN0aW5mby9pb21tdQ== 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 X-Spam-Level: X-Spam-Status: No, score=-5.3 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 7516FC4363C for ; Fri, 2 Oct 2020 23:59:33 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 2AE52206DB for ; Fri, 2 Oct 2020 23:59:33 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="uV51v3n2" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1725379AbgJBX7c (ORCPT ); Fri, 2 Oct 2020 19:59:32 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58472 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1725283AbgJBX7c (ORCPT ); Fri, 2 Oct 2020 19:59:32 -0400 Received: from mail-pf1-x442.google.com (mail-pf1-x442.google.com [IPv6:2607:f8b0:4864:20::442]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 7D96BC0613D0; Fri, 2 Oct 2020 16:59:32 -0700 (PDT) Received: by mail-pf1-x442.google.com with SMTP id n14so2470219pff.6; Fri, 02 Oct 2020 16:59:32 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:content-transfer-encoding:in-reply-to :user-agent; bh=cEwMT8zhmCOTK4dn4wJHnutFKvkcPru3sg5M5Ou9uiQ=; b=uV51v3n2ao+EwdBPLJkXHu/VhIwvBHMFZOSMkL+r9h5h+g274KejT08OjfQL7We0m9 Uw0QBu/jr9QMu1eOOv462dhyLHHg060jPWfyiq19WRnOCRZ/ELwmDGoxWzEalm9r/6SE KbIVX8T2SV1FxezmpWLedTryoK9HL2aWhme5O6GWoPKTFZRuwPJlcl+oloVUM7NoLW6a r4CDmPofaYOgrBuVER3otbXC/aAag6zaooih893H/nmfQvy11fAmgeCD/sxu10/JZbo7 Ucnsc/8peSOS+WbzdqeA65q6aWJIl+2EV5yTr+wb4s/vm/0N8Y5UfBpaI62D9GXsRi4H tF4Q== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:content-transfer-encoding :in-reply-to:user-agent; bh=cEwMT8zhmCOTK4dn4wJHnutFKvkcPru3sg5M5Ou9uiQ=; b=KI6jDdHbfd6CyhHvBYR1HNdKD87Eg7jo8Z6SAMEHkxiT2ie9a6E/lT+3sqolzx/sVZ O0yQFurLZOtmcMpJmoNmIe7JhF+r0YkX5DmI4xpsqpcIt9crImojzo3reKyZ4AckYRrC MgL3n8M6sxbPz5B5j/snv2Ni4GbE4of4EWYjUhzQ91RBsvaLmUXjY9/Jf0Evn9A811YR So1VjQooe1jieKaOEa0Gn/RNF9wRM/S4iB8++wUuQ4An0W1z3OF4du+EmymP6uiU0cTD wKLlZa/0xnHlHsyA6qWxYcSTjfMf7beMkKw/4w94rUhxpOucwcIuSK12jtfinbQcynJY kQYg== X-Gm-Message-State: AOAM5315kvzwoFid7w3qyIuENBclarGDT/7UK59C4Q5LvYcHP3An/ntM 4Sl2r84dL/rloeX/2rJCY8Y= X-Google-Smtp-Source: ABdhPJwAQRO8CGixT56i5K6YcLkU40fsKQ1AdlA/xQtYW8SDhWeD6964MtOOPLb5Vz4J0S+BkHIf9A== X-Received: by 2002:a05:6a00:174a:b029:152:2bb6:4877 with SMTP id j10-20020a056a00174ab02901522bb64877mr4848736pfc.29.1601683171943; Fri, 02 Oct 2020 16:59:31 -0700 (PDT) Received: from Asurada-Nvidia (thunderhill.nvidia.com. [216.228.112.22]) by smtp.gmail.com with ESMTPSA id i62sm3405092pfe.140.2020.10.02.16.59.31 (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Fri, 02 Oct 2020 16:59:31 -0700 (PDT) Date: Fri, 2 Oct 2020 16:53:30 -0700 From: Nicolin Chen To: Dmitry Osipenko Cc: thierry.reding@gmail.com, joro@8bytes.org, vdumpa@nvidia.com, jonathanh@nvidia.com, linux-tegra@vger.kernel.org, iommu@lists.linux-foundation.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v4 1/3] iommu/tegra-smmu: Use fwspec in tegra_smmu_(de)attach_dev Message-ID: <20201002235329.GA11409@Asurada-Nvidia> References: <20201002060807.32138-1-nicoleotsuka@gmail.com> <20201002060807.32138-2-nicoleotsuka@gmail.com> <20201002194508.GD29706@Asurada-Nvidia> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/1.9.4 (2018-02-28) Precedence: bulk List-ID: X-Mailing-List: linux-tegra@vger.kernel.org On Fri, Oct 02, 2020 at 11:12:18PM +0300, Dmitry Osipenko wrote: > 02.10.2020 22:45, Nicolin Chen пишет: > > On Fri, Oct 02, 2020 at 05:41:50PM +0300, Dmitry Osipenko wrote: > >> 02.10.2020 09:08, Nicolin Chen пишет: > >>> static int tegra_smmu_attach_dev(struct iommu_domain *domain, > >>> struct device *dev) > >>> { > >>> + struct iommu_fwspec *fwspec = dev_iommu_fwspec_get(dev); > >>> struct tegra_smmu *smmu = dev_iommu_priv_get(dev); > >>> struct tegra_smmu_as *as = to_smmu_as(domain); > >>> - struct device_node *np = dev->of_node; > >>> - struct of_phandle_args args; > >>> unsigned int index = 0; > >>> int err = 0; > >>> > >>> - while (!of_parse_phandle_with_args(np, "iommus", "#iommu-cells", index, > >>> - &args)) { > >>> - unsigned int swgroup = args.args[0]; > >>> - > >>> - if (args.np != smmu->dev->of_node) { > >>> - of_node_put(args.np); > >>> - continue; > >>> - } > >>> - > >>> - of_node_put(args.np); > >>> + if (!fwspec) > >>> + return -ENOENT; > >> > >> Could the !fwspec ever be true here as well? > > > > There are multiple callers of this function. It's really not that > > straightforward to track every one of them. So I'd rather have it > > here as other iommu drivers do. We are human beings, so we could > > have missed something somewhere, especially callers are not from > > tegra-* drivers. > > > > I'm looking at the IOMMU core and it requires device to be in IOMMU > group before attach_dev() could be called. > > The group can't be assigned to device without the fwspec, see > tegra_smmu_device_group(). > > Seems majority of IOMMU drivers are checking dev_iommu_priv_get() for > NULL in attach_dev(), some not checking anything, some check both and > only arm-smmu checks the fwspec. As I said a couple of days ago, I don't like to assume that the callers won't change. And this time, it's from open code. So I don't want to assume that there won't be a change. If you are confident that there is no need to add such a check, please send patches to remove those checks in those drivers to see if others would agree. I would be willing to remove it after that. Otherwise, I'd like to keep this. Thanks for the review.