From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.131]) (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 5233827F171 for ; Tue, 5 Aug 2025 14:05:11 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754402713; cv=none; b=MGbi88mVgCEUZA3rF+tTGaIgTQ/N7zxF7OFEvatkSHskH4vszwmmeyGUzs9NSdCXKEBS4NwmqmmATwykPs/cCuESnVTAO7D36rnUlE+0AvIZ8T2f8t8um5yFR2EtMbF3JYLYgPraNp+ojVTYL6UYq8dOxx0LIgigNGUGlxSnpRs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754402713; c=relaxed/simple; bh=VwwDGPh3ds9A+awGPqwqcnNh6Gh4WFl0HqGJuWyvkkU=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=bGkkhO1Jg0S9CuhbSPJHyfAbFg21oBWk0Na5ypLCRhapEYhXlYueJUnpjibja1rEUbpNRE6+DhpT+b9NIK4xlJOWIQp1UxPtzicdwrvIdY3/7e3RohYgLtnLvkwsK7yTI8KPqoJUmITDWjotuEsFLCyrZ0B2b4R7UshecvCrZeY= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=JswbVhN9; arc=none smtp.client-ip=205.220.168.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="JswbVhN9" Received: from pps.filterd (m0279865.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 5757Hsnm003472 for ; Tue, 5 Aug 2025 14:05:10 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= FUcLnureUr1s/7YsNTnvW5Csv2CIZeOY1lQ56p5/Cj8=; b=JswbVhN9EiO/3bXB KFFT2FK5zmrdDsAM9Z4O6z5zh7QsngVwmJQyJVPLIRQZy9tXuLlXFzx5q8QeK24Z UuASYCo8lC5r5kPQilhktpk99gMDFXxhv/dT55dfFGKBCQXaaStS6SqslNBHs+Pk 6lpCnGjbmxM9hWGEUq1bYfANmhL1k2BCrodCT2z4jODPS9GZvl5Ka+gigTeBZa4j hrLEdMvf9GRA+didrRj1V7EI/VkbiYXs7SbD328oyEfQp3RkEeUY0iTi8A1CYyOi HGtkEWfS1GX9/TR+jdWzEglaT9vngedtZgPPmRClZ0L8g80N1lRCZfR6JAq2vIX7 73cJtw== Received: from mail-qk1-f199.google.com (mail-qk1-f199.google.com [209.85.222.199]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 48b3j3tm7t-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128 verify=NOT) for ; Tue, 05 Aug 2025 14:05:09 +0000 (GMT) Received: by mail-qk1-f199.google.com with SMTP id af79cd13be357-7e80acb8a6fso11220685a.2 for ; Tue, 05 Aug 2025 07:05:09 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1754402707; x=1755007507; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=FUcLnureUr1s/7YsNTnvW5Csv2CIZeOY1lQ56p5/Cj8=; b=lVkYxVeldpujcDPgu/rAU/+W1zNz+erBi9uOvITMssTXAS6nmNLg9TS/oc8yrNViCj rs3O0R8vxeeFoZpCed/Uy40rRXQz0LL3uSmBMaMoqOuvQoXL/upgSjZXi00XflCYsXhq bpZ2A2YJyhngvhT640UnhcboICJJXn79pzHNWQZ/usuNG1veDo6HSVdgj3XCke9KalBn xLQQ14XyV5uxZZ49ikPDlZB7O3eD56La8BnrGNz9IYX+uDeE5TdlhOCO5OR1OfR7go7c hRxGb7MFW15ZF73CiWwe2FE3OdrXx8tmFx7ExHWmAMfMs4sr83m0FQxD2rVhqip4oMTj MGng== X-Forwarded-Encrypted: i=1; AJvYcCU4dpdv83SLQZ/aifg+CNA/cB1x8KqrYSD2A9951WsAxGNlgQRWdesExXwpMTnz2nEoo+3Gh+0Trap7Lw==@lists.linux.dev X-Gm-Message-State: AOJu0Yyj0zSsf04WEX+Wygnuuofe0nnOGwobUYgZoXb+FbXwIt6sU2IQ xiO4FNKqB2163h+Mu9vUTsXOG++7iV1csyGoiXoEcNDZ0yBQZVYrQ1yPdFPUowWOXprs6YQVLsq IizfnYP6JcDWNHzxkN5lCo/iq7+I9qVczVIpPtTaPxvgnT24dEbOmP3JoFJeMYZhjNQ== X-Gm-Gg: ASbGncst8+NHsFVnz+ukRawTIuGs4Os7cr4oyXixw4YBNk/l3iUNOC8Ug3Pd8SdBhkQ R6BFEjJ9Yum9EeIsVk4iq110sS4h5MZFygAc10VnFiiceg8pIrXN3uCIOmcYCn3BsqO/Z8LivLB cDNfEl74EdgxjwDQR3GzwufWmAYdGa5ooxDHahHf6XDgrvO3Z24P7YY26J7Okf3EVftT4fwvlu1 L0bs4D0nW+5aNJF4rQ8zFhRJ9sJaQj6btNSbMy8gXjPhxUwXOjz//gQCqbxvxtfSvD/vdhZkD85 DAsHvRFR+BLgsXeBicJxYi4PeOJ1laAHD+JjXWeSr9tWbWWPVJk1IHNsx57yzJI/tQKOEwuZ/JV lO8nHTGoqMhzoe+WsOA== X-Received: by 2002:a05:620a:2a14:b0:7e7:fbbe:a193 with SMTP id af79cd13be357-7e7fbbea796mr612196785a.3.1754402706921; Tue, 05 Aug 2025 07:05:06 -0700 (PDT) X-Google-Smtp-Source: AGHT+IGu0CtTCnSeovH7oZZIpOsR6SihrS+IeJBeYEWKx2ZbtzZuol650T7aTGGDwlvwPzoxkQZS7A== X-Received: by 2002:a05:620a:2a14:b0:7e7:fbbe:a193 with SMTP id af79cd13be357-7e7fbbea796mr612187885a.3.1754402706081; Tue, 05 Aug 2025 07:05:06 -0700 (PDT) Received: from [192.168.43.16] (078088045245.garwolin.vectranet.pl. [78.88.45.245]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-615a8f17829sm8316020a12.19.2025.08.05.07.04.59 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 05 Aug 2025 07:05:05 -0700 (PDT) Message-ID: <96cf6022-ef69-4749-88b0-e18a07bb8427@oss.qualcomm.com> Date: Tue, 5 Aug 2025 16:04:58 +0200 Precedence: bulk X-Mailing-List: linux-sunxi@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RFC 2/6] dmaengine: Make of_dma_request_slave_channel pass a cookie to of_xlate To: Laurent Pinchart Cc: Frank Li , Konrad Dybcio , Vinod Koul , Sven Peter , Janne Grunau , Alyssa Rosenzweig , Neal Gompa , Ludovic Desroches , Florian Fainelli , Broadcom internal kernel review list , Ray Jui , Scott Branden , Paul Cercueil , Eugeniy Paltsev , Viresh Kumar , Andy Shevchenko , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Taichi Sugaya , Takao Orito , =?UTF-8?Q?Andreas_F=C3=A4rber?= , Manivannan Sadhasivam , Daniel Mack , Haojian Zhuang , Robert Jarzmik , Geert Uytterhoeven , Magnus Damm , Patrice Chotard , Linus Walleij , =?UTF-8?Q?Am=C3=A9lie_Delaunay?= , Maxime Coquelin , Alexandre Torgue , Chen-Yu Tsai , Jernej Skrabec , Samuel Holland , Laxman Dewangan , Jon Hunter , Thierry Reding , Peter Ujfalusi , Kunihiko Hayashi , Masami Hiramatsu , Michal Simek , Rob Herring , Saravana Kannan , =?UTF-8?Q?Martin_Povi=C5=A1er?= , Liam Girdwood , Mark Brown , Jaroslav Kysela , Takashi Iwai , Kuninori Morimoto , Mukesh Kumar Savaliya , Viken Dadhaniya , Andi Shyti , Krzysztof Kozlowski , Conor Dooley , Bjorn Andersson , Marijn Suijten , dmaengine@vger.kernel.org, linux-kernel@vger.kernel.org, asahi@lists.linux.dev, linux-arm-kernel@lists.infradead.org, linux-rpi-kernel@lists.infradead.org, linux-mips@vger.kernel.org, imx@lists.linux.dev, linux-actions@lists.infradead.org, linux-arm-msm@vger.kernel.org, linux-renesas-soc@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-sunxi@lists.linux.dev, linux-tegra@vger.kernel.org, devicetree@vger.kernel.org, linux-sound@vger.kernel.org, linux-i2c@vger.kernel.org, linux-spi@vger.kernel.org References: <20250730-topic-dma_genise_cookie-v1-0-b505c1238f9f@oss.qualcomm.com> <20250730-topic-dma_genise_cookie-v1-2-b505c1238f9f@oss.qualcomm.com> <20250730180417.GC21430@pendragon.ideasonboard.com> <20250801120007.GB4906@pendragon.ideasonboard.com> <0c2cc631-21fd-41fd-9293-fd86dd09a2d2@oss.qualcomm.com> <20250804235140.GB12087@pendragon.ideasonboard.com> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <20250804235140.GB12087@pendragon.ideasonboard.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-GUID: FF792ap3HLbRRyLFfNyv9An6nq1HOy74 X-Authority-Analysis: v=2.4 cv=TZ+WtQQh c=1 sm=1 tr=0 ts=68920f95 cx=c_pps a=HLyN3IcIa5EE8TELMZ618Q==:117 a=FpWmc02/iXfjRdCD7H54yg==:17 a=IkcTkHD0fZMA:10 a=2OwXVqhp2XgA:10 a=EUspDBNiAAAA:8 a=IUwAV-ZVHCOG9dU_XxAA:9 a=QEXdDO2ut3YA:10 a=bTQJ7kPSJx9SKPbeHEYW:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUwODA1MDEwMiBTYWx0ZWRfXw5h2cQVWoGEK j+21SA5iFrtGBDfualbkj64NuotJ9KFzHat6kAMhJwAb4nPy+qXqGeOwfCtxfQRRpWq7cQTL0Pb dav3DYG/eq9n0Et95xHGUZJs22kkFpFaF0AhSQwwn2WtTGBvIeeoAYmcUCpOhw+h0OIn+nigcHW uP5Wk6klKXXEvEWhzwPEIMxrGlpmsm9Wm4VwkoWPPUjgitaDHirwr7IeVqqCKniZ0jnjenex4O6 hgedvzB1PvmtXiI0JkvbVIHHE1/yxlXFrgX/3nHRp6iKDU5thleKZjX+JX7XgIohH4Kr4B7HgjW IxJ2nNEIDOhHwNtGnY6Um4trWjqcaH44Tv8DtlBI1GJsvS0eiXEGvEnp4OZaHMXZh/hVfisGytL puU2ImCOjzDd8xccm9h3u737XL0dnzec7O3Tb4veXZ4gnUz8HotmBPOkU0IoJwHwozpZXHCl X-Proofpoint-ORIG-GUID: FF792ap3HLbRRyLFfNyv9An6nq1HOy74 X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1099,Hydra:6.1.9,FMLib:17.12.80.40 definitions=2025-08-05_04,2025-08-04_01,2025-03-28_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 clxscore=1015 priorityscore=1501 mlxscore=0 suspectscore=0 impostorscore=0 spamscore=0 adultscore=0 lowpriorityscore=0 malwarescore=0 mlxlogscore=999 phishscore=0 bulkscore=0 classifier=spam authscore=0 authtc=n/a authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.19.0-2505280000 definitions=main-2508050102 On 8/5/25 1:51 AM, Laurent Pinchart wrote: > On Sat, Aug 02, 2025 at 02:37:54PM +0200, Konrad Dybcio wrote: >> On 8/1/25 2:00 PM, Laurent Pinchart wrote: >>> On Wed, Jul 30, 2025 at 02:37:54PM -0400, Frank Li wrote: >>>> On Wed, Jul 30, 2025 at 09:04:17PM +0300, Laurent Pinchart wrote: >>>>> On Wed, Jul 30, 2025 at 12:39:43PM -0400, Frank Li wrote: >>>>>> On Wed, Jul 30, 2025 at 11:33:29AM +0200, Konrad Dybcio wrote: >>>>>>> From: Konrad Dybcio >>>>>>> >>>>>>> The DMA subsystem attempts to make it theoretically possible to pair >>>>>>> any DMA block with any user. While that's convenient from a >>>>>>> codebase sanity perspective, some blocks are more intertwined. >>>>>>> >>>>>>> One such case is the Qualcomm GENI, where each wrapper contains a >>>>>>> number of Serial Engine instances, each one of which can be programmed >>>>>>> to support a different protocol (such as I2C, I3C, SPI, UART, etc.). >>>>>>> >>>>>>> The GPI DMA it's designed together with, needs to receive the ID of the >>>>>>> protocol that's in use, to adjust its behavior accordingly. Currently, >>>>>>> that's done through passing that ID through device tree, with each >>>>>>> Serial Engine expressed NUM_PROTOCOL times, resulting in terrible >>>>>>> dt-bindings that are full of useless copypasta. >>>>>>> >>>>>>> In a step to cut down on that, let the DMA user give the engine driver >>>>>>> a hint at request time. >>>>>>> >>>>>>> Signed-off-by: Konrad Dybcio >>>>>>> --- [...] >> So.. do you folks want me to alter the patch in any way? > > I think the runtime PM issue is orthogonal to the problem this series > addresses. It can be addressed separately. > > That being said, I'm not a big fan of passing a void pointer to > .of_xlate() to carry device-specific information, in a device-specific > format. This seems prone to mismatch between clients and DMA engines. > .of_xlate() also seems the wrong place to do this. It would be cleaner > if we could use another operation, such as dmaengine_slave_config() for > instance. Right, I noticed that the drivers in question already apparently do the very same thing (passing data through a voidptr), inside slave_config (via the peripheral_config/_size fields).. But that doesn't solve the issue this series is trying to tackle, since every DMA consumer driver passes its own structure (or NULL). My immediate idea is to add yet another field, but that's lame. The other immediate idea would be to put an `u8 variant` at the start of all of these structs ("we have tagged enums at home"), but that's lame *and* dangerous. Ideas? Konrad