From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-alma10-1.taild15c8.ts.net [100.103.45.18]) (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 A890247B413; Tue, 16 Jun 2026 17:29:04 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=100.103.45.18 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781630946; cv=none; b=EuFBk8xyW/OC9pX9ThJAvOqjYO8ylQDQLRl3R0NmP+YdCNiHIYIF4nRaxOWsgeDmUtscfLibYRsCM7OHRAb6GB5Iqr15rMKPAl+p6n6LlFyEosMU8qAg7UiSw5lCWbc1aaj1mFgLSsbIId+56hgc40motvXoPaYjY1RULaq/5Nk= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1781630946; c=relaxed/simple; bh=NaxzfdP9ayheFK9mT097DNbSlCN1Kw1VdldDufukB7s=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=SMzjW+CK3p0StlkCH58/S+ab/VDZ2XWyH/TMF4/8WUFomfWcWPOBDuDOy6pbOksv8UnNa3DsD07GifdXVpAH5tYy6HB6pxz3hTvvbq5cyche6HWtcHIk+R8gjaSS0NHGnFxBRwycXBf8Pn3gXtTrycfaEj0TjrvyBDFZUix0xyo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=JLG6mbuV; arc=none smtp.client-ip=100.103.45.18 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="JLG6mbuV" Received: by smtp.kernel.org (Postfix) with ESMTPSA id E72061F00A3D; Tue, 16 Jun 2026 17:29:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=kernel.org; s=k20260515; t=1781630944; bh=tPiaiG+PobOwjWjYLwLmVH9nP3FU1Y5GnnLskZEHqPc=; h=Date:From:To:Cc:Subject:References:In-Reply-To; b=JLG6mbuV6OmZVfx/OkWLfxHdd+7JHDrEDOgeYNZ1Dy+OlDKHR8RGz7N+HKIWkkRAT gRjNidRDXowAGyOucAgF2jKCzqpBl9lu8xoCCC/Owwjy0gEeCkUqJPpAH3I7hSO22U AzyC+YOVwi1ls/vyQvLMu+yD4jVKd65D2Hrx8v46uT9LpVyUonLCusfQw5ebb+QyfL 8BRNCdv93pBzGfQPom+7KWQq01lOvO/rQ6AoFmYmXA7SKCF/4YUI7rx2Tek7dczALY qzWNwLnjEu6ZSoAH5KZOt5ODjkIVgXOdnf+rzxzDP3jGRBo1Ftvg9fPDpb5ZfXXZo/ kxDv6UArbNFTg== Date: Tue, 16 Jun 2026 12:28:52 -0500 From: "Rob Herring (Arm)" To: Tapio Reijonen Cc: linux-serial@vger.kernel.org, Conor Dooley , Hugo Villeneuve , Jiri Slaby , linux-kernel@vger.kernel.org, Greg Kroah-Hartman , devicetree@vger.kernel.org, Krzysztof Kozlowski Subject: Re: [PATCH v3 2/3] dt-bindings: serial: maxim,max310x: describe per-channel rs485 subnodes Message-ID: <178163093205.2469869.2417966435525371534.robh@kernel.org> References: <20260615-b4-max310x-rs485-dt-v3-0-7e79f064bdd7@vaisala.com> <20260615-b4-max310x-rs485-dt-v3-2-7e79f064bdd7@vaisala.com> Precedence: bulk X-Mailing-List: linux-serial@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: <20260615-b4-max310x-rs485-dt-v3-2-7e79f064bdd7@vaisala.com> On Mon, 15 Jun 2026 10:27:36 +0000, Tapio Reijonen wrote: > The MAX310x is a family of one- (max3107, max3108), two- (max3109) and > four-channel (max14830) UARTs. The binding pulls in > /schemas/serial/rs485.yaml at the chip level, describing a single set of > RS-485 properties - enough for the single-channel parts, but a > multi-channel chip can wire RS-485 differently on each channel. > > Split the binding per compatible: > > - single-channel parts (max3107, max3108): the chip node is itself the > serial port and carries the RS-485 properties, as before; > > - multi-channel parts (max3109, max14830): the chip node is only a > container and is no longer a serial node; each channel is a "serial@N" > subnode that carries the standard serial.yaml/rs485.yaml properties > (and may host a serial slave device). max3109 has channels 0-1, > max14830 has 0-3. > > This avoids a chip node that is simultaneously a serial node and the > parent of serial nodes. The driver still reads chip-level RS-485 for > single-channel and legacy device trees, so existing users are unaffected. > > Signed-off-by: Tapio Reijonen > --- > .../devicetree/bindings/serial/maxim,max310x.yaml | 92 +++++++++++++++++++++- > 1 file changed, 90 insertions(+), 2 deletions(-) > Reviewed-by: Rob Herring (Arm)