From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (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 DA5C1340A6B; Mon, 30 Mar 2026 15:23:39 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774884219; cv=none; b=ZXpamrL8X5CMcm1I8fmqvL/6cqCJjkRZKhEUB3s8WxE5ZUU9JukfUlk7mCLphjSAadXWMpiXu/frjxIcsJ/Jw8zZy7pi4ORw4WuswLjSupKMrQiG9qmFl6IOKe6IfKI+sYt/A3JKy5WQ8b9cZZwaNfcv/eikCQy9taXINy5qmPw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1774884219; c=relaxed/simple; bh=VULu542ABKvcNdY+7D9CdlmIg9J7r7UGrQBVKdeLiMY=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=S6VBv0ciPrxK+mce2QBQN0RW+AxmPPvvBA8gIQzwg17FCzhyh/pidBB+k85WDjyVw6X2F6ZtBeAFzNAOymQIjhblv621OJFQ1QxBhMJ0PrNgTIWFW1+IpkgFgLS7goaa0fdChVwdMCMvFJ16HROXrzTV0nxt1Zjag2LrgFmlbeY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b=prqTKGyK; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="prqTKGyK" Received: by smtp.kernel.org (Postfix) with ESMTPSA id B8D43C4CEF7; Mon, 30 Mar 2026 15:23:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1774884219; bh=VULu542ABKvcNdY+7D9CdlmIg9J7r7UGrQBVKdeLiMY=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=prqTKGyKEq6bTMdFHWFNCYT/lKNg2YpV5IlAQwiIXHGhkCSZWIL3OCS5TpKgH8Wgf XLZGHVoliDtSLgDYHhKQnVtc/Ls03TOKKqU3Wj0wwKUQ+VxQ+VJI8OfqUiv2vkOOoS NGntgeCI2idZ0VBEPbG4PiWV7R2YM6GaCwinftD2PCvkrpIEKa/O7mmO6oDOOUeczc u2gLeC/R3Z/DayowwA12TxHzD3ZYvH/WTn55qNZDSFKjoN5z9+K6EVIxsgGwxRPQQg UVpIf1muwbocTSqJ9u66iCI8wQ7oNQdixAVTsqwFWT07WzHnYTy+SraeSgsmugk7yJ Fs0w5WGFHfJRA== Date: Mon, 30 Mar 2026 16:23:33 +0100 From: Conor Dooley To: Krzysztof Kozlowski Cc: Hui Min Mina Chou , pjw@kernel.org, palmer@dabbelt.com, aou@eecs.berkeley.edu, alex@ghiti.fr, geert+renesas@glider.be, prabhakar.mahadev-lad.rj@bp.renesas.com, magnus.damm@gmail.com, ben717@andestech.com, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, jonathan.cameron@huawei.com, devicetree@vger.kernel.org, linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org, linux-renesas-soc@vger.kernel.org, tim609@andestech.com, alex749@andestech.com, az70021@gmail.com Subject: Re: [PATCH 2/7] cache: andes_llcache: refactor initialization and cache operations Message-ID: <20260330-rug-coeditor-82f88ac95e7f@spud> References: <20260330102724.1012470-1-minachou@andestech.com> <20260330102724.1012470-3-minachou@andestech.com> <8c01c910-21b4-4a16-98e4-197c20883d23@kernel.org> Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha512; protocol="application/pgp-signature"; boundary="XSEEsXFnQ9xlKiil" Content-Disposition: inline In-Reply-To: <8c01c910-21b4-4a16-98e4-197c20883d23@kernel.org> --XSEEsXFnQ9xlKiil Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Mar 30, 2026 at 03:02:11PM +0200, Krzysztof Kozlowski wrote: > On 30/03/2026 12:27, Hui Min Mina Chou wrote: > > This patch cleans up the Andes LLC cache driver: > > - improved error handling in andes_cache_init() by using goto labels > > - updated andes_dma_cache_inv/wback() to check for !size instead of > > start =3D=3D end > > - cache-line-size mismatch from an error to a warning > > - Use ALIGN and ALIGN_DOWN helpers instead of the alignment logic in > > andes_dma_cache_inv() and andes_dma_cache_wback(). >=20 > Please read submitting patches document. One thing per commit with > proper rationale WHY you are doing this. Applies to multiple patches too. Anything here with a bullet list needs to be several patches. --XSEEsXFnQ9xlKiil Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYKAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCacqVcgAKCRB4tDGHoIJi 0vBhAQCwaaV8c1kS1cbuCi/Pa4fZ8f4Oq1pfsKXH8bMxDxOimgEApnKQp+tY6hDU KPlDV7uSgwfAYilwkjTxwTIG7cKTkQI= =gqdu -----END PGP SIGNATURE----- --XSEEsXFnQ9xlKiil--