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 53C1E27F177 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=1754402714; cv=none; b=A4TRArPDhQ7WYEFlPkw/k5bHL1kCrY7fwWg7N+k43b3TYYM1YDtFgffKaPNUjYr3X+ssUTVmZCHssx7fJbPZRZWxi9i1UdPI/mYPnnNu/S3X5fe4g0bTfYCemMPU8PPk+aSOZvd43GJGrxUxitVrW/AaaETni9+KQnIeIIWpy6c= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754402714; c=relaxed/simple; bh=VwwDGPh3ds9A+awGPqwqcnNh6Gh4WFl0HqGJuWyvkkU=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=qIR9Qe+xqAa7pNbWucTLt0+xK4FrX8Fz4ffQWq26XIRPzcIlN9nrriHJwU5+jfpLyfGPh1BF51KSQ+dqxFGObufiQusn6qkSNLPwtscQsdujXtxqxYBo1PIBZYqGiSxwggNhjszj/a1kTXS26P5aFtGGekS2axLALmm6M1MMixo= 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 (m0279862.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.2/8.18.1.2) with ESMTP id 5759AUQe018139 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 489bek8ph2-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-7e802ee0d2aso14089185a.1 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=wx+rP3/sRcbl5qCWQEmiJtRowGnInyCOgFuhi11c0FvPtbTtZXibs9UdW44WYIbrzA 6S41gxxF4ktMt6uTqxx/du4N05WyKlVkDnVRUWWdLtW8aZ7Rg8yGRWYpHJbtosrmReOO GokbPZbQWXpMWoTbnhce5JlfRclH9704udm461Wr4lhmvI6BpU6BgV8Hx7OAAki7o3D/ 5o7TerivdzFMo7kWJefSiW1Uz2p90YJ77p+1O9w9C8l/Qj8gjYSiy4MgowrnFO9Dg38A 3EqgLgbm0SKbp8/bnmwA//WNGRhQ/scSyQSBCDGT9rt8ClwdqgztEfHbPtiTHfVdyGkE Cosw== X-Forwarded-Encrypted: i=1; AJvYcCVKOv1BLJt5At6J++IjR66aODd2hAmKu8z3ogukuvqBPZ3GBbZP+teUyrwX2s4jqolGO1pF1EI8pytG@vger.kernel.org X-Gm-Message-State: AOJu0Yxpm1hViGzMmiG3fl9XC1DJwaKi7WKTWb95Zn//tmKS6F7dxIfN jDRQjfliKDr2AmMFyKowXo7V7GqogcnxoONigO91lfdG5WgLhCSVCfHbBK/KVemnHLma5dTcBZb 3FTXZ3O1ApOOT/XA/530kDWFWirER5Ky8Wu/OpmMpjLK8fomFpIPoHimhitAl025a X-Gm-Gg: ASbGnctJ/lpE6A/uYclOQELCeBRm5G5GNhwyxW1ETWsv9HUjhtxWuNq0G9VBRdLLKvT BPPUMN/zUQR1x1bw8sCl6UHgMMxbTy9jio1oEyTF26ELxDrugMw7sgqkUR5igA7OUxRPODKH6yM Y/p1zL6IvBsktX2Ux38DHduAbp9J5bMX1azBUoHCk6aDU0+5Sb6svuyo0QATFjYJY1huNICK9aY jAdd6FYPXqNoSDyi7vS3PUoLmRVxR+L3bqLzwPG7vrEr4iEgVfIFgW8cM+501pHXGnBE3gRPYHz MbjlVNHSUWk+IvzD/U/IaBNsGP4ezN6fzUwtHXO3LfH7yQdFqd7CjiExx9lG8RYJOoMPNxj0ikD J6JA1+1tIbplSMmJijA== X-Received: by 2002:a05:620a:2a14:b0:7e7:fbbe:a193 with SMTP id af79cd13be357-7e7fbbea796mr612193785a.3.1754402706838; 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: devicetree@vger.kernel.org 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-Authority-Analysis: v=2.4 cv=M7tNKzws 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: AW1haW4tMjUwODA1MDEwMyBTYWx0ZWRfX1LbBXciMJQ4R JqOQ1SzELhJfRO1CmhTqnjmdUjj857K1scZaJYcJfp62TI5mBqQ0m+7v1nakRIl2itzVcKrkWpm Jb5d9EsO3K4AcQkWv/95MUT+RRDvpE8dIgFO3GcK68/wHPUTwoW5+2dAVxYAI1G4TqlYvw8J44q JH73sRg22ZPoJ+X4gtsvm3t4cu41Ot9LIqRO5YI176yCrkpXtVu6LYHNDsIRiAXBce7lYmKad4l iW53XjivjWaF3FEXf3Tk9Bn6YSsuAZDJJdsRDff0A1zQX5RFrdpLHqSRRI6HVkkdGRc/0Q/zaxO zQ96fB/BdtikKe4fSWT53NnAa2JCPHJER4mXENy2WwmgpwY7Reu6Zd152L68Vc/PnsxVNVDo8Sh pCe+FIza+dFBxR1ca+xUA4jLDIk/vRaYTlmcpcQGhk5zaGt9GsDnX9CzJdJgQrEYWvazUnD5 X-Proofpoint-ORIG-GUID: b7zcZYF86y1e2BV8Eo8aPPd9DtBlUljl X-Proofpoint-GUID: b7zcZYF86y1e2BV8Eo8aPPd9DtBlUljl 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 impostorscore=0 lowpriorityscore=0 priorityscore=1501 malwarescore=0 clxscore=1015 mlxlogscore=999 mlxscore=0 bulkscore=0 spamscore=0 suspectscore=0 phishscore=0 adultscore=0 classifier=spam authscore=0 authtc=n/a authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.19.0-2505280000 definitions=main-2508050103 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