crypto: ccp - Constify 'dev_vdata' member

sp_device->dev_vdata points to only const data (see 'static const struct
sp_dev_vdata dev_vdata'), so can be made pointer to const for code
safety.

Update also sp_get_acpi_version() function which returns this pointer to
'pointer to const' for code readability, even though it is not needed.

On the other hand, do not touch similar function sp_get_of_version()
because it will be immediately removed in next patches.

Acked-by: Tom Lendacky <thomas.lendacky@amd.com>
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>

diff --git a/drivers/crypto/ccp/sp-dev.h b/drivers/crypto/ccp/sp-dev.h
index 6f9d7063257d70c7f836c99b4a3bf036d6b7057f..1335a83fe052ea90c0e998a3fee1a8ccc87f5a9a 100644 (file)
--- a/drivers/crypto/ccp/sp-dev.h
+++ b/drivers/crypto/ccp/sp-dev.h
@@ -95,7 +95,7 @@ struct sp_device {
 
        struct device *dev;
 
-       struct sp_dev_vdata *dev_vdata;
+       const struct sp_dev_vdata *dev_vdata;
        unsigned int ord;
        char name[SP_MAX_NAME_LEN];
 

diff --git a/drivers/crypto/ccp/sp-platform.c b/drivers/crypto/ccp/sp-platform.c
index 3933cac1694d4e15a799b522055d1f48a91283fc..de8a8183efdb4c63fcddb3dad63f8c964bb850bd 100644 (file)
--- a/drivers/crypto/ccp/sp-platform.c
+++ b/drivers/crypto/ccp/sp-platform.c
@@ -63,13 +63,13 @@ static struct sp_dev_vdata *sp_get_of_version(struct platform_device *pdev)
        return NULL;
 }
 
-static struct sp_dev_vdata *sp_get_acpi_version(struct platform_device *pdev)
+static const struct sp_dev_vdata *sp_get_acpi_version(struct platform_device *pdev)
 {
        const struct acpi_device_id *match;
 
        match = acpi_match_device(sp_acpi_match, &pdev->dev);
        if (match && match->driver_data)
-               return (struct sp_dev_vdata *)match->driver_data;
+               return (const struct sp_dev_vdata *)match->driver_data;
 
        return NULL;
 }