diff mbox series

soc: samsung: pmu: fix BUT->BIT macro typo

Message ID 20210517141250.55655-1-krzysztof.kozlowski@canonical.com (mailing list archive)
State Superseded
Headers show
Series soc: samsung: pmu: fix BUT->BIT macro typo | expand

Commit Message

Krzysztof Kozlowski May 17, 2021, 2:12 p.m. UTC
The macro EXYNOS5_USE_STANDBYWFI_ARM_CORE1 should use BIT, not BUT.  Fix
does not have real effect as the macro is not used in the code.

Fixes: af2e0a0754ac ("ARM: EXYNOS: Add PMU support for exynos5420")
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
---
 include/linux/soc/samsung/exynos-regs-pmu.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Alim Akhtar May 19, 2021, 2:36 p.m. UTC | #1
Hello Krzysztof

On Mon, May 17, 2021 at 7:59 PM Krzysztof Kozlowski
<krzysztof.kozlowski@canonical.com> wrote:
>
> The macro EXYNOS5_USE_STANDBYWFI_ARM_CORE1 should use BIT, not BUT.  Fix
> does not have real effect as the macro is not used in the code.
>
Can we consider removing this? As this is not used in code.

> Fixes: af2e0a0754ac ("ARM: EXYNOS: Add PMU support for exynos5420")
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@canonical.com>
> ---


>  include/linux/soc/samsung/exynos-regs-pmu.h | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/include/linux/soc/samsung/exynos-regs-pmu.h b/include/linux/soc/samsung/exynos-regs-pmu.h
> index fc9250fb3133..4bfd9a210da2 100644
> --- a/include/linux/soc/samsung/exynos-regs-pmu.h
> +++ b/include/linux/soc/samsung/exynos-regs-pmu.h
> @@ -613,7 +613,7 @@
>
>  /* For EXYNOS_CENTRAL_SEQ_OPTION */
>  #define EXYNOS5_USE_STANDBYWFI_ARM_CORE0                       BIT(16)
> -#define EXYNOS5_USE_STANDBYWFI_ARM_CORE1                       BUT(17)
> +#define EXYNOS5_USE_STANDBYWFI_ARM_CORE1                       BIT(17)
>  #define EXYNOS5_USE_STANDBYWFE_ARM_CORE0                       BIT(24)
>  #define EXYNOS5_USE_STANDBYWFE_ARM_CORE1                       BIT(25)
>
> --
> 2.27.0
>
Krzysztof Kozlowski May 19, 2021, 2:47 p.m. UTC | #2
On 19/05/2021 10:36, Alim Akhtar wrote:
> Hello Krzysztof
> 
> On Mon, May 17, 2021 at 7:59 PM Krzysztof Kozlowski
> <krzysztof.kozlowski@canonical.com> wrote:
>>
>> The macro EXYNOS5_USE_STANDBYWFI_ARM_CORE1 should use BIT, not BUT.  Fix
>> does not have real effect as the macro is not used in the code.
>>
> Can we consider removing this? As this is not used in code.

Sure, I guess, if someone needs this bit, can always check in the
documentation :)

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/include/linux/soc/samsung/exynos-regs-pmu.h b/include/linux/soc/samsung/exynos-regs-pmu.h
index fc9250fb3133..4bfd9a210da2 100644
--- a/include/linux/soc/samsung/exynos-regs-pmu.h
+++ b/include/linux/soc/samsung/exynos-regs-pmu.h
@@ -613,7 +613,7 @@ 
 
 /* For EXYNOS_CENTRAL_SEQ_OPTION */
 #define EXYNOS5_USE_STANDBYWFI_ARM_CORE0			BIT(16)
-#define EXYNOS5_USE_STANDBYWFI_ARM_CORE1			BUT(17)
+#define EXYNOS5_USE_STANDBYWFI_ARM_CORE1			BIT(17)
 #define EXYNOS5_USE_STANDBYWFE_ARM_CORE0			BIT(24)
 #define EXYNOS5_USE_STANDBYWFE_ARM_CORE1			BIT(25)