From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f45.google.com (mail-ed1-f45.google.com [209.85.208.45]) (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 6252625EFB5 for ; Mon, 24 Mar 2025 13:29:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.45 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742822975; cv=none; b=KasxP8c0SQoNS/nolmGmG0jsop6B5KCmiu4ecU559sS6TirLTfean+oXs2L4lI1wyamoOlXWr5cQimbjeIT4LPSvxb9ezvFieperA9gF3SG1fJkMg68ZTuT/GMyT+A+s/S8bl/l5b7LrF7stk2g5rcNJdsIxenny9gqSmN1ORfM= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1742822975; c=relaxed/simple; bh=ToQFGwDDDePBcLj0Tl6e42Pcn77uyEQUZi/Zk7Ze/gY=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=akf4hSHS7KXdm/5JtMheCzO8g0tQvuausrZmiT73hHA4yqov/BD6WpPHbnhQsXl3JeGD06RshqvRoi+xSh7DZ3o17pk7dydxIydDxjKG9h4zGkI4v1hQQwjLhEAbh4eGt91FG+H+f9xBmto3ZtCuzuJpoebkMqrVKKQDwLDEfE0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=FBQ0IH7L; arc=none smtp.client-ip=209.85.208.45 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="FBQ0IH7L" Received: by mail-ed1-f45.google.com with SMTP id 4fb4d7f45d1cf-5e5dce099f4so5887455a12.1 for ; Mon, 24 Mar 2025 06:29:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1742822972; x=1743427772; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=wTTdxiEXTLI0RB3J4azgwISZcRhYfAuhdc+KevBi5vw=; b=FBQ0IH7Lil4t9nylwAriL9+AADyK4D7SWkKRjsG5xi2VT33mg92IGpF2cykNROCn8O lJ6+ME1dBfD8L3JFQlJAqqod+S0Jv3EegET70RAB8WqXIT0GKNzdQiAAcnFYZRY78khY Nav4G2mzWCl2GjoU64fyhBhGqRpn4svcfpH2CrFDXm/byrSDLnoNizxS6FS/ix0njbeI +BCrEqbckeqoybCo5eqE35sy9XNeIf22lEaoP/L3y4G1JNe9y628aD5cdgbkIfbEzST4 MnwkxEuVcO46+zVaOgJdpNb8Rv7uDxFzM1AJpg/+9WzshEim0fjjkvEwLWB/PYgw54/O XgCQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1742822972; x=1743427772; 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=wTTdxiEXTLI0RB3J4azgwISZcRhYfAuhdc+KevBi5vw=; b=gzyNntsUX883HqIPjsxiPNzt7y2/GrFG/b/OxxoRjftALUW3/cZCKUE/m+n9lvBJCx Ue7mypwSO0utfP3W42kD5o5rCKnOk+LKXYn45sHfyZ/ifRrzTjbL6Mx6rlgefmoVmpCa DUkhXr6John96gLDB+hRBxBf7wQvMAEwXgXlXu23shYO2cq48pbyJ2TbQwAm6dxujRJE 6/tt7Nfp1eGKJTdbjsvPIIjCQuEyooZtY3x92s+H5xkiNicMdwRB+KGdvvBH/nTwIsVE qTs+LF+56uscELCzPv70vlaLuXr4JG4HbMjrXvqxfTBmNkiGK8r5g0IjqUq4OIM3LkSY TWfw== X-Forwarded-Encrypted: i=1; AJvYcCUYejflhV5nGKka26EIYXaltHXsV0BGkWi6bOOn2VRm56txsEfu0pLBP3kqYwVeFV/nWrMy11/5lc9op6ra@vger.kernel.org X-Gm-Message-State: AOJu0YwfRD7DfQYA4htVG2Kbu2lt4+bcucZUQEFw1d1gqkg9s40c9Eu+ PosJUDRkBDvq/SjStT07lUQ5R1rWHjiGNrS32+QNHUUyvhsur96hPMY2Pm7xssM= X-Gm-Gg: ASbGncuYWwFsel7h0SoeW1L+bxUK4V+tLboQ015u9Q9Lj0cWA1bHzS+Fp3j0feASk9C YHqd1xmGBxUqMhhkNnYNN4fMvfmRiOr+CKqNvEB9hdAMSrKvxJ23G1BzYCWlPSzQHsR1HShxaSK NoT8z5ted6R99ffWcl2ck2l2kA0Pkc9mt6fbY6YlZTyQw5nMUQTsIpASBzdafWHM3MwfcxynsDm hCDJ33eZdD8q4n6Kv1N5PIaGDwhwhDtRbx4fEOu9P3fFs61iJJQpVPE/22cS2vFTL5Pm2fd9kvJ HZ3X5VnKCn0nraLUOoW017of43ah72A9FXXpNT+VI/4J5dLBikbZGJ1ZZ2sr8VA= X-Google-Smtp-Source: AGHT+IH+XEdLXnQX6nhvXQgyxnwEG52GJc4c2uceRNSxMxazFeWfoa7SI9NnS80gXy0yOwDc7i3hHg== X-Received: by 2002:a17:907:9482:b0:ac3:10e3:7fa5 with SMTP id a640c23a62f3a-ac3f20f51cbmr1074443466b.21.1742822971571; Mon, 24 Mar 2025 06:29:31 -0700 (PDT) Received: from [192.168.68.117] ([5.133.47.210]) by smtp.googlemail.com with ESMTPSA id a640c23a62f3a-ac3efbdc78esm682685766b.134.2025.03.24.06.29.30 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 24 Mar 2025 06:29:31 -0700 (PDT) Message-ID: <90f2bcab-d89f-40c0-9851-7ff25f2c1eda@linaro.org> Date: Mon, 24 Mar 2025 13:29:30 +0000 Precedence: bulk X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v2 2/3] misc: fastrpc: add support for gpdsp remoteproc To: Dmitry Baryshkov Cc: Ling Xu , andersson@kernel.org, konradybcio@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, amahesh@qti.qualcomm.com, arnd@arndb.de, gregkh@linuxfoundation.org, quic_kuiw@quicinc.com, quic_ekangupt@quicinc.com, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org References: <20250320091446.3647918-1-quic_lxu5@quicinc.com> <20250320091446.3647918-3-quic_lxu5@quicinc.com> <30bba296-8e6f-41ee-880e-2d5ecc8fe5a4@linaro.org> <5r72xrv5rtw6bemh5onygkroyasroviijlta4hvwgm5c5hzvax@3icylchlufu3> Content-Language: en-US From: Srinivas Kandagatla In-Reply-To: <5r72xrv5rtw6bemh5onygkroyasroviijlta4hvwgm5c5hzvax@3icylchlufu3> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 21/03/2025 14:07, Dmitry Baryshkov wrote: > On Thu, Mar 20, 2025 at 05:11:20PM +0000, Srinivas Kandagatla wrote: >> >> >> On 20/03/2025 09:14, Ling Xu wrote: >>> The fastrpc driver has support for 5 types of remoteprocs. There are >>> some products which support GPDSP remoteprocs. Add changes to support >>> GPDSP remoteprocs. >>> >>> Reviewed-by: Dmitry Baryshkov >>> Signed-off-by: Ling Xu >>> --- >>> drivers/misc/fastrpc.c | 10 ++++++++-- >>> 1 file changed, 8 insertions(+), 2 deletions(-) >>> >>> diff --git a/drivers/misc/fastrpc.c b/drivers/misc/fastrpc.c >>> index 7b7a22c91fe4..80aa554b3042 100644 >>> --- a/drivers/misc/fastrpc.c >>> +++ b/drivers/misc/fastrpc.c >>> @@ -28,7 +28,9 @@ >>> #define SDSP_DOMAIN_ID (2) >>> #define CDSP_DOMAIN_ID (3) >>> #define CDSP1_DOMAIN_ID (4) >>> -#define FASTRPC_DEV_MAX 5 /* adsp, mdsp, slpi, cdsp, cdsp1 */ >>> +#define GDSP0_DOMAIN_ID (5) >>> +#define GDSP1_DOMAIN_ID (6) >> >> We have already made the driver look silly here, Lets not add domain ids for >> each instance, which is not a scalable. >> >> Domain ids are strictly for a domain not each instance. >> >> >>> +#define FASTRPC_DEV_MAX 7 /* adsp, mdsp, slpi, cdsp, cdsp1, gdsp0, gdsp1 */ >>> #define FASTRPC_MAX_SESSIONS 14 >>> #define FASTRPC_MAX_VMIDS 16 >>> #define FASTRPC_ALIGN 128 >>> @@ -107,7 +109,9 @@ >>> #define miscdev_to_fdevice(d) container_of(d, struct fastrpc_device, miscdev) >>> static const char *domains[FASTRPC_DEV_MAX] = { "adsp", "mdsp", >>> - "sdsp", "cdsp", "cdsp1" }; >>> + "sdsp", "cdsp", >>> + "cdsp1", "gdsp0", >>> + "gdsp1" }; >>> struct fastrpc_phy_page { >>> u64 addr; /* physical address */ >>> u64 size; /* size of contiguous region */ >>> @@ -2338,6 +2342,8 @@ static int fastrpc_rpmsg_probe(struct rpmsg_device *rpdev) >>> break; >>> case CDSP_DOMAIN_ID: >>> case CDSP1_DOMAIN_ID: >>> + case GDSP0_DOMAIN_ID: >>> + case GDSP1_DOMAIN_ID: >>> data->unsigned_support = true; >>> /* Create both device nodes so that we can allow both Signed and Unsigned PD */ >>> err = fastrpc_device_register(rdev, data, true, domains[domain_id]); >> >> >> Can you try this patch: only compile tested. >> >> ---------------------------------->cut<--------------------------------------- >> From 3f8607557162e16673b26fa253d11cafdc4444cf Mon Sep 17 00:00:00 2001 >> From: Srinivas Kandagatla >> Date: Thu, 20 Mar 2025 17:07:05 +0000 >> Subject: [PATCH] misc: fastrpc: cleanup the domain names >> >> Currently the domain ids are added for each instance of domain, this is >> totally not scalable approch. >> >> Clean this mess and create domain ids for only domains not its >> instances. >> This patch also moves the domain ids to uapi header as this is required >> for FASTRPC_IOCTL_GET_DSP_INFO ioctl. >> >> Signed-off-by: Srinivas Kandagatla >> --- >> drivers/misc/fastrpc.c | 45 ++++++++++++++++++++----------------- >> include/uapi/misc/fastrpc.h | 7 ++++++ >> 2 files changed, 32 insertions(+), 20 deletions(-) >> > > >> diff --git a/include/uapi/misc/fastrpc.h b/include/uapi/misc/fastrpc.h >> index f33d914d8f46..89516abd258f 100644 >> --- a/include/uapi/misc/fastrpc.h >> +++ b/include/uapi/misc/fastrpc.h >> @@ -133,6 +133,13 @@ struct fastrpc_mem_unmap { >> __s32 reserved[5]; >> }; >> >> +#define ADSP_DOMAIN_ID (0) >> +#define MDSP_DOMAIN_ID (1) >> +#define SDSP_DOMAIN_ID (2) >> +#define CDSP_DOMAIN_ID (3) >> +#define GDSP_DOMAIN_ID (4) > > Why are you adding these to uAPI? How are they going to be used by the > userspace? > >> + >> +#define FASTRPC_DOMAIN_MAX 4 >> struct fastrpc_ioctl_capability { >> __u32 domain; here, in domain value of fastrpc_ioctl_capability. >> __u32 attribute_id; >> -- >> 2.25.1 >> >> >> ---------------------------------->cut<--------------------------------------- >