From nobody Mon Jun 8 20:41:14 2026 Received: from mail-qt1-f179.google.com (mail-qt1-f179.google.com [209.85.160.179]) (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 59164393DF0 for ; Tue, 26 May 2026 20:16:12 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.160.179 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779826574; cv=none; b=dUzfntqOAdjTLf+cUOiF2RZMr10BM9Iw4elstwOeGstMFKjFWekzitdcRvQ8UY5UE5o5CDXMB5UEkFwyc74Pl3eIbPO3cVe8se4NnoeV/VQqaziy48Ix3qFvQqh/ucrkvwnA4D6ZzWoP3rraJuu3cepxsaQkoBV8uXGPTUZA/N8= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1779826574; c=relaxed/simple; bh=MzyTjID+CWb/MWHklcG9BXLprF41TsBCJfGI/gConv8=; h=From:To:Cc:Subject:Date:Message-ID:MIME-Version; b=SmGLAeCReFRt5k/weTurST/bILcSQUG9AZ/8tCPHDSaAwBvV8ijl2xSR3DsIbOrzpsw9kTPA0xuulqpNitoFpP3QWu+Ea99yjiT721NPC6qLXTYmQsVjIR+Y/pdjXaNL8YJSU94T1MDB+OGVslMwoc/FDnD91Ctigrtm+sK1RHY= 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=DUdFhHFj; arc=none smtp.client-ip=209.85.160.179 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="DUdFhHFj" Received: by mail-qt1-f179.google.com with SMTP id d75a77b69052e-5148cbdea08so91664691cf.2 for ; Tue, 26 May 2026 13:16:12 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1779826571; x=1780431371; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=ku/vxDrBu7T4TGqb3O0QJz0UZRxcqf3H+SXkaiCq2AE=; b=DUdFhHFjDGpUyuPlF6VKN9qOXlnBqBa5NcLWJYlFCJQvdunvi5c+10MSDkhGmAgLbW X6GpIfNvVRyp+axHvrfnglG1hldkExhYeweu3sRk4UBhPnESfg+gAm90KVVGblKXCTUC QWFf3zFbo/HJKBNIqNG6oEt63cb+aJTa3WceVVx/YHT2y1jJcdAFXfajAFPQ+GsI7SJ/ b5ortALFpnIRsjANgkkAMvERXVHnhkaqDWSQ4OyopTY/CTOsI0bTZa2CqdNrA+4tZNJ0 zi6K8+37j1eJ8lO73W9bvfRQdDwuoXzunpuldv71/w5Td0nLIaxkshDG/ailnSYIl4BO w3vg== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1779826571; x=1780431371; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to; bh=ku/vxDrBu7T4TGqb3O0QJz0UZRxcqf3H+SXkaiCq2AE=; b=A7kfE2Wbxzd43T4fnEjQZHF8gcNZs5YhiX4BaaQJXRcFTDxpI5NHq0MN2jELIOiBTS /Q1XKAtqk+Mtf5xZZJnpCYqHiZWmzra0S6FQjD5zafI6Yqtn4EfFTKfFXvWsdr/sbRWs FGvwA7ZepkmGyosriYbvKGBvi/H0QCND/uI/pHv4kePVXW5Ote3WEW8uwwv+gmjUEWY0 bwOOatTRCp8puEhlc5ZVpvdRUcCUKmWeyf240Oy9jB9Ko4DW5rYPbifLLzNBZaXN3aEe UbDiVo7UNu4lADPmHZkX+HwlQbzPq4zved84c8xObr863meOWXDv7BvdZXF6kz8eSXXR NM/A== X-Forwarded-Encrypted: i=1; AFNElJ/AsuKarOWUB/cF/iXVRWV3xg5lbBPE0Q4WMMS0SkjapsQO/43jxe3C42RNblLJr57i5mzrZn03eezHZXY=@vger.kernel.org X-Gm-Message-State: AOJu0YwJnV1f807p2YF0V48K+fDJL7pzkyHiQPLswLpAfxhfVIcLDPZ9 7p2iQ9mS7n+NP7CKYXVEdJONpASkSaEFVK0JIxFHLjg6+6o7ohtM5eTD X-Gm-Gg: Acq92OGQpsMBot33sEmWskDMhrntZG//8FPkz78PhYEc/IuaIdbnXHKcBhqdOCKmAxM Z6tYlW8lYWNjHm5DUrh6aI0lQ8SbQwMqIHZHpwRcqGuMVh/2LzD31+b64Zpn1/5io74dx9z9pWz oZ7Y7h4h2/KQOEokBIMpqRnHw5s9EfKNLNcPIQn0LSIB7JINeHDkg6hCMs/lLYxD9lQN4gkVw0u cnYPh0I4Fn1kFzvk8tl2R02GVEq2gckHfjWFDBAdx/9nRSicE5dlYSTjodjChCQrNzufNT/Pl7r +zIigVj7suVfSBi8P4o/UW6ONNtn+n6vdCbcPPCYz8BWgiC/LuyiVKdmwNPssis+xX1iIursoVP DDNhZXlscSPioGkTLtOVrH601hWH0GdgnWscynw/3J6b0gngUQsejqqlx1S6j91RFITDZkXwYgr AMijwi1vinfxey5bfCT3lY2g9cHfVPgoev516ar+WxQMeX9LGHsivQ8WMDDBbS0T6WlyVfUweiF cpdoOL5Flq8VqlY7HLQ5zMbW0GQyPYoJxznh25tLuPz7Q== X-Received: by 2002:a05:622a:4a09:b0:50f:bd79:2642 with SMTP id d75a77b69052e-516d463bd85mr282690521cf.48.1779826571221; Tue, 26 May 2026 13:16:11 -0700 (PDT) Received: from ryzen ([2601:644:8000:5b5d:7285:c2ff:fe45:8a32]) by smtp.gmail.com with ESMTPSA id d75a77b69052e-51706adc9cfsm25433541cf.19.2026.05.26.13.16.09 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 26 May 2026 13:16:10 -0700 (PDT) From: Rosen Penev To: dmaengine@vger.kernel.org Cc: Linus Walleij , Vinod Koul , Frank Li , linux-arm-kernel@lists.infradead.org (moderated list:ARM/NOMADIK/Ux500 ARCHITECTURES), linux-kernel@vger.kernel.org (open list) Subject: [PATCH] dmaengine: ste_dma40: turn d40_base phy_chans into a flexible array Date: Tue, 26 May 2026 13:15:52 -0700 Message-ID: <20260526201552.13376-1-rosenp@gmail.com> X-Mailer: git-send-email 2.54.0 Precedence: bulk X-Mailing-List: linux-kernel@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" Convert the separately-offset phy_chans pointer to a C99 flexible array member at the end of struct d40_base, and switch the allocation to struct_size(). The log_chans and memcpy_chans slots continue to live in the same allocation immediately after phy_chans, indexed via base->log_chans. This removes the hand-rolled pointer fixup that recomputed phy_chans from base + ALIGN(sizeof(struct d40_base), 4). Assisted-by: Claude:Opus-4.7 Signed-off-by: Rosen Penev --- drivers/dma/ste_dma40.c | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/dma/ste_dma40.c b/drivers/dma/ste_dma40.c index 9b803c0aec25..d3e3c4cd43f1 100644 --- a/drivers/dma/ste_dma40.c +++ b/drivers/dma/ste_dma40.c @@ -602,7 +602,6 @@ struct d40_base { struct dma_device dma_both; struct dma_device dma_slave; struct dma_device dma_memcpy; - struct d40_chan *phy_chans; struct d40_chan *log_chans; struct d40_chan **lookup_log_chans; struct d40_chan **lookup_phy_chans; @@ -621,6 +620,7 @@ struct d40_base { u32 *regs_interrupt; u16 gcc_pwr_off_mask; struct d40_gen_dmac gen_dmac; + struct d40_chan phy_chans[]; }; =20 static struct device *chan2dev(struct d40_chan *d40c) @@ -3128,6 +3128,7 @@ static int __init d40_hw_detect_init(struct platform_= device *pdev, struct clk *clk; void __iomem *virtbase; struct d40_base *base; + size_t alloc_size; int num_log_chans; int num_phy_chans; int num_memcpy_chans; @@ -3197,10 +3198,9 @@ static int __init d40_hw_detect_init(struct platform= _device *pdev, "hardware rev: %d with %d physical and %d logical channels\n", rev, num_phy_chans, num_log_chans); =20 - base =3D devm_kzalloc(dev, - ALIGN(sizeof(struct d40_base), 4) + - (num_phy_chans + num_log_chans + num_memcpy_chans) * - sizeof(struct d40_chan), GFP_KERNEL); + alloc_size =3D struct_size(base, phy_chans, num_phy_chans); + alloc_size +=3D sizeof(*base->log_chans) * (num_log_chans + num_memcpy_ch= ans); + base =3D devm_kzalloc(dev, alloc_size, GFP_KERNEL); =20 if (!base) return -ENOMEM; @@ -3213,7 +3213,6 @@ static int __init d40_hw_detect_init(struct platform_= device *pdev, base->virtbase =3D virtbase; base->plat_data =3D plat_data; base->dev =3D dev; - base->phy_chans =3D ((void *)base) + ALIGN(sizeof(struct d40_base), 4); base->log_chans =3D &base->phy_chans[num_phy_chans]; =20 if (base->plat_data->num_of_phy_chans =3D=3D 14) { --=20 2.54.0