From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=185.132.180.163 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1793603381; b=1uL6SHgk9O41/k/fUiFqqpnlukG0H2w3ku7QAuRsgWRl9H9Kjv+PedJLvSOMff1+gkPN tWcq9QPsEzV4O39oTlapjA/uRSMl4nOVVDN6pWSNsEZD9O0Ml+Yci0VD6t9QVARZ57nUI Sb5FM5ud5j+Gkht+GOT/SJe/VFU4p3PKc5OmGF8g2yADP6b3q3Ezg+bjuFMsa88IZ8Rtx /BgMoumQVzohro3+oUGke04SbzV0/WeptZA1CpuBq7wpXYDabSyd2GiHI1/eTerhllcMj okT06nH5MulT9XxQB8PdNOTNP1brpIaPY8/MVsRkRTzOpK4LJHgF1p4E/fO1igYwh9g== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1793603381; h=DMARC-Filter:Received:DKIM-Signature:Received:Received:From:Date: Subject:MIME-Version:Content-Type:Content-Transfer-Encoding: Message-ID:References:In-Reply-To:To:CC:X-Mailer: X-Developer-Signature:X-Developer-Key:X-Originating-IP: X-Proofpoint-Spam-Details-Enc:X-Proofpoint-ORIG-GUID: X-Proofpoint-GUID:X-Authority-Analysis:X-Proofpoint-Spam-Info; bh=nl+ZTQ29g0X1AjvjZkhWS+s9s3MSc8Ju6CfIIVOuOlY=; b=lQODGM/w2tXdGYDLfH0081DBFuNZ279tomP65o75oKX5TETYrpFhlLxyfJmGqtGUdfsU T7df450EmldewJeglDmo3T3lbI1MSG1DMP3OEMjD9/bUfH3e1hex7TF/gLUoeVRGGEf4I eCDbXIhKPSX1E/Jh5VBIQwT5BFzFy97w/oieXWu682sZ7bkWFCqm2P+/Q7LkMLH4y92kD tCcbaNvA3fPg/fS5emhC4VQ2FAdBuUSyyHwIFRakejy3Zs6UVTL8xHJ1eVu4rM3LyNd9n 4+SiU2RiJlpWtxAck/pmFQTt2tDCVe1ZemCCg/EslrPxjmJLbzElpQL9izg6ZUhtoFg== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass header.from=imgtec.com; spf=pass smtp.mailfrom=imgtec.com; dkim=pass header.d=imgtec.com header.i=@imgtec.com header.a=rsa-sha256 header.s=dk201812 header.b=H7A48dhc; arc=none smtp.remote-ip=185.132.180.163 Received: from mx07-00376f01.pphosted.com (mx07-00376f01.pphosted.com [185.132.180.163]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 01E7460A260F for ; Mon, 2 Nov 2026 07:09:40 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.2 smtp.subspace.kernel.org 01E7460A260F Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=imgtec.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=imgtec.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key, unprotected) header.d=imgtec.com header.i=@imgtec.com header.a=rsa-sha256 header.s=dk201812 header.b=H7A48dhc Received: from pps.filterd (m0168889.ppops.net [127.0.0.1]) by mx07-00376f01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 6A26GB1t3838432; Mon, 2 Nov 2026 07:09:08 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=imgtec.com; h=cc :content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=dk201812; bh=n l+ZTQ29g0X1AjvjZkhWS+s9s3MSc8Ju6CfIIVOuOlY=; b=H7A48dhcqNiDHznMT KCkvY2KMT4l4ZUo8QfcXn1Z5nhovzwFuDg+zuHcC9pEGWqTSh20c11IsOlDU8RfF qdNw4J+qIqcwPSmQlsBREMw7MA77LxHIcBShgGnCQzD170htRQIXAI8S4tcUtN2X 8p+smyFokbcML9QRG47ly6LDSlEaAtMT2E38Tkt2HrpMeR7QWx7PCNEIIcdWmv82 0lHrqMUj5FEFXuMa7my8v+Nw/43mWDndsASwksL6Rjzl7rV3NIQ1xlDB72aUf1Io jvu6VHoWoZWEOMGHg6fpjmidua7muwEDf2lzi/OElsEEyTfOdCLlB3DNj3QZfHUz SOXSw== Received: from hhmail01.hh.imgtec.org (83-244-153-141.cust-83.exponential-e.net [83.244.153.141]) by mx07-00376f01.pphosted.com (PPS) with ESMTPS id 4hn99whb6d-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Mon, 02 Nov 2026 07:09:08 +0000 (GMT) Received: from [127.0.1.1] (172.25.5.26) by HHMAIL01.hh.imgtec.org (10.100.10.19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.2562.56; Mon, 2 Nov 2026 07:09:06 +0000 From: Alexandru Dadu Date: Mon, 2 Nov 2026 09:08:36 +0200 Subject: [PATCH v5 4/4] drm/imagination: Simplify MIPS PTE definitions X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Message-ID: <20261102-b4-simplify-mips-pte-defs-v5-4-d5bcf7d95a1a@imgtec.com> References: <20261102-b4-simplify-mips-pte-defs-v5-0-d5bcf7d95a1a@imgtec.com> In-Reply-To: <20261102-b4-simplify-mips-pte-defs-v5-0-d5bcf7d95a1a@imgtec.com> To: Alessio Belle , Luigi Santivetti , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Paul Walmsley , "Palmer Dabbelt" , Albert Ou , "Alexandre Ghiti" CC: , , , Alexandru Dadu , Matt Coster X-Mailer: b4 0.16.0 X-Developer-Signature: v=1; a=ed25519-sha256; t=1793603340; l=7675; i=alexandru.dadu@imgtec.com; s=20260304; h=from:subject:message-id; bh=gZ+1jGk3nIcTdTnoMuSwuwb14tHqVwWtI5PUAt8LcEU=; b=Xj0bJG0mCxxhvE6MfVgEwq8cXCdH6k11llQHAVL8erjBGhV9IFzOPSAxlaG9upg45fURBVib3 mLpcGetr6deAr7f1ONknmtKgWka71Zk+puYK13Uzvhp9Rxbut6sGOLS X-Developer-Key: i=alexandru.dadu@imgtec.com; a=ed25519; pk=FU07SyNrYGwhahqSxbWEuzyXlUh47xBXvLvOR7UA6+U= X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYxMTAyMDAyOCBTYWx0ZWRfX1wJkqFFTdFMV 2xKOODyuwifc6CG7GlJHUFopBoLiN9jEt0XHS1Z4VXAgDfa4bNQ9JJ77wT08fjyYdCsnQSrGvPX kKzGRGHNovplzKj4HHLkl/n5HAZ3oisvlJKZQu3axaeXI6Ooki25q87shzWRs32302nuYgoubu8 ORGYr+BVJcMOaK4gLE2odAjYKnejgzu/FSIccAV+wTeYG8h6W32v4dw/7AUG3jNj2Y/PjrgmzUy A7mQtfnZQcRC96hx/xZ50hvn7SyQvCg3GIuejf3fa94G6fVZj2h1H7/QFqiDAJMoSxW96Y5DxUy 3a8PjRY/SNXt5NgZBXH44VKY3/CcRO99TWzz/dU3Zg0vkxPQjvkQ9U7Ff8UXejzhWDRrn7XqntJ Y36Q8YSw+ALLYxtFQOcEtW5I3FHriB2719rutKUb48ljlKe3LPvSxdDYcWoh17AmTT72PkIP5aU x/+gptS5icAfexwzFUQ== X-Proofpoint-ORIG-GUID: v6W_rk-a9e57fiX5oAXVcfGUwsDR6Pbv X-Proofpoint-GUID: v6W_rk-a9e57fiX5oAXVcfGUwsDR6Pbv X-Authority-Analysis: v=2.4 cv=BZlNQbt2 c=1 sm=1 tr=0 ts=6ae83714 cx=c_pps a=AKOq//PuzOIrVTIF9yBwbA==:117 a=AKOq//PuzOIrVTIF9yBwbA==:17 a=h_3jC4m0AyQA:10 a=IkcTkHD0fZMA:10 a=zJRp6_cAMcgA:10 a=VkNPw1HP01LnGYTKEx00:22 a=kQ-hrUj2-E3RCbRHssb7:22 a=7RYWX5rxfSByPNLylY2M:22 a=r_1tXGB3AAAA:8 a=vkN0XxrHRKDbMJCO9W8A:9 a=QEXdDO2ut3YA:10 a=t8nPyN_e6usw4ciXM-Pk:22 X-Proofpoint-Spam-Info: AW1haW4tMjYxMTAyMDAyOCBTYWx0ZWRfXwIQemD5UQrEp xC2MGB5N0GOZsalFHfYe9FOmPsEhAntqmZcdginDUlsh4YVmEFasB2qA9NqTSo/aGlLEg9C6ps3 FIQLHtQDnVRkm8W7H8JsBbEECDQmR24= From: Matt Coster Use BIT() and GENMASK() in the definitions of the MIPS PTE format, and remove some unused definitions that add no value. Signed-off-by: Matt Coster Signed-off-by: Alexandru Dadu Reviewed-by: Alessio Belle --- drivers/gpu/drm/imagination/pvr_fw_mips_internal.h | 74 +++++++--------------- drivers/gpu/drm/imagination/pvr_vm_mips.c | 25 ++++---- 2 files changed, 35 insertions(+), 64 deletions(-) diff --git a/drivers/gpu/drm/imagination/pvr_fw_mips_internal.h b/drivers/gpu/drm/imagination/pvr_fw_mips_internal.h index f13829e63e1c..cbbf3982179a 100644 --- a/drivers/gpu/drm/imagination/pvr_fw_mips_internal.h +++ b/drivers/gpu/drm/imagination/pvr_fw_mips_internal.h @@ -39,63 +39,33 @@ #define PVR_MIPSFW_MAX_NUM_PAGETABLE_PAGES (4) /* Total number of TLB entries. */ #define PVR_MIPSFW_NUMBER_OF_TLB_ENTRIES (16) -/* "Uncached" caching policy. */ -#define PVR_MIPSFW_UNCACHED_CACHE_POLICY (2) -/* "Write-back write-allocate" caching policy. */ -#define PVR_MIPSFW_WRITEBACK_CACHE_POLICY (3) -/* "Write-through no write-allocate" caching policy. */ -#define PVR_MIPSFW_WRITETHROUGH_CACHE_POLICY (1) + +enum pvr_fw_mips_cache_policy { + /* "Write-through no write-allocate" caching policy. */ + PVR_MIPSFW_CACHE_POLICY_WRITETHROUGH = 1, + /* "Uncached" caching policy. */ + PVR_MIPSFW_CACHE_POLICY_UNCACHED = 2, + /* "Write-back write-allocate" caching policy. */ + PVR_MIPSFW_CACHE_POLICY_WRITEBACK = 3, +}; + /* Cached policy used by MIPS in case of physical bus on 32 bit. */ -#define PVR_MIPSFW_CACHED_POLICY (PVR_MIPSFW_WRITEBACK_CACHE_POLICY) +#define PVR_MIPSFW_CACHE_POLICY_CACHED (PVR_MIPSFW_CACHE_POLICY_WRITEBACK) /* Cached policy used by MIPS in case of physical bus on more than 32 bit. */ -#define PVR_MIPSFW_CACHED_POLICY_ABOVE_32BIT (PVR_MIPSFW_WRITETHROUGH_CACHE_POLICY) +#define PVR_MIPSFW_CACHE_POLICY_CACHED_ABOVE_32BIT (PVR_MIPSFW_CACHE_POLICY_WRITETHROUGH) + /* Total number of Remap entries. */ #define PVR_MIPSFW_NUMBER_OF_REMAP_ENTRIES (2 * PVR_MIPSFW_NUMBER_OF_TLB_ENTRIES) /* MIPS EntryLo/PTE format. */ - -#define PVR_MIPSFW_ENTRYLO_READ_INHIBIT_SHIFT (31U) -#define PVR_MIPSFW_ENTRYLO_READ_INHIBIT_CLRMSK (0X7FFFFFFF) -#define PVR_MIPSFW_ENTRYLO_READ_INHIBIT_EN (0X80000000) - -#define PVR_MIPSFW_ENTRYLO_EXEC_INHIBIT_SHIFT (30U) -#define PVR_MIPSFW_ENTRYLO_EXEC_INHIBIT_CLRMSK (0XBFFFFFFF) -#define PVR_MIPSFW_ENTRYLO_EXEC_INHIBIT_EN (0X40000000) - -/* Page Frame Number */ -#define PVR_MIPSFW_ENTRYLO_PFN_SHIFT (6) -#define PVR_MIPSFW_ENTRYLO_PFN_ALIGNSHIFT (12) -/* Mask used for the MIPS Page Table in case of physical bus on 32 bit. */ -#define PVR_MIPSFW_ENTRYLO_PFN_MASK (0x03FFFFC0) -#define PVR_MIPSFW_ENTRYLO_PFN_SIZE (20) -/* Mask used for the MIPS Page Table in case of physical bus on more than 32 bit. */ -#define PVR_MIPSFW_ENTRYLO_PFN_MASK_ABOVE_32BIT (0x3FFFFFC0) -#define PVR_MIPSFW_ENTRYLO_PFN_SIZE_ABOVE_32BIT (24) -#define PVR_MIPSFW_ADDR_TO_ENTRYLO_PFN_RSHIFT (PVR_MIPSFW_ENTRYLO_PFN_ALIGNSHIFT - \ - PVR_MIPSFW_ENTRYLO_PFN_SHIFT) - -#define PVR_MIPSFW_ENTRYLO_CACHE_POLICY_SHIFT (3U) -#define PVR_MIPSFW_ENTRYLO_CACHE_POLICY_CLRMSK (0XFFFFFFC7) - -#define PVR_MIPSFW_ENTRYLO_DIRTY_SHIFT (2U) -#define PVR_MIPSFW_ENTRYLO_DIRTY_CLRMSK (0XFFFFFFFB) -#define PVR_MIPSFW_ENTRYLO_DIRTY_EN (0X00000004) - -#define PVR_MIPSFW_ENTRYLO_VALID_SHIFT (1U) -#define PVR_MIPSFW_ENTRYLO_VALID_CLRMSK (0XFFFFFFFD) -#define PVR_MIPSFW_ENTRYLO_VALID_EN (0X00000002) - -#define PVR_MIPSFW_ENTRYLO_GLOBAL_SHIFT (0U) -#define PVR_MIPSFW_ENTRYLO_GLOBAL_CLRMSK (0XFFFFFFFE) -#define PVR_MIPSFW_ENTRYLO_GLOBAL_EN (0X00000001) - -#define PVR_MIPSFW_ENTRYLO_DVG (PVR_MIPSFW_ENTRYLO_DIRTY_EN | \ - PVR_MIPSFW_ENTRYLO_VALID_EN | \ - PVR_MIPSFW_ENTRYLO_GLOBAL_EN) -#define PVR_MIPSFW_ENTRYLO_UNCACHED (PVR_MIPSFW_UNCACHED_CACHE_POLICY << \ - PVR_MIPSFW_ENTRYLO_CACHE_POLICY_SHIFT) -#define PVR_MIPSFW_ENTRYLO_DVG_UNCACHED (PVR_MIPSFW_ENTRYLO_DVG | \ - PVR_MIPSFW_ENTRYLO_UNCACHED) +#define PVR_MIPSFW_ENTRYLO_READ_INHIBIT BIT_U32(31) +#define PVR_MIPSFW_ENTRYLO_EXEC_INHIBIT BIT_U32(30) +#define PVR_MIPSFW_ENTRYLO_PFN GENMASK_U32(25, 6) +#define PVR_MIPSFW_ENTRYLO_PFN_ABOVE_32BIT GENMASK_U32(29, 6) +#define PVR_MIPSFW_ENTRYLO_CACHE_POLICY GENMASK_U32(5, 3) +#define PVR_MIPSFW_ENTRYLO_DIRTY BIT_U32(2) +#define PVR_MIPSFW_ENTRYLO_VALID BIT_U32(1) +#define PVR_MIPSFW_ENTRYLO_GLOBAL BIT_U32(0) /* Remap Range Config Addr Out. */ /* These defines refer to the upper half of the Remap Range Config register. */ @@ -293,7 +263,7 @@ struct pvr_mipsfw_boot_data { #define PVR_MIPSFW_TLB_GET_PFN(entry_lo) (((entry_lo) >> 6) & 0XFFFFFU) /* GET_PA uses a non-standard PFN mask for 36 bit addresses. */ #define PVR_MIPSFW_TLB_GET_PA(entry_lo) (((u64)(entry_lo) & \ - PVR_MIPSFW_ENTRYLO_PFN_MASK_ABOVE_32BIT) << 6) + PVR_MIPSFW_ENTRYLO_PFN_ABOVE_32BIT) << 6) #define PVR_MIPSFW_TLB_GET_INHIBIT(entry_lo) (((entry_lo) >> 30) & 0x3U) #define PVR_MIPSFW_TLB_GET_DGV(entry_lo) ((entry_lo) & 0x7U) #define PVR_MIPSFW_TLB_GLOBAL BIT(0) diff --git a/drivers/gpu/drm/imagination/pvr_vm_mips.c b/drivers/gpu/drm/imagination/pvr_vm_mips.c index b74386032e6a..f6fa723649d3 100644 --- a/drivers/gpu/drm/imagination/pvr_vm_mips.c +++ b/drivers/gpu/drm/imagination/pvr_vm_mips.c @@ -97,11 +97,13 @@ pvr_vm_mips_init(struct pvr_device *pvr_dev) goto err_free_pages; } - vm_data->pfn_mask = (phys_bus_width > 32) ? PVR_MIPSFW_ENTRYLO_PFN_MASK_ABOVE_32BIT : - PVR_MIPSFW_ENTRYLO_PFN_MASK; - - vm_data->cache_policy = (phys_bus_width > 32) ? PVR_MIPSFW_CACHED_POLICY_ABOVE_32BIT : - PVR_MIPSFW_CACHED_POLICY; + if (phys_bus_width > 32) { + vm_data->pfn_mask = PVR_MIPSFW_ENTRYLO_PFN_ABOVE_32BIT; + vm_data->cache_policy = PVR_MIPSFW_CACHE_POLICY_CACHED_ABOVE_32BIT; + } else { + vm_data->pfn_mask = PVR_MIPSFW_ENTRYLO_PFN; + vm_data->cache_policy = PVR_MIPSFW_CACHE_POLICY_CACHED; + } pvr_dev->fw_dev.processor_data.mips_data.vm_data = vm_data; @@ -145,15 +147,15 @@ get_mips_pte_flags(bool read, bool write, u32 cache_policy) u32 flags = 0; if (read && write) /* Read/write. */ - flags |= PVR_MIPSFW_ENTRYLO_DIRTY_EN; + flags |= PVR_MIPSFW_ENTRYLO_DIRTY; else if (write) /* Write only. */ - flags |= PVR_MIPSFW_ENTRYLO_READ_INHIBIT_EN; + flags |= PVR_MIPSFW_ENTRYLO_READ_INHIBIT; else WARN_ON(!read); - flags |= cache_policy << PVR_MIPSFW_ENTRYLO_CACHE_POLICY_SHIFT; + flags |= FIELD_PREP(PVR_MIPSFW_ENTRYLO_CACHE_POLICY, cache_policy); - flags |= PVR_MIPSFW_ENTRYLO_VALID_EN | PVR_MIPSFW_ENTRYLO_GLOBAL_EN; + flags |= PVR_MIPSFW_ENTRYLO_VALID | PVR_MIPSFW_ENTRYLO_GLOBAL; return flags; } @@ -199,7 +201,7 @@ pvr_vm_mips_map(struct pvr_device *pvr_dev, struct pvr_fw_object *fw_obj) end_pfn = (end & fw_dev->fw_heap_info.offset_mask) >> PVR_MIPSFW_LOG2_PAGE_SIZE_4K; if (pvr_obj->flags & PVR_BO_FW_FLAGS_DEVICE_UNCACHED) - cache_policy = PVR_MIPSFW_UNCACHED_CACHE_POLICY; + cache_policy = PVR_MIPSFW_CACHE_POLICY_UNCACHED; else cache_policy = vm_data->cache_policy; @@ -216,8 +218,7 @@ pvr_vm_mips_map(struct pvr_device *pvr_dev, struct pvr_fw_object *fw_obj) if (err) goto err_unmap_pages; - pte = ((dma_addr >> PVR_MIPSFW_LOG2_PAGE_SIZE_4K) - << PVR_MIPSFW_ENTRYLO_PFN_SHIFT) & vm_data->pfn_mask; + pte = field_prep(vm_data->pfn_mask, dma_addr >> PVR_MIPSFW_LOG2_PAGE_SIZE_4K); pte |= pte_flags; WRITE_ONCE(vm_data->pt[pfn], pte); -- 2.43.0