From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wm1-f42.google.com (mail-wm1-f42.google.com [209.85.128.42]) (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 70BCE3876C6 for ; Sat, 7 Mar 2026 10:37:29 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.42 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772879850; cv=none; b=rckNWhwmPwXay/tiaixFhEG2Tvm5b9oOQU8R55zzKR1SuoRqNm4vMP+gqoA3Ei0As5AV4iK/n9Sfv4tH2HuFeEy4rm3447ON8/AU102x94s24KtN/61nF2SvjDWPvkexfvCVbFA2RXiiP3entIkbvb6pkyc2c03ax9UUdhZd36E= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1772879850; c=relaxed/simple; bh=6KqKy/Dd60nk2QrYoe3ZsdhugKxz6bXkdh0cmEC7594=; h=Message-ID:Date:From:To:Cc:Subject:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=EAbwNeO46GMXKUETDEsyetytPAml45iP86JXMdbIzQXU6cZLOvI3DAwZoR/2Ji3Rs6m7CNgPZHE9ctGuRNXUhyqFsmpvvJ8g14tXgCiSx3IzqacLGFf3+vMJjgPsD6ggMjcuaTQWxBgPN1Njflw3qciTsp20uTdEFZw91qBU2ek= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=dxnMXs27; arc=none smtp.client-ip=209.85.128.42 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="dxnMXs27" Received: by mail-wm1-f42.google.com with SMTP id 5b1f17b1804b1-483abed83b6so84015335e9.0 for ; Sat, 07 Mar 2026 02:37:29 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1772879848; x=1773484648; darn=vger.kernel.org; h=in-reply-to:content-disposition:mime-version:references:subject:cc :to:from:date:message-id:from:to:cc:subject:date:message-id:reply-to; bh=f5dRvU5hSjZG/d9NEVJAuQDH8Oh+ysj/HM9E+z96h0A=; b=dxnMXs27+cJiLSkIIQpyQglmgzTN5pKyM/TS7X51h6v0hVmGGQvgtB0J8M4ZQXSOTi dPkWrCx56zurdI22bqOovgrkn6EF9YY2WVyAHK/pF2FVxfCz235RIcVf1o8VIb0VxJRM d1vHWsB5d0AZCfGGxFDRki19Bl6lVOXRu3SrpobJe1muLs/IQUS9KGgMWBZI0x6YxSnE KQO/GAabxJApMFdHgL0XMYRfy6/1AZ1E0kH+sxLltAZm+Ht/jQSMA5zJEqRNEnr6FPpp hZVpRAbEw8PTMfPrhBp6IQqdqZXOkzvaZq7B+pKTuqmvsRq41ZE++7aEXqJQ1mdKVJrD IXLQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1772879848; x=1773484648; h=in-reply-to:content-disposition:mime-version:references:subject:cc :to:from:date:message-id:x-gm-gg:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=f5dRvU5hSjZG/d9NEVJAuQDH8Oh+ysj/HM9E+z96h0A=; b=Wt9aFpu5+dBG7yVpTiK3X3k56cC9zN2Yh/wLTk8+ejhTmJMJvcTTg3szSomoIUoIsQ k8Ozha2/ZD73HJwUyy7LV7dj1LeN8rbOTNQfkWEXB2FtPwn9Qan4oUifiXWuSEo2mmwR fAlhPLK2dZehJalwkrcs/rCdZB1Lg6WegV5nFKQDNk5FX5pnxD7jJSXcZ8RIkpLb9J82 GXqRecCoHY3XxHX3LfYIOLZ3pxCx2/qXzC/3/F8hdgUqgbVwdL0JQvBiO40Pg+H5nvxO PbVMlLUvOg0WuDoP8gA9cD7HoStcOm5ui08jDadyuZbRRXFVgyyeARJL29j/sl/bW5wP s62A== X-Forwarded-Encrypted: i=1; AJvYcCX8Smm3wdnERi6lsgBfhDk41LtHLAjNJVi3XYLQ+iaKZZIoXGYkOHR7X+eiMJngkFOkvy82rd3Y22vDV7E=@vger.kernel.org X-Gm-Message-State: AOJu0YyxcYSIUC3yFtTnQanNH9RQYuxNWmjqA+J+pm1HrwqraqhgX6I8 DD4jXwMcbY8fhrPF+DjrcebQdKyvFOFwCCei2fYHDLBTgU1706NFzaDf X-Gm-Gg: ATEYQzxfjQFTVhRpIr9R0S35LU3ZKcMJD3cqjeg/zuVpeoC4v931W+66zEnhfYW/g3L Gxk3aVRiTaTplOjHsGhWxNdRBUlGv8EgeWO9usIneZNWrvbR0UGckTmEnyAtbwh+hhSEA47vmDV 4oXMXTF+uRXRNh4+ieHt5HKMV7PboPsA6roxheSS3ve2xCFA0Yzrcmy5QgJEjHJsn+3y/oqZI+0 xQvYSmo6oqKTKQFlrlhU95r6aTZlJxyeqK6/i4j6ERPjZREO8hlLyQymg2EEaYQtX4Oz3OT/w54 aFaZD8FfgoUyZ7SCZ6SawL21CZOkJm/k8rIRGPgv4XN/OKT69519ufdZ6wuJRDFNdMTBf1b+6iL sHEb1bql/V13yWEia0FMAobEyXepIZysku8klIrH8+6pX9PZ+FprIaAFaFGtXtxaK3omBLRqOK0 AD9UqC4HKIrtyDAt2gLfBRuxyTN7Uzxe0G6XkLB02THMS9MFmacGDSHg== X-Received: by 2002:a05:600c:1d04:b0:47e:e7e5:ff32 with SMTP id 5b1f17b1804b1-48526983292mr76044105e9.34.1772879847564; Sat, 07 Mar 2026 02:37:27 -0800 (PST) Received: from Ansuel-XPS. (93-34-88-122.ip49.fastwebnet.it. [93.34.88.122]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-485246ece8bsm55380715e9.4.2026.03.07.02.37.25 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 07 Mar 2026 02:37:26 -0800 (PST) Message-ID: <69abffe6.050a0220.18164c.58d6@mx.google.com> X-Google-Original-Message-ID: Date: Sat, 7 Mar 2026 11:37:23 +0100 From: Christian Marangi To: Krzysztof Kozlowski Cc: Vinod Koul , Neil Armstrong , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Lorenzo Bianconi , linux-arm-kernel@lists.infradead.org, linux-phy@lists.infradead.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v6 4/4] phy: airoha: Add support for Airoha AN7581 USB PHY References: <20260306190156.22297-1-ansuelsmth@gmail.com> <20260306190156.22297-5-ansuelsmth@gmail.com> <20260307-otter-of-imminent-merriment-d3475d@quoll> Precedence: bulk X-Mailing-List: linux-kernel@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: <20260307-otter-of-imminent-merriment-d3475d@quoll> On Sat, Mar 07, 2026 at 11:32:57AM +0100, Krzysztof Kozlowski wrote: > On Fri, Mar 06, 2026 at 08:01:53PM +0100, Christian Marangi wrote: > > + for (index = 0; index < AIROHA_PHY_USB_MAX; index++) { > > + enum an7581_usb_phy_instance_type phy_type; > > + struct an7581_usb_phy_instance *instance; > > + > > + switch (index) { > > + case AIROHA_PHY_USB2: > > + phy_type = PHY_TYPE_USB2; > > + break; > > + case AIROHA_PHY_USB3: > > + phy_type = PHY_TYPE_USB3; > > + break; > > + } > > + > > + if (phy_type == PHY_TYPE_USB3) { > > + ret = of_property_read_u32(dev->of_node, "airoha,usb3-serdes", > > + &priv->serdes_port); > > + if (ret) > > + return dev_err_probe(dev, ret, "missing serdes line for USB 3.0\n"); > > + > > + priv->scu = syscon_regmap_lookup_by_compatible("airoha,en7581-scu"); > > Nope, you need phandle to express proper device links. > > Don't sprinkle compatible lookups for new code which does not need to > keep things backwards compatible. How do you manage device links > without phandle? How do you manage device probe ordering? > Hi, the phandle to SCU was present in old implementation but later dropped as it was said that it wouldn't describe the HW. I will readd as airoha,scu. Is it ok for you? (I actually prefer phandle than hardcoding compatible as it makes things more descriptive and prevent all kind of problem in the future, so I'm more than happy than using that) -- Ansuel