From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from gloria.sntech.de (gloria.sntech.de [185.11.138.130]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B10F13EAC74; Wed, 20 May 2026 14:21:37 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.11.138.130 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779286901; cv=none; b=PQoftHDZ+8QuT91T0013rmQYPt42DAPYAtLL6H3da5wLVE+JZDZyuMh1gZVsbTK1+jYSmB54JEpfocteXGbRH0p4a6X+3wbBlc84WOIy+RukvepPRA+BMajjaGSGrSsteJyOlb5iP/66MljSNZ02/YGiIMjcEEOjWHOKJqzCDtk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779286901; c=relaxed/simple; bh=GDgiypVIwA/PQvdX7RyrKY/+g5n+NMwhEKvQxfGp7xE=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=Bqz6Ept4mHi8WX3+KQs08o/aTDNKTqqDUWD+MrY4CROasMNN97xoDrGAasUfUh36pV92zAYH6moKnnRRsbKpxix8xMkwgpCYjeqfu+g+31OHu2s9nsQwrkS3EJ9O4TVecMEpo1Ac4Obod4LPLSTIu+y5htkF8p8esSBerubbEmc= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=sntech.de; spf=pass smtp.mailfrom=sntech.de; dkim=pass (2048-bit key) header.d=sntech.de header.i=@sntech.de header.b=qKOs1rSH; arc=none smtp.client-ip=185.11.138.130 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=sntech.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=sntech.de Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=sntech.de header.i=@sntech.de header.b="qKOs1rSH" DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=sntech.de; s=gloria202408; h=Content-Type:Content-Transfer-Encoding:MIME-Version: References:In-Reply-To:Message-ID:Date:Subject:Cc:To:From:Reply-To; bh=GDgiypVIwA/PQvdX7RyrKY/+g5n+NMwhEKvQxfGp7xE=; b=qKOs1rSHUfFyQjQ2QU90qLmsEG E4rO+KqFDvZdMnNVkhT2xM9SqMD/CMUMADckAN8zJzIgHiiRGrfYvDLfVYwvte7zRaDJPjinr9dN7 W41mCtDYikWfvkdAm1kyTt5Ns2JB93TsQ4vLGQVdLdw8UDj+b5s76SDCSf59Tvpostexn9puA2nth 1WSXxPyw1cawkSEQwtUp6XpmyN9Z6+X49zn1Jkj0dxFSvWFTrY7Udx+xmwDa/G9o8rsguZyCHlSmu oG53XHOV5DRGeBYTKUVFMdMkD5aBSLI4rVQszy9k+eZF8tHtjXWW6HmPSv6IqLIL+WyoyuC5IVfQI V1VVfT8w==; From: Heiko Stuebner To: linux-phy@lists.infradead.org, Vladimir Oltean Cc: Vinod Koul , Neil Armstrong , dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org, linux-arm-kernel@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-can@vger.kernel.org, linux-gpio@vger.kernel.org, linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org, linux-media@vger.kernel.org, linux-pci@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-riscv@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-samsung-soc@vger.kernel.org, linux-scsi@vger.kernel.org, linux-sunxi@lists.linux.dev, linux-tegra@vger.kernel.org, linux-usb@vger.kernel.org, netdev@vger.kernel.org, spacemit@lists.linux.dev, UNGLinuxDriver@microchip.com, Sandy Huang , Andy Yan , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter Subject: Re: [PATCH v8 phy-next 15/31] drm/rockchip: dw_hdmi: avoid direct dereference of phy->dev.of_node Date: Wed, 20 May 2026 16:21:24 +0200 Message-ID: <3758596.1xdlsreqCQ@phil> In-Reply-To: <20260505100523.1922388-16-vladimir.oltean@nxp.com> References: <20260505100523.1922388-1-vladimir.oltean@nxp.com> <20260505100523.1922388-16-vladimir.oltean@nxp.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" Hi Vladimir, Am Dienstag, 5. Mai 2026, 12:05:07 Mitteleurop=C3=A4ische Sommerzeit schrie= b Vladimir Oltean: > The dw_hdmi-rockchip driver validates pixel clock rates against the > HDMI PHY's internal clock provider on certain SoCs like RK3328. > This is currently achieved by dereferencing hdmi->phy->dev.of_node > to obtain the provider node, which violates the Generic PHY API's > encapsulation (the goal is for struct phy to be an opaque pointer > with a hidden definition, to be interacted with only using API > functions or NULL pointer checks, for the case where optional variants > of phy_get() did not find a PHY). >=20 > Refactor dw_hdmi_rockchip_bind() to perform a manual phandle lookup > on the "hdmi" PHY index within the controller's DT node. This provides > a parallel path to the clock provider's OF node without relying on the > internal structure of the struct phy handle. >=20 > Signed-off-by: Vladimir Oltean > Reviewed-by: Heiko Stueber there is now already more stuff depending on this change [0], and the change itself also is sort of independent of the whole phy-series. And somehow this series itself sadly hasn't gotten much review yet. So would you be ok with me just picking this one patch for the drm-misc-tree? Thanks Heiko [0] https://lore.kernel.org/dri-devel/20260518193748.2482823-1-jonas@kwiboo= =2Ese/