Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 63d93588 authored by Rajesh Kemisetti's avatar Rajesh Kemisetti
Browse files

msm: kgsl: Add quirk support to use alternative secure context bank 



Add quirk to select alternative secure context bank node
which can use different SID during secure context play back.

Change-Id: I60b8f84e5ca2c851c591145c9122f284eae6e4bb
Signed-off-by: default avatarRajesh Kemisetti <rajeshk@codeaurora.org>
parent 32fc4cc4

Documentation/devicetree/bindings/gpu/adreno-iommu.txt

+6 −0
Original line number Diff line number Diff line
@@ -42,6 +42,7 @@ Optional properties: 
  currently supported names are:

  - gfx3d_user : Used for the 'normal' GPU address space.

  - gfx3d_secure : Used for the content protection address space.

  - gfx3d_secure_alt : Used for the content protection address space for alternative SID.

  Each sub node has the following required properties:



	- compatible : "qcom,smmu-kgsl-cb"
@@ -83,4 +84,9 @@ Example: 
			compatible = "qcom,smmu-kgsl-cb";

			iommus = <&kgsl_smmu 2>;

		};



		gfx3d_secure_alt: gfx3d_secure_alt {

			compatible = "qcom,smmu-kgsl-cb";

			iommus = <&kgsl_smmu 2>, <&kgsl_smmu 1>;

		};

	};

Documentation/devicetree/bindings/gpu/adreno.txt

+3 −0
Original line number Diff line number Diff line
@@ -207,6 +207,9 @@ GPU Quirks: 
- qcom,gpu-quirk-limit-uche-gbif-rw:

				Limit number of read and write transactions from UCHE block to

				GBIF to avoid possible deadlock between GBIF, SMMU and MEMNOC.

- qcom,gpu-quirk-mmu-secure-cb-alt:

				Select alternate secure context bank to generate SID1 for

				secure playback.



KGSL Memory Pools:

- qcom,gpu-mempools:		Container for sets of GPU mempools.Multiple sets

drivers/gpu/msm/adreno.c

+1 −0
Original line number Diff line number Diff line
@@ -771,6 +771,7 @@ static struct { 
	{ ADRENO_QUIRK_SECVID_SET_ONCE, "qcom,gpu-quirk-secvid-set-once" },

	{ ADRENO_QUIRK_LIMIT_UCHE_GBIF_RW,

			"qcom,gpu-quirk-limit-uche-gbif-rw" },

	{ ADRENO_QUIRK_MMU_SECURE_CB_ALT, "qcom,gpu-quirk-mmu-secure-cb-alt" },

};



static struct device_node *

drivers/gpu/msm/adreno.h

+2 −0
Original line number Diff line number Diff line
@@ -150,6 +150,8 @@ 
 * between GBIF, SMMU and MEMNOC.

 */

#define ADRENO_QUIRK_LIMIT_UCHE_GBIF_RW BIT(8)

/* Select alternate secure context bank for mmu */

#define ADRENO_QUIRK_MMU_SECURE_CB_ALT BIT(9)



/* Flags to control command packet settings */

#define KGSL_CMD_FLAGS_NONE             0

drivers/gpu/msm/kgsl_iommu.c

+11 −2
Original line number Diff line number Diff line
@@ -2522,6 +2522,7 @@ static const struct { 
} kgsl_iommu_cbs[] = {

	{ KGSL_IOMMU_CONTEXT_USER, "gfx3d_user", },

	{ KGSL_IOMMU_CONTEXT_SECURE, "gfx3d_secure" },

	{ KGSL_IOMMU_CONTEXT_SECURE, "gfx3d_secure_alt" },

};



static int _kgsl_iommu_cb_probe(struct kgsl_device *device,
@@ -2529,12 +2530,20 @@ static int _kgsl_iommu_cb_probe(struct kgsl_device *device, 
{

	struct platform_device *pdev = of_find_device_by_node(node);

	struct kgsl_iommu_context *ctx = NULL;

	struct adreno_device *adreno_dev = ADRENO_DEVICE(device);

	int i;



	for (i = 0; i < ARRAY_SIZE(kgsl_iommu_cbs); i++) {

		if (!strcmp(node->name, kgsl_iommu_cbs[i].name)) {

			int id = kgsl_iommu_cbs[i].id;



			if (ADRENO_QUIRK(adreno_dev,

				ADRENO_QUIRK_MMU_SECURE_CB_ALT)) {

				if (!strcmp(node->name, "gfx3d_secure"))

					continue;

			} else if (!strcmp(node->name, "gfx3d_secure_alt"))

				continue;



			ctx = &iommu->ctx[id];

			ctx->id = id;

			ctx->cb_num = -1;
@@ -2545,8 +2554,8 @@ static int _kgsl_iommu_cb_probe(struct kgsl_device *device, 
	}



	if (ctx == NULL) {

		KGSL_CORE_ERR("dt: Unknown context label %s\n", node->name);

		return -EINVAL;

		KGSL_CORE_ERR("dt: Unused context label %s\n", node->name);

		return 0;

	}



	if (ctx->id == KGSL_IOMMU_CONTEXT_SECURE)