From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-wr1-f47.google.com (mail-wr1-f47.google.com [209.85.221.47]) (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 D23D73EBF16 for ; Mon, 6 Jul 2026 22:13:44 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.221.47 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783376026; cv=none; b=TndcvfZeCrEgiu2qPxxqu2KrjabLCV+Z4bxfT8BnhFMImaJQWVXgd5WkdDdiTdNa2FWyMUGt6ZJIP92DUdnpQEsPhpyljpuJWQGw/9b8R+J42nungENwM2fCLD5Y2CyN/GWAkcigRuh4LYGsLXgJKCoOx0Qzh0gNOWWqLhsGZ+o= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1783376026; c=relaxed/simple; bh=m3kaKDuOinnzWNHwerx5pFFY+Eflp1o8yKhTqlHb8xw=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=sBQyeaVNBEWfFAxJxHmt/il0dFqEvQZ2U19Ad2foV22uWUo2q+s8Ukx0KPb1I3hmxMoaaFOlYmCMmUbZU3Cy+Z2vF5AphcdaSbg2sYeNCxWUWXF6Z0J+J3caAJnCJ6zDi53Y2dl5s4LWw51QCeLC9LkEwwNB4ozB1tOENRZzCnY= 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=leLpuzmy; arc=none smtp.client-ip=209.85.221.47 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="leLpuzmy" Received: by mail-wr1-f47.google.com with SMTP id ffacd0b85a97d-476a130c138so3658862f8f.0 for ; Mon, 06 Jul 2026 15:13:44 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1783376023; x=1783980823; darn=vger.kernel.org; h=content-transfer-encoding:content-type: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 :content-type; bh=3VUxjcvyG7o7DixEmYl5pZcfD8TvxQHqFhtXA36U/44=; b=leLpuzmy1MCQFfJQdZjSGo15S981y1FEu5PVYg8YXgkUIoQ/7LepkVujXJJoQEDxQ3 kU/hB7Jt8HtgcK2tnysM6bJnBKPyeJGjGTLqtl+PTliQ9XruvHi/h45L58ISLcY2zwR5 fOQ/DsVFFiuyTcVqC2zTT3kBu6GdXFayg0Em4vaPCUhgxUmg2FdjCwPR5WLbb+GU9MAe QPBzP8wNGQLhrCbkN40ju3KgLgD1Sh7uCkzYo7wF+59QVm7EbIqFA8MGJ89j3mrvx30W fUyjMyPhJ59Lov7ghR0dwlP+DfwtwKanbQDUdyY0DurTEa1dgHaxmRcnQEY3V4NHDZIY xCTA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1783376023; x=1783980823; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=3VUxjcvyG7o7DixEmYl5pZcfD8TvxQHqFhtXA36U/44=; b=D4RydmwaDNjcOecsPIe7To50b2ZfXUhL9+1rxhV2kYTKkYSwSdRl5UB9f4PvRrfWQd 1xcVsnReGvMXjMjCvhhM6R7aow1Oy3QXVadFmbanmDm3effzkhK74Kxf4VU7hRPxumGV VacwOFxQ71YmrIm21AsLnSuYufJ7rQq1UoaVh4kmVrA8fUj7zgGYA+e1kzvUzAwQuX/l gRM/Z5FQVysTwgWSht0woSOtjsf8GqhfFRheFPuqSk7hpzx+SFTThQ0PbTUp18ZeKG6+ 6sfQNUsqLD2kWmORI8HRFFjvl5sBoteS0mmuDUcBwk/QP2h/9GpG7DUN6zI2JM+Zx3eY TFEQ== X-Forwarded-Encrypted: i=1; AHgh+Rq7xlCTnhcEAu6yITwfw8rPxFhYex/eKs0mu9p4kYqIxreyBL8y+F8DioC/N4Apt5tIGrJGtBn4/792mgs=@vger.kernel.org X-Gm-Message-State: AOJu0YzuCyLgzn4LQ9JKWRxHVqm5mOmW3E6+X9sv/y119a1KTLsb2si0 k2p/LBpVjj6Aqs0nep5bQEbapvp2MrjBSvtrh3OZPCaPmBjt1DO1B74= X-Gm-Gg: AfdE7clfXeOoBXi9q0claXJNrBOHx1xYV115kpjqhdlJgHXrIzophTh+VaMpAaN/j+t BiuU9TK2lwczDz/BGsOs7XLqdI7QyxAmybJGjFV0pW/geQSGkMWRR/tSCsdx8E96EgTXWmTZvdh 5hggn1z7jrFmbo0tCqc1EasNTFqzgTA/E0Esi1OFOfcumPQTkzTcWns2oOX8sBHSkPz7LbNipJC GnH9bgmMfZipi5NDdbfaQbJmEPdN+mOIEJVCDo6YfsIbDSBZkf8KWB4FDkBBmhx51xbGXjHuWaa FpTAux++0Dgr5hWBIwhZR7HyLnom/gZh9ZvL1o5dO0OZUeZ+xuCjeeWnUs0Wdd33o2pVSiuqgep AXHvnwfIAjZRcZWHxl95HEh+ogTFKUj4lMjP/icT64GmvEaJuwMIYl6z/88egbLhFtUnFOoHf6e 5l/sCOZ504RYRA7+J9opoMwicie8Uk9K4NHGWaRWpdb0Yab4910Wfj8w1bqCh6Fpo+IBTpDg== X-Received: by 2002:a5d:45ce:0:b0:476:c622:d41f with SMTP id ffacd0b85a97d-47de66dd5e9mr1597139f8f.47.1783376022918; Mon, 06 Jul 2026 15:13:42 -0700 (PDT) Received: from ?IPV6:2001:b07:2ec:601d:4b26:1672:75c7:805a? ([2001:b07:2ec:601d:4b26:1672:75c7:805a]) by smtp.gmail.com with ESMTPSA id ffacd0b85a97d-47aa039afacsm26722332f8f.19.2026.07.06.15.13.41 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 06 Jul 2026 15:13:42 -0700 (PDT) Message-ID: Date: Tue, 7 Jul 2026 00:13:40 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RFC 10/13] drm/msm: initialize LPAC ring To: Akhil P Oommen , Konrad Dybcio , Rob Clark , Will Deacon , Robin Murphy , Joerg Roedel , Sean Paul , Konrad Dybcio , Dmitry Baryshkov , Abhinav Kumar , Jessica Zhang , Marijn Suijten , David Airlie , Simona Vetter , Bjorn Andersson , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann Cc: iommu@lists.linux.dev, linux-arm-msm@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org, devicetree@vger.kernel.org References: <20260705-descriptive-name-lpac-upstream-v1-0-01d50c3e0c99@gmail.com> <20260705-descriptive-name-lpac-upstream-v1-10-01d50c3e0c99@gmail.com> <93b78299-866f-4a3c-9423-a990d9a7d4cd@oss.qualcomm.com> <64a919ab-8b36-45fe-8da8-5eaaf4d55359@gmail.com> <2a556ca0-d276-4c56-a44e-9d8b92a3af6b@oss.qualcomm.com> Content-Language: en-US From: Anna Maniscalco In-Reply-To: <2a556ca0-d276-4c56-a44e-9d8b92a3af6b@oss.qualcomm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit On 7/6/26 11:56 PM, Akhil P Oommen wrote: > On 7/6/2026 4:37 PM, Anna Maniscalco wrote: >> On 7/6/26 10:44 AM, Konrad Dybcio wrote: >>> On 7/5/26 9:13 PM, Anna Maniscalco wrote: >>>> Program all the register needed to start up LPAC and send initialization >>>> sequence. >>>> >>>> Additionally, now that the LPAC ring is live, make the code aware of it. >>>> >>>> Signed-off-by: Anna Maniscalco >>>> --- >>> I believe some/all of these writes need to be guarded by if (has_lpac), >>> as many GPUs lack these registers and there may be side effects on >>> others >> That's correct yeah. I'm not sure what has_lpac should be though. >> >> Something we add to catalog? Or checking if it is a gen that has LPAC? > We can introduce a new QUIRK (ADRENO_QUIRK_LPAC?). > >>> >>> [...] >>> >>>>           a6xx_gpu->shadow = msm_gem_kernel_new(gpu->dev, >>>> -                              sizeof(u32) * gpu->nr_rings, >>>> +                              sizeof(u32) * (gpu->nr_rings + !!gpu- >>>>> lpac_rb), >>> Should we rename nr_rings to something like nr_XXX_rings >>> where XXX is some uarch-generic (i.e. not SQE because we also >>> support pre-a6xx) name for the main number cruncher? >> Qualcom calls it GC so nr_gc_rings perhaps? > How about nr_gfx_rings as those rings can handle graphics workload? That could be more intuitive yeah > > -Akhil. > >>> Konrad >> >> Best regards, Best regards, -- Anna Maniscalco