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 Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 2C503CA5FFD for ; Tue, 6 Oct 2026 03:27:20 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=Sd6/pyfGyG/n7CfU1w9rhrQ2BedMbB6MOvnx5xheKLQ=; b=3j4f2gWS6wO1WZ E4tJfusaE6n5iJPeGmct14LOygL7+K+x1E75WOMLsMlbYnANF5UDldlQdZR1gK4MD0WOoMmyDoN57 9asSFr96C4c+UBoGtReH0TIjticolfifGDQAcgiWtrvgIPqBhf9zPAiu0am6g1PGZff9bb8+Y1KOm i8LvR5LiiUHGQ3L/wlD0v2/DHuHKEim2W/QsoWDiVzB+FDy40aoeUpcpG6Z6xxPAFYEqh1QguLZoc Ys/UpNq+E7Qmcmn6Z8VkXtuJcNR1yN9JbEtaEqUX07GcE8VTZyqa/G8d+s4Y5TtZ1QVAbnictuGUI UP7gQq0heOuZ8THyEP3g==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.99.1 #2 (Red Hat Linux)) id 1xDvpi-0000000HYqY-3wMF; Tue, 06 Oct 2026 03:27:14 +0000 Received: from tor.source.kernel.org ([2600:3c04:e001:324:0:1991:8:25]) by bombadil.infradead.org with esmtps (Exim 4.99.1 #2 (Red Hat Linux)) id 1xDvph-0000000HYq8-16M3; Tue, 06 Oct 2026 03:27:13 +0000 Received: from smtp.kernel.org (quasi.space.kernel.org [100.103.45.18]) by tor.source.kernel.org (Postfix) with ESMTP id 5AC04601F1; Tue, 6 Oct 2026 03:27:11 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id D12061F000FF; Tue, 6 Oct 2026 03:27:10 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1791257231; bh=BjT4eIz3gx9cpb3bB/A7BLnqv/sJrakTzUqKFsTtq7g=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=agzj5webOc/vjusOJXsIWgm1dpyC1w68XusjyTZ22WS+ZxS35CYb6KU++CaagoguV 6bEtjDOplbw0XofFS/QqmaVzjlTqrv4MqAceJZAmiXbw18hwuPli07voe6E+eiwDMC 2yr0OC6rbEMR9FAiQEqX5Azp6Y3FvSMkIpAuVcxfPHs+wFYui5WfTyuynN6oP6wFyw sgChu/npvlZDM1olQrNozKtIthl0xYUv+IgSG+GkCEHABK5sAOXojyI+Mans1aV+wh pZ3txDOEAyFQK4iue2cbvg4Yl8oLPlSGTigoek9n3TtDYcd5KKqDaYjpykneD9dZ55 ytnPfxGF1seHQ== Date: Mon, 5 Oct 2026 22:27:10 -0500 From: Rob Herring To: Coia Prant Cc: Vinod Koul , Neil Armstrong , Manivannan Sadhasivam , Krzysztof Kozlowski , Conor Dooley , Heiko Stuebner , linux-phy@lists.infradead.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v11 0/2] phy: rockchip: naneng-combphy: add SGMII MAC selection for RK3568 Message-ID: <20261006032710.GA3539706-robh@kernel.org> References: <20261005225944.1171574-1-coiaprant@gmail.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20261005225944.1171574-1-coiaprant@gmail.com> X-BeenThere: linux-rockchip@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Upstream kernel work for Rockchip platforms List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "Linux-rockchip" Errors-To: linux-rockchip-bounces+linux-rockchip=archiver.kernel.org@lists.infradead.org On Tue, Oct 06, 2026 at 06:59:42AM +0800, Coia Prant wrote: > Changes since v10: > > - Drop the top-level "default: 1" for "rockchip,sgmii-mac-sel". > dtschema injects top-level defaults into every matching node, and the > injected value conflicts with the "else" branch that forbids the > property on non-RK3568 variants, breaking dtbs_check on those > platforms. The driver already treats an absent property as 1 (GMAC1), > so behavior is unchanged. What? That's not right. 'default' has no effect on the validation. Please add it back. If the property doesn't apply to some variant, then default doesn't matter. Rob _______________________________________________ Linux-rockchip mailing list Linux-rockchip@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-rockchip