From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f54.google.com (mail-wr1-f54.google.com [209.85.221.54]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 191CC1F0E2F for ; Thu, 20 Feb 2025 09:47:56 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.54 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740044879; cv=none; b=eL9VCf1hnfLqfL3KuSFlxV4X31U8fe83fbgC2OIJQaI+ib5eN2EmKYLQWcXNj8eFwC9PjbG7eQiniaxTaGlRgUcUpGR4SHd9DKmBI0WOs/EyOpZV/IZMNzL7arOMx5CgVtQlMbWBkn7e1szDZlhH7qZxW5mc3ZUGrC3JczATfMg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1740044879; c=relaxed/simple; bh=IWsg9d6lbEashhnUPHSkRjmEXhpeKPSMS9vOhHCIlfY=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=mb4YFA6kx0CYshfc0qddU4qW+5SBSKniGuruVWzphkJqVDxwLBnOF2pdi911SKTOHiALLt9hYAD/lq2GgMX6LsRkik2FrLA+Ti2qHnHA0eODu1LE6JcIoT88dyyQGZawch31JJgCLrMI0OmDkBiVc9e7j3Ch5/0mn4yZJRE7dyw= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=T3FUE++P; arc=none smtp.client-ip=209.85.221.54 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="T3FUE++P" Received: by mail-wr1-f54.google.com with SMTP id ffacd0b85a97d-38de1a5f039so547287f8f.2 for ; Thu, 20 Feb 2025 01:47:56 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1740044875; x=1740649675; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:from:to:cc:subject:date:message-id:reply-to; bh=3C+kognpjmHLZgIVXwXKpXpKvfpiYLWcBqUjAjWp7Tc=; b=T3FUE++PN0U3+a3SEBSaJHD8QqhO+UgkgQYM8dZRMXZTGqpAYZyJgQT0Q1+6jGr07H H/iUU00igxlY5xiwrjsc19xEVFpDgGInfAqueY4x3/u4hGxRg3JATpGpYuMbtOAnuilk MXfFw17oXJIEnn2s+1bTZOg920iciE+LRztDeurB7uIsV1j3UQQFZ3O4+0jWkcyvf1ca 82mvTdeOWU62CIiq8nCmKCtRc/Hsg/Q+wuS7wV2OIJiyuee0tC0WsLCdj1zUQgiTbC4u qHmGahno5mqJsr+fY/iMRbFD/Vl1pJ8HWZfCVueAYZw1spkCPghqbvnjm3snlyEnP747 b1kg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1740044875; x=1740649675; h=in-reply-to:content-disposition:mime-version:references:message-id :subject:cc:to:from:date:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=3C+kognpjmHLZgIVXwXKpXpKvfpiYLWcBqUjAjWp7Tc=; b=irCVshCgV6nxkyXvrtlIfuoM+9NHULFf9Pdcca0M3k4lcrR0kaaOggk98/uxvJMa4n OXNHTbwvxjomU3e7MTITHdN1Up8Pn1Qsi3kpJKoTjtl3+0iWD1A7FocOC+Pi8KK8axZ3 bZH/vp6tA91cOPFDmTzfgkvkNBBuN3p2Hwb5PetPPYoEvYZeq28hKRBwMGDetnR2lE1Z XBCOTf2f6GyZI6iq9VcVM5zd3YwJMYOeFWlu1/8msX2JdLHjCPLD938Xy9llnbvUS3gV D5GVIP9aJXq5RBXfdV2jMrid4nAG264Ego+Et87uPtQA1VXZ/DPT+Tn6Nd2FXGdJg3m8 f7rA== X-Forwarded-Encrypted: i=1; AJvYcCWhZ0q6jDuCO8pmDxPaOWxvlQPdAUgJGHHRX106su+bfvN4n1uPiDIz/Cjzgh59zjEJK8iwFKlc/Jlh@vger.kernel.org X-Gm-Message-State: AOJu0YymzfFhDtUTfc9Tr2JVDumeyf4aCpDEEq7wNsCDt9CsR4y/+sZf Mrgl35u2EFT1icsu+aaBCO+cmE07L1RbdDnIEUnvKzLq689FPb6eqN9rDzJgevk= X-Gm-Gg: ASbGncsnLycpNE1mYZLklcu9P8oac/SyCZ42mpPucN+ghpxYhkDpzntIft4KAkHuVk5 O+mQ4/7HBMOIY7l/9YijGaYNFycZXdY9YYMLeZAPIqTn3ss5fhmScEwuyjlPx5K98Ihpnrnh40p fJppDUmAGkYY1r0/8WlmXFVhAzh1RX1Bk9hxOT8LXOKh8t3Bx6J/595mtE8a0mrbGsmKXm7rP5S vkiveDCtllRsniMLHjfItn2RZFRVQX+VvT7cbMpoy+a3dnHGuElhyhvdG8kzrH5ShftdYOVX3P2 NaS89y957TPTGSHYKXV6ByNLeB8= X-Google-Smtp-Source: AGHT+IEsCez3GPLSZunwwFDyUVqq7Wkm5lBgpW3JvRE36iu83J5JlqvN9UXFhpA6HMjqTat9XWSXuQ== X-Received: by 2002:adf:f08c:0:b0:38f:30c7:eae4 with SMTP id ffacd0b85a97d-38f33f56507mr18120522f8f.52.1740044875163; Thu, 20 Feb 2025 01:47:55 -0800 (PST) Received: from linaro.org ([2a02:2454:ff21:ef30:b30c:3d94:4d4a:a6eb]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-38f561bee3esm5514525f8f.21.2025.02.20.01.47.54 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 20 Feb 2025 01:47:54 -0800 (PST) Date: Thu, 20 Feb 2025 10:47:49 +0100 From: Stephan Gerhold To: Andi Shyti Cc: Wolfram Sang , Andy Gross , Bjorn Andersson , Rob Herring , Krzysztof Kozlowski , Conor Dooley , linux-arm-msm@vger.kernel.org, linux-i2c@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Konrad Dybcio Subject: Re: [PATCH 3/3] i2c: qup: Vote for interconnect bandwidth to DRAM Message-ID: References: <20231128-i2c-qup-dvfs-v1-0-59a0e3039111@kernkonzept.com> <20231128-i2c-qup-dvfs-v1-3-59a0e3039111@kernkonzept.com> <5dr5ps4vb57xj2mfelgsxgoyrr3gg4ggwqggqchff6pda3ffsn@thxpg4h6kgel> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Wed, Feb 19, 2025 at 08:30:35PM +0100, Andi Shyti wrote: > On Wed, Feb 19, 2025 at 11:40:16AM +0100, Stephan Gerhold wrote: > > On Wed, Feb 19, 2025 at 12:02:06AM +0100, Andi Shyti wrote: > > > > > > sorry for the very late reply here. Just one question. > > > > > > > Thanks for bringing the patch back up after such a long time. I've been > > meaning to resend it, but never found the time to do so... :-) > > We have a long list of forgotten patches that belong to the far > past. I'm trying to revive them. > Thanks, this is much appreciated! > [...] > > > > @@ -1745,6 +1775,11 @@ static int qup_i2c_probe(struct platform_device *pdev) > > > > goto fail_dma; > > > > } > > > > qup->is_dma = true; > > > > + > > > > + qup->icc_path = devm_of_icc_get(&pdev->dev, NULL); > > > > + if (IS_ERR(qup->icc_path)) > > > > + return dev_err_probe(&pdev->dev, PTR_ERR(qup->icc_path), > > > > + "failed to get interconnect path\n"); > > > > > > Can we live without it if it fails? > > > > > > > of_icc_get() returns NULL if the interconnect API is disabled, or if > > "interconnects" is not defined in the device tree, so this is already > > handled. If "interconnects" is enabled and defined, I think we shouldn't > > ignore errors. Therefore, this should work as intended. > > yes, because qup_i2c_vote_bw() checks inside for NULL values. > > My idea was that: > > if (IS_ERR(...)) { > dev_warn(...) > qup->icc_path = NULL; > } > > and let things work. Anyway, if you want to keep it this way, > fine with me, I don't have a strong opinion, rather than a > preference to keep going. I would prefer to keep it the way it is. It's okay to omit the "interconnects" in the DT (either for old device trees, or because you don't define the "dmas" either). But if they are defined, we should not be ignoring errors. -EPROBE_DEFER definitely needs to be handled, but even for -EINVAL or similar it would be better to make it obvious in my opinion. None of the existing users should be affected, since no one defines "interconnects" at the moment. Thanks, Stephan