dt-bindings: mfd: bd71847-pmic: Remove unneeded LED header

[PATCH] dt-bindings: mfd: bd71847-pmic: Remove unneeded LED header

Posted by Fabio Estevam 2 years, 1 month ago

From: Fabio Estevam <festevam@denx.de>

There is nothing from <dt-bindings/leds/common.h> that is needed by
the binding example.

Remove the unneeded inclusion.

Signed-off-by: Fabio Estevam <festevam@denx.de>
---
 Documentation/devicetree/bindings/mfd/rohm,bd71847-pmic.yaml | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Documentation/devicetree/bindings/mfd/rohm,bd71847-pmic.yaml b/Documentation/devicetree/bindings/mfd/rohm,bd71847-pmic.yaml
index 7ab7b2c7f3e6..d783cc4e4e86 100644
--- a/Documentation/devicetree/bindings/mfd/rohm,bd71847-pmic.yaml
+++ b/Documentation/devicetree/bindings/mfd/rohm,bd71847-pmic.yaml
@@ -130,7 +130,6 @@ dependencies:
 examples:
   - |
     #include <dt-bindings/interrupt-controller/irq.h>
-    #include <dt-bindings/leds/common.h>
 
     i2c {
         #address-cells = <1>;
-- 
2.34.1

Re: (subset) [PATCH] dt-bindings: mfd: bd71847-pmic: Remove unneeded LED header

Posted by Lee Jones 2 years ago

On Thu, 03 Aug 2023 23:16:57 -0300, Fabio Estevam wrote:
> There is nothing from <dt-bindings/leds/common.h> that is needed by
> the binding example.
> 
> Remove the unneeded inclusion.
> 
> 

Applied, thanks!

[1/1] dt-bindings: mfd: bd71847-pmic: Remove unneeded LED header
      commit: af18b213632b2cc002773a81f2dec75b8b266048

--
Lee Jones [李琼斯]

Re: [PATCH] dt-bindings: mfd: bd71847-pmic: Remove unneeded LED header

Posted by Conor Dooley 2 years, 1 month ago

On Thu, Aug 03, 2023 at 11:16:57PM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> There is nothing from <dt-bindings/leds/common.h> that is needed by
> the binding example.
> 
> Remove the unneeded inclusion.
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Acked-by: Conor Dooley <conor.dooley@microchip.com>

Thanks,
Conor.

Re: [PATCH] dt-bindings: mfd: bd71847-pmic: Remove unneeded LED header

Posted by Matti Vaittinen 2 years, 1 month ago

On 8/4/23 05:16, Fabio Estevam wrote:
> From: Fabio Estevam <festevam@denx.de>
> 
> There is nothing from <dt-bindings/leds/common.h> that is needed by
> the binding example.
> 
> Remove the unneeded inclusion.
> 
> Signed-off-by: Fabio Estevam <festevam@denx.de>

Reviewed-by: Matti Vaittinen <mazziesaccount@gmail.com>

> ---
>   Documentation/devicetree/bindings/mfd/rohm,bd71847-pmic.yaml | 1 -
>   1 file changed, 1 deletion(-)
> 
> diff --git a/Documentation/devicetree/bindings/mfd/rohm,bd71847-pmic.yaml b/Documentation/devicetree/bindings/mfd/rohm,bd71847-pmic.yaml
> index 7ab7b2c7f3e6..d783cc4e4e86 100644
> --- a/Documentation/devicetree/bindings/mfd/rohm,bd71847-pmic.yaml
> +++ b/Documentation/devicetree/bindings/mfd/rohm,bd71847-pmic.yaml
> @@ -130,7 +130,6 @@ dependencies:
>   examples:
>     - |
>       #include <dt-bindings/interrupt-controller/irq.h>
> -    #include <dt-bindings/leds/common.h>
>   
>       i2c {
>           #address-cells = <1>;

-- 
Matti Vaittinen
Linux kernel developer at ROHM Semiconductors
Oulu Finland

~~ When things go utterly wrong vim users can always type :help! ~~