linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v3] nvmem: mtk-efuse: Remove EFUSE register write support
@ 2020-08-26  8:39 Chih-En Hsu
  2020-08-26 13:31 ` andrew-ct chen
  2020-09-04 11:07 ` Srinivas Kandagatla
  0 siblings, 2 replies; 3+ messages in thread
From: Chih-En Hsu @ 2020-08-26  8:39 UTC (permalink / raw)
  To: Srinivas Kandagatla, Matthias Brugger
  Cc: linux-arm-kernel, linux-mediatek, linux-kernel, srv_heupstream,
	Andrew-CT.Chen, Michael.Mei, Chih-En Hsu

This patch is to remove function "mtk_reg_write" since
Mediatek EFUSE hardware only supports read functionality
for NVMEM consumers.

Fixes: ba360fd040e3 ("nvmem: mtk-efuse: remove nvmem regmap dependency")
Signed-off-by: Chih-En Hsu <chih-en.hsu@mediatek.com>
---
 drivers/nvmem/mtk-efuse.c | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/drivers/nvmem/mtk-efuse.c b/drivers/nvmem/mtk-efuse.c
index 856d9c3fc38e..6a537d959f14 100644
--- a/drivers/nvmem/mtk-efuse.c
+++ b/drivers/nvmem/mtk-efuse.c
@@ -28,19 +28,6 @@ static int mtk_reg_read(void *context,
 	return 0;
 }
 
-static int mtk_reg_write(void *context,
-			 unsigned int reg, void *_val, size_t bytes)
-{
-	struct mtk_efuse_priv *priv = context;
-	u32 *val = _val;
-	int i = 0, words = bytes / 4;
-
-	while (words--)
-		writel(*val++, priv->base + reg + (i++ * 4));
-
-	return 0;
-}
-
 static int mtk_efuse_probe(struct platform_device *pdev)
 {
 	struct device *dev = &pdev->dev;
@@ -61,7 +48,6 @@ static int mtk_efuse_probe(struct platform_device *pdev)
 	econfig.stride = 4;
 	econfig.word_size = 4;
 	econfig.reg_read = mtk_reg_read;
-	econfig.reg_write = mtk_reg_write;
 	econfig.size = resource_size(res);
 	econfig.priv = priv;
 	econfig.dev = dev;
-- 
2.18.0

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [PATCH v3] nvmem: mtk-efuse: Remove EFUSE register write support
  2020-08-26  8:39 [PATCH v3] nvmem: mtk-efuse: Remove EFUSE register write support Chih-En Hsu
@ 2020-08-26 13:31 ` andrew-ct chen
  2020-09-04 11:07 ` Srinivas Kandagatla
  1 sibling, 0 replies; 3+ messages in thread
From: andrew-ct chen @ 2020-08-26 13:31 UTC (permalink / raw)
  To: Chih-En Hsu
  Cc: Srinivas Kandagatla, Matthias Brugger, linux-arm-kernel,
	linux-mediatek, linux-kernel, srv_heupstream, Michael.Mei

Hi Chih-En,


On Wed, 2020-08-26 at 16:39 +0800, Chih-En Hsu wrote:
> This patch is to remove function "mtk_reg_write" since
> Mediatek EFUSE hardware only supports read functionality
> for NVMEM consumers.

Acked-by: Andrew-CT Chen <andrew-ct.chen@mediatek.com>

> 
> Fixes: ba360fd040e3 ("nvmem: mtk-efuse: remove nvmem regmap dependency")
> Signed-off-by: Chih-En Hsu <chih-en.hsu@mediatek.com>
> ---
>  drivers/nvmem/mtk-efuse.c | 14 --------------
>  1 file changed, 14 deletions(-)
> 
> diff --git a/drivers/nvmem/mtk-efuse.c b/drivers/nvmem/mtk-efuse.c
> index 856d9c3fc38e..6a537d959f14 100644
> --- a/drivers/nvmem/mtk-efuse.c
> +++ b/drivers/nvmem/mtk-efuse.c
> @@ -28,19 +28,6 @@ static int mtk_reg_read(void *context,
>  	return 0;
>  }
>  
> -static int mtk_reg_write(void *context,
> -			 unsigned int reg, void *_val, size_t bytes)
> -{
> -	struct mtk_efuse_priv *priv = context;
> -	u32 *val = _val;
> -	int i = 0, words = bytes / 4;
> -
> -	while (words--)
> -		writel(*val++, priv->base + reg + (i++ * 4));
> -
> -	return 0;
> -}
> -
>  static int mtk_efuse_probe(struct platform_device *pdev)
>  {
>  	struct device *dev = &pdev->dev;
> @@ -61,7 +48,6 @@ static int mtk_efuse_probe(struct platform_device *pdev)
>  	econfig.stride = 4;
>  	econfig.word_size = 4;
>  	econfig.reg_read = mtk_reg_read;
> -	econfig.reg_write = mtk_reg_write;
>  	econfig.size = resource_size(res);
>  	econfig.priv = priv;
>  	econfig.dev = dev;


^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH v3] nvmem: mtk-efuse: Remove EFUSE register write support
  2020-08-26  8:39 [PATCH v3] nvmem: mtk-efuse: Remove EFUSE register write support Chih-En Hsu
  2020-08-26 13:31 ` andrew-ct chen
@ 2020-09-04 11:07 ` Srinivas Kandagatla
  1 sibling, 0 replies; 3+ messages in thread
From: Srinivas Kandagatla @ 2020-09-04 11:07 UTC (permalink / raw)
  To: Chih-En Hsu, Matthias Brugger
  Cc: linux-arm-kernel, linux-mediatek, linux-kernel, srv_heupstream,
	Andrew-CT.Chen, Michael.Mei



On 26/08/2020 09:39, Chih-En Hsu wrote:
> This patch is to remove function "mtk_reg_write" since
> Mediatek EFUSE hardware only supports read functionality
> for NVMEM consumers.
> 
> Fixes: ba360fd040e3 ("nvmem: mtk-efuse: remove nvmem regmap dependency")
> Signed-off-by: Chih-En Hsu <chih-en.hsu@mediatek.com>
> ---
>   drivers/nvmem/mtk-efuse.c | 14 --------------
>   1 file changed, 14 deletions(-)


Applied thanks!

--srini

> 
> diff --git a/drivers/nvmem/mtk-efuse.c b/drivers/nvmem/mtk-efuse.c
> index 856d9c3fc38e..6a537d959f14 100644
> --- a/drivers/nvmem/mtk-efuse.c
> +++ b/drivers/nvmem/mtk-efuse.c
> @@ -28,19 +28,6 @@ static int mtk_reg_read(void *context,
>   	return 0;
>   }
>   
> -static int mtk_reg_write(void *context,
> -			 unsigned int reg, void *_val, size_t bytes)
> -{
> -	struct mtk_efuse_priv *priv = context;
> -	u32 *val = _val;
> -	int i = 0, words = bytes / 4;
> -
> -	while (words--)
> -		writel(*val++, priv->base + reg + (i++ * 4));
> -
> -	return 0;
> -}
> -
>   static int mtk_efuse_probe(struct platform_device *pdev)
>   {
>   	struct device *dev = &pdev->dev;
> @@ -61,7 +48,6 @@ static int mtk_efuse_probe(struct platform_device *pdev)
>   	econfig.stride = 4;
>   	econfig.word_size = 4;
>   	econfig.reg_read = mtk_reg_read;
> -	econfig.reg_write = mtk_reg_write;
>   	econfig.size = resource_size(res);
>   	econfig.priv = priv;
>   	econfig.dev = dev;
> 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2020-09-04 11:07 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-08-26  8:39 [PATCH v3] nvmem: mtk-efuse: Remove EFUSE register write support Chih-En Hsu
2020-08-26 13:31 ` andrew-ct chen
2020-09-04 11:07 ` Srinivas Kandagatla

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).