diff mbox series

[1/4] ARM: dts: aspeed: greatlakes: correct Mellanox multi-host property

Message ID 20231209104412.12916-1-krzysztof.kozlowski@linaro.org
State New
Headers show
Series [1/4] ARM: dts: aspeed: greatlakes: correct Mellanox multi-host property | expand

Commit Message

Krzysztof Kozlowski Dec. 9, 2023, 10:44 a.m. UTC
"mlx,multi-host" is using incorrect vendor prefix and is not documented.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Krzysztof Kozlowski April 3, 2024, 10:04 a.m. UTC | #1
On Sat, 09 Dec 2023 11:44:09 +0100, Krzysztof Kozlowski wrote:
> "mlx,multi-host" is using incorrect vendor prefix and is not documented.
> 
> 

These wait for ~4 months and they were not picked up. Let me know if anyone
else wants to take these.

Applied, thanks!

[1/4] ARM: dts: aspeed: greatlakes: correct Mellanox multi-host property
      https://git.kernel.org/krzk/linux-dt/c/7da85354c4fa35b862294dbbb450baeb405b5a92
[2/4] ARM: dts: aspeed: minerva-cmc: correct Mellanox multi-host property
      https://git.kernel.org/krzk/linux-dt/c/e515719c17beb9625a90039f6c45fa36d58bdda2
[3/4] ARM: dts: aspeed: yosemite4: correct Mellanox multi-host property
      https://git.kernel.org/krzk/linux-dt/c/af3deaf9bcb4571feb89a4050c7ad75de9aa8e1e
[4/4] ARM: dts: aspeed: yosemitev2: correct Mellanox multi-host property
      https://git.kernel.org/krzk/linux-dt/c/cac1c1dda6130771e06ace030b1b0ed62096a912

Best regards,
Andrew Jeffery April 4, 2024, 1:41 a.m. UTC | #2
On Wed, 2024-04-03 at 12:04 +0200, Krzysztof Kozlowski wrote:
> On Sat, 09 Dec 2023 11:44:09 +0100, Krzysztof Kozlowski wrote:
> > "mlx,multi-host" is using incorrect vendor prefix and is not documented.
> > 
> > 
> 
> These wait for ~4 months and they were not picked up. Let me know if anyone
> else wants to take these.
> 
> Applied, thanks!
> 
> [1/4] ARM: dts: aspeed: greatlakes: correct Mellanox multi-host property
>       https://git.kernel.org/krzk/linux-dt/c/7da85354c4fa35b862294dbbb450baeb405b5a92
> [2/4] ARM: dts: aspeed: minerva-cmc: correct Mellanox multi-host property
>       https://git.kernel.org/krzk/linux-dt/c/e515719c17beb9625a90039f6c45fa36d58bdda2
> [3/4] ARM: dts: aspeed: yosemite4: correct Mellanox multi-host property
>       https://git.kernel.org/krzk/linux-dt/c/af3deaf9bcb4571feb89a4050c7ad75de9aa8e1e
> [4/4] ARM: dts: aspeed: yosemitev2: correct Mellanox multi-host property
>       https://git.kernel.org/krzk/linux-dt/c/cac1c1dda6130771e06ace030b1b0ed62096a912
> 
> Best regards,

Ah, my apologies. Joel's on leave and I'm accumulating patches in a
tree for him in the mean time. I've had some things going on
professionally (changed jobs) and personally, and these fell into a bit
of a hole.

I'm okay for these patches to be integrated through your tree, given
you've already applied them. Feel free to add acks if your branch
allows:

Acked-by: Andrew Jeffery <andrew@codeconstruct.com.au>

I'm working to stay on top of things a bit more now than I have in the
recent past, so hopefully I won't miss patches again in the future.

Andrew
Krzysztof Kozlowski April 4, 2024, 6:13 a.m. UTC | #3
On 04/04/2024 03:41, Andrew Jeffery wrote:
> On Wed, 2024-04-03 at 12:04 +0200, Krzysztof Kozlowski wrote:
>> On Sat, 09 Dec 2023 11:44:09 +0100, Krzysztof Kozlowski wrote:
>>> "mlx,multi-host" is using incorrect vendor prefix and is not documented.
>>>
>>>
>>
>> These wait for ~4 months and they were not picked up. Let me know if anyone
>> else wants to take these.
>>
>> Applied, thanks!
>>
>> [1/4] ARM: dts: aspeed: greatlakes: correct Mellanox multi-host property
>>       https://git.kernel.org/krzk/linux-dt/c/7da85354c4fa35b862294dbbb450baeb405b5a92
>> [2/4] ARM: dts: aspeed: minerva-cmc: correct Mellanox multi-host property
>>       https://git.kernel.org/krzk/linux-dt/c/e515719c17beb9625a90039f6c45fa36d58bdda2
>> [3/4] ARM: dts: aspeed: yosemite4: correct Mellanox multi-host property
>>       https://git.kernel.org/krzk/linux-dt/c/af3deaf9bcb4571feb89a4050c7ad75de9aa8e1e
>> [4/4] ARM: dts: aspeed: yosemitev2: correct Mellanox multi-host property
>>       https://git.kernel.org/krzk/linux-dt/c/cac1c1dda6130771e06ace030b1b0ed62096a912
>>
>> Best regards,
> 
> Ah, my apologies. Joel's on leave and I'm accumulating patches in a
> tree for him in the mean time. I've had some things going on
> professionally (changed jobs) and personally, and these fell into a bit
> of a hole.
> 
> I'm okay for these patches to be integrated through your tree, given
> you've already applied them. Feel free to add acks if your branch
> allows:
> 
> Acked-by: Andrew Jeffery <andrew@codeconstruct.com.au>
> 
> I'm working to stay on top of things a bit more now than I have in the
> recent past, so hopefully I won't miss patches again in the future.

Stephen reported conflict, although trivial, but maybe better if you
take them? I can rebase and resend.

Best regards,
Krzysztof
Andrew Jeffery April 4, 2024, 11:21 p.m. UTC | #4
On Thu, 2024-04-04 at 08:13 +0200, Krzysztof Kozlowski wrote:
> On 04/04/2024 03:41, Andrew Jeffery wrote:
> > On Wed, 2024-04-03 at 12:04 +0200, Krzysztof Kozlowski wrote:
> > > On Sat, 09 Dec 2023 11:44:09 +0100, Krzysztof Kozlowski wrote:
> > > > "mlx,multi-host" is using incorrect vendor prefix and is not documented.
> > > > 
> > > > 
> > > 
> > > These wait for ~4 months and they were not picked up. Let me know if anyone
> > > else wants to take these.
> > > 
> > > Applied, thanks!
> > > 
> > > [1/4] ARM: dts: aspeed: greatlakes: correct Mellanox multi-host property
> > >       https://git.kernel.org/krzk/linux-dt/c/7da85354c4fa35b862294dbbb450baeb405b5a92
> > > [2/4] ARM: dts: aspeed: minerva-cmc: correct Mellanox multi-host property
> > >       https://git.kernel.org/krzk/linux-dt/c/e515719c17beb9625a90039f6c45fa36d58bdda2
> > > [3/4] ARM: dts: aspeed: yosemite4: correct Mellanox multi-host property
> > >       https://git.kernel.org/krzk/linux-dt/c/af3deaf9bcb4571feb89a4050c7ad75de9aa8e1e
> > > [4/4] ARM: dts: aspeed: yosemitev2: correct Mellanox multi-host property
> > >       https://git.kernel.org/krzk/linux-dt/c/cac1c1dda6130771e06ace030b1b0ed62096a912
> > > 
> > > Best regards,
> > 
> > Ah, my apologies. Joel's on leave and I'm accumulating patches in a
> > tree for him in the mean time. I've had some things going on
> > professionally (changed jobs) and personally, and these fell into a bit
> > of a hole.
> > 
> > I'm okay for these patches to be integrated through your tree, given
> > you've already applied them. Feel free to add acks if your branch
> > allows:
> > 
> > Acked-by: Andrew Jeffery <andrew@codeconstruct.com.au>
> > 
> > I'm working to stay on top of things a bit more now than I have in the
> > recent past, so hopefully I won't miss patches again in the future.
> 
> Stephen reported conflict, although trivial, but maybe better if you
> take them?
> 

Yeah, happy to.

> I can rebase and resend.

Thanks.

Andrew
Krzysztof Kozlowski April 5, 2024, 6:48 a.m. UTC | #5
On 05/04/2024 01:21, Andrew Jeffery wrote:
>>>
>>> Acked-by: Andrew Jeffery <andrew@codeconstruct.com.au>
>>>
>>> I'm working to stay on top of things a bit more now than I have in the
>>> recent past, so hopefully I won't miss patches again in the future.
>>
>> Stephen reported conflict, although trivial, but maybe better if you
>> take them?
>>
> 
> Yeah, happy to.
> 
>> I can rebase and resend.

I dropped these patches (and one more) from my tree, rebased and resent
everything as v2.

Thanks!

Best regards,
Krzysztof
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts
index 7a53f54833a0..9a6757dd203f 100644
--- a/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts
+++ b/arch/arm/boot/dts/aspeed/aspeed-bmc-facebook-greatlakes.dts
@@ -66,7 +66,7 @@  &mac3 {
 	pinctrl-0 = <&pinctrl_rmii4_default>;
 	no-hw-checksum;
 	use-ncsi;
-	mlx,multi-host;
+	mellanox,multi-host;
 	ncsi-ctrl,start-redo-probe;
 	ncsi-ctrl,no-channel-monitor;
 	ncsi-package = <1>;