diff mbox series

ARM: dts: omap am5729-beagleboneai: drop unneeded ti,enable-id-detection

Message ID 20240615174904.39012-1-krzysztof.kozlowski@linaro.org (mailing list archive)
State New
Headers show
Series ARM: dts: omap am5729-beagleboneai: drop unneeded ti,enable-id-detection | expand

Commit Message

Krzysztof Kozlowski June 15, 2024, 5:49 p.m. UTC
There is a ti,enable-id-detection property in the Extcon Palmas
(extcon-palmas), but not in the Extcon USB GPIO binding and driver.

Reported-by: kernel test robot <lkp@intel.com>
Closes: https://lore.kernel.org/oe-kbuild-all/202406152004.F2fNnorG-lkp@intel.com/
Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm/boot/dts/ti/omap/am5729-beagleboneai.dts | 1 -
 1 file changed, 1 deletion(-)

Comments

Krzysztof Kozlowski June 26, 2024, 11:33 a.m. UTC | #1
On 15/06/2024 19:49, Krzysztof Kozlowski wrote:
> There is a ti,enable-id-detection property in the Extcon Palmas
> (extcon-palmas), but not in the Extcon USB GPIO binding and driver.
> 
> Reported-by: kernel test robot <lkp@intel.com>
> Closes: https://lore.kernel.org/oe-kbuild-all/202406152004.F2fNnorG-lkp@intel.com/
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>

Tony,

Do you still pick up patches for OMAP? Any comments on this?

Best regards,
Krzysztof
Robert Nelson June 26, 2024, 4:48 p.m. UTC | #2
On Wed, Jun 26, 2024 at 6:34 AM Krzysztof Kozlowski
<krzysztof.kozlowski@linaro.org> wrote:
>
> On 15/06/2024 19:49, Krzysztof Kozlowski wrote:
> > There is a ti,enable-id-detection property in the Extcon Palmas
> > (extcon-palmas), but not in the Extcon USB GPIO binding and driver.
> >
> > Reported-by: kernel test robot <lkp@intel.com>
> > Closes: https://lore.kernel.org/oe-kbuild-all/202406152004.F2fNnorG-lkp@intel.com/
> > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>
> Tony,
>
> Do you still pick up patches for OMAP? Any comments on this?

Krzysztof,

It looks like Tony didn't send his final MAINTAINERS update:

https://lore.kernel.org/linux-arm-kernel/20240419055249.GE5156@atomide.com/T/

@Andrew F. Davis
and @Roger Quadros have you guys set up a git tree for omap patches?

Regards,

--
Robert Nelson
https://rcn-ee.com/
Krzysztof Kozlowski June 26, 2024, 4:57 p.m. UTC | #3
On 26/06/2024 18:48, Robert Nelson wrote:
> On Wed, Jun 26, 2024 at 6:34 AM Krzysztof Kozlowski
> <krzysztof.kozlowski@linaro.org> wrote:
>>
>> On 15/06/2024 19:49, Krzysztof Kozlowski wrote:
>>> There is a ti,enable-id-detection property in the Extcon Palmas
>>> (extcon-palmas), but not in the Extcon USB GPIO binding and driver.
>>>
>>> Reported-by: kernel test robot <lkp@intel.com>
>>> Closes: https://lore.kernel.org/oe-kbuild-all/202406152004.F2fNnorG-lkp@intel.com/
>>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>>
>> Tony,
>>
>> Do you still pick up patches for OMAP? Any comments on this?
> 
> Krzysztof,
> 
> It looks like Tony didn't send his final MAINTAINERS update:
> 
> https://lore.kernel.org/linux-arm-kernel/20240419055249.GE5156@atomide.com/T/

Oh, thanks for letting me know.

> 
> @Andrew F. Davis
> and @Roger Quadros have you guys set up a git tree for omap patches?

I will pick up this patch to my DTS cleanups/fixes, but let me know if
anyone prefers to take it instead.

Best regards,
Krzysztof
Krzysztof Kozlowski June 26, 2024, 4:59 p.m. UTC | #4
On Sat, 15 Jun 2024 19:49:04 +0200, Krzysztof Kozlowski wrote:
> There is a ti,enable-id-detection property in the Extcon Palmas
> (extcon-palmas), but not in the Extcon USB GPIO binding and driver.
> 
> 

Follow up to:
https://lore.kernel.org/all/CAOCHtYh8YucHNwV6+S-3vfHvygs=5_UGVwPt6R+i+qBTc3eOTA@mail.gmail.com/

Let me know if someone else should/wants to take this.

Applied, thanks!

[1/1] ARM: dts: omap am5729-beagleboneai: drop unneeded ti,enable-id-detection
      https://git.kernel.org/krzk/linux-dt/c/90b6de4550aac6ac97448d3d26429a0a55dbaa34

Best regards,
Kevin Hilman June 28, 2024, 8:24 p.m. UTC | #5
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> writes:

> On 26/06/2024 18:48, Robert Nelson wrote:
>> On Wed, Jun 26, 2024 at 6:34 AM Krzysztof Kozlowski
>> <krzysztof.kozlowski@linaro.org> wrote:
>>>
>>> On 15/06/2024 19:49, Krzysztof Kozlowski wrote:
>>>> There is a ti,enable-id-detection property in the Extcon Palmas
>>>> (extcon-palmas), but not in the Extcon USB GPIO binding and driver.
>>>>
>>>> Reported-by: kernel test robot <lkp@intel.com>
>>>> Closes: https://lore.kernel.org/oe-kbuild-all/202406152004.F2fNnorG-lkp@intel.com/
>>>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>>>
>>> Tony,
>>>
>>> Do you still pick up patches for OMAP? Any comments on this?
>> 
>> Krzysztof,
>> 
>> It looks like Tony didn't send his final MAINTAINERS update:
>> 
>> https://lore.kernel.org/linux-arm-kernel/20240419055249.GE5156@atomide.com/T/
>
> Oh, thanks for letting me know.
>
>> 
>> @Andrew F. Davis
>> and @Roger Quadros have you guys set up a git tree for omap patches?
>
> I will pick up this patch to my DTS cleanups/fixes, but let me know if
> anyone prefers to take it instead.

As discussed with Tony, I will be starting to manage patches for these
platforms next month, but I'm happy for you to take this.

A bit late, since I see you already applied it, but FWIW:

Acked-by: Kevin Hilman <khilman@baylibre.com>

Thank you for picking this up.

Kevin
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/ti/omap/am5729-beagleboneai.dts b/arch/arm/boot/dts/ti/omap/am5729-beagleboneai.dts
index eb1ec85aba28..e6a18954e449 100644
--- a/arch/arm/boot/dts/ti/omap/am5729-beagleboneai.dts
+++ b/arch/arm/boot/dts/ti/omap/am5729-beagleboneai.dts
@@ -196,7 +196,6 @@  brcmf_pwrseq: brcmf_pwrseq {
 
 	extcon_usb1: extcon_usb1 {
 		compatible = "linux,extcon-usb-gpio";
-		ti,enable-id-detection;
 		id-gpios = <&gpio3 13 GPIO_ACTIVE_HIGH>;
 	};
 };