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=-12.7 required=3.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,FREEMAIL_FORGED_FROMDOMAIN,FREEMAIL_FROM, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=unavailable 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 D1863C4338F for ; Fri, 6 Aug 2021 15:39:54 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id AF37D60EE7 for ; Fri, 6 Aug 2021 15:39:54 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S244753AbhHFPkJ (ORCPT ); Fri, 6 Aug 2021 11:40:09 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:46920 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S244736AbhHFPkG (ORCPT ); Fri, 6 Aug 2021 11:40:06 -0400 Received: from mail-ed1-x530.google.com (mail-ed1-x530.google.com [IPv6:2a00:1450:4864:20::530]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9F878C0613CF; Fri, 6 Aug 2021 08:39:50 -0700 (PDT) Received: by mail-ed1-x530.google.com with SMTP id x14so13557090edr.12; Fri, 06 Aug 2021 08:39:50 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20161025; h=from:to:cc:subject:date:message-id:in-reply-to:references :mime-version:content-transfer-encoding; bh=eTk4yr0eN3hm7lLtT7TJ+GcZ/BI3UsKqBaJTxX9s20I=; b=pe/zubRk3mozz1yIUrLLk1qW9TXvaqf1wg8raPZZYwlgaDMsBtQ5EDWfwopd9beKl4 USnByrcmC0Cw74780PIXSlzgIoQkiso5bHCfa/BdkLff7QVTcYcsZe3no4MRwTd3aipr 7Y+h8H2Ni2e6qkmhORuMQUvJCW8VZW2dfwrXQZeCrW9IwUlX4HvWMdfxBkmTHkOV54A/ Lb0VsTz+XP6P/gNmMQsaGMlQQ4B32Rf2qlFeryKtYndgmXoej2XSU0XWlQBbiQsHi9j/ kDoCZYS/MMIepYDppzIGPTiUFkj8816MtmjKrXnw+3bkVqM2BaOg8BXDzH9n3O9o3dx5 Txgw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references:mime-version:content-transfer-encoding; bh=eTk4yr0eN3hm7lLtT7TJ+GcZ/BI3UsKqBaJTxX9s20I=; b=rr7Cf8pHFFwagvV078n6heTwEihb8tHFFomuvndSUAtVUoo027mgo3N1YYyyQW3Yin WaQB0zFHescjEzHcP3NmetYLq2w8gnYOzp9vF6JaQgOR9urPANnsdIfMvwrfYPzODoCb gVCk23sQBmvnEZ8np7eG5slypAMiY/Nu0FTb4B6hZLMWlYGOVENIcdNT7W4omo176O43 tTXGi/TsTM+NJj5odcTaPHE7ceVNxw1EXQ+TIfY3L0jb/wdnsEM/zXRy8nX9Fqba1lI/ 4P0ib3psmi5WqfuAd39MZaHlNGVMzvftLekEpC0IFFB5YZb9J7kJmxKgz4b1KthZ1waU C+8g== X-Gm-Message-State: AOAM532z9ZVLJpiCINVqknbKY/qabNtcjN3QXHLXyqt1FApy7JaK+RYg Yd+ZYHAWkR05Qc/4sG6pr2k= X-Google-Smtp-Source: ABdhPJwf+uCRRQJXeuBWe+3Hl730847C6jJePnZrXtrwCgIpfunEruQsEVRxOsUqaRz/YFxnW+J1LA== X-Received: by 2002:a05:6402:1ad9:: with SMTP id ba25mr13595781edb.255.1628264389312; Fri, 06 Aug 2021 08:39:49 -0700 (PDT) Received: from jernej-laptop.localnet (89-212-178-211.dynamic.t-2.net. [89.212.178.211]) by smtp.gmail.com with ESMTPSA id bm1sm2997309ejb.38.2021.08.06.08.39.48 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Fri, 06 Aug 2021 08:39:48 -0700 (PDT) From: Jernej =?utf-8?B?xaBrcmFiZWM=?= To: p.zabel@pengutronix.de, Ezequiel Garcia Cc: mchehab@kernel.org, gregkh@linuxfoundation.org, hverkuil-cisco@xs4all.nl, emil.velikov@collabora.com, linux-media@vger.kernel.org, linux-rockchip@lists.infradead.org, linux-staging@lists.linux.dev, linux-kernel@vger.kernel.org Subject: Re: [PATCH] media: hantro: Fix check for single irq Date: Fri, 06 Aug 2021 17:39:47 +0200 Message-ID: <1909651.VgzLf9ffF6@jernej-laptop> In-Reply-To: <79673df0562db410753c90f9957125f202c5a1b2.camel@collabora.com> References: <20210805190416.332563-1-jernej.skrabec@gmail.com> <8101406.vZ8PxZ7URt@jernej-laptop> <79673df0562db410753c90f9957125f202c5a1b2.camel@collabora.com> MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="UTF-8" Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Dne petek, 06. avgust 2021 ob 16:13:46 CEST je Ezequiel Garcia napisal(a): > On Fri, 2021-08-06 at 06:44 +0200, Jernej =C5=A0krabec wrote: > > Dne petek, 06. avgust 2021 ob 00:03:36 CEST je Ezequiel Garcia napisal(= a): > > > Hi Jernej, > > >=20 > > > On Thu, 2021-08-05 at 21:04 +0200, Jernej Skrabec wrote: > > > > Some cores use only one interrupt and in such case interrupt name in > > > > DT > > > > is not needed. Driver supposedly accounted that, but due to the wro= ng > > > > field check it never worked. Fix that. > > > >=20 > > > > Fixes: 18d6c8b7b4c9 ("media: hantro: add fallback handling for sing= le > > > > irq/clk") Signed-off-by: Jernej Skrabec > > > > --- > > > > drivers/staging/media/hantro/hantro_drv.c | 2 +- > > > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > >=20 > > > > diff --git a/drivers/staging/media/hantro/hantro_drv.c > > > > b/drivers/staging/media/hantro/hantro_drv.c index > > > > 8a2edd67f2c6..20e508158871 100644 > > > > --- a/drivers/staging/media/hantro/hantro_drv.c > > > > +++ b/drivers/staging/media/hantro/hantro_drv.c > > > > @@ -919,7 +919,7 @@ static int hantro_probe(struct platform_device > > > > *pdev) > > > > if (!vpu->variant->irqs[i].handler) > > > > continue; > > > > =20 > > > > - if (vpu->variant->num_clocks > 1) { > > > > + if (vpu->variant->num_irqs > 1) { > > >=20 > > > Oops, thanks for spotting this. > > >=20 > > > How about this instead? > >=20 > > No, original solution is more robust. With solution below, you're assum= ing > > that irq order in driver array is same as in DT. That doesn't matter if > > there is only one name or if names match. However, if there is a typo, > > either in DT node or in driver, driver will still happily assign clock > > based on index and that might not be correct one. Even if it works out, > > you can easily miss that you have a typo. Driver doesn't tell you which > > irq is used, if it is successfully acquired. >=20 > I find it odd to iterate up to num_irqs but then > have a case for num_irqs =3D=3D 1, and call > platform_get_irq(vpu->pdev, 0). True, it was also strange for me at first, but then it's robust and simple.= =20 Because of that, I just fixed obvious issue. >=20 > But OTOH, your fix is correct and it's a oneliner. >=20 > Reviewed-by: Ezequiel Garcia >=20 Thanks! Best regards, Jernej > Thanks, > Ezequiel