* [PATCH] arm64: defconfig: Enable HSR driver
@ 2024-04-19 6:00 Ravi Gunasekaran
2024-04-19 6:02 ` Arnd Bergmann
` (2 more replies)
0 siblings, 3 replies; 14+ messages in thread
From: Ravi Gunasekaran @ 2024-04-19 6:00 UTC (permalink / raw)
To: catalin.marinas, will
Cc: quic_bjorande, krzysztof.kozlowski, arnd, linux-arm-kernel,
linux-kernel, srk, r-gunasekaran
HSR is a redundancy protocol that can be realized with any
two port ethernet controller.
Many of TI's K3 SoCs support multi port ethernet controller.
So enable HSR driver inorder to support this protocol on
such SoCs.
Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
---
arch/arm64/configs/defconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 11fa4aa40094..750343564f41 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -176,6 +176,7 @@ CONFIG_NET_CLS_FLOWER=m
CONFIG_NET_CLS_ACT=y
CONFIG_NET_ACT_GACT=m
CONFIG_NET_ACT_MIRRED=m
+CONFIG_HSR=m
CONFIG_NET_ACT_GATE=m
CONFIG_QRTR_SMD=m
CONFIG_QRTR_TUN=m
--
2.17.1
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [PATCH] arm64: defconfig: Enable HSR driver
2024-04-19 6:00 [PATCH] arm64: defconfig: Enable HSR driver Ravi Gunasekaran
@ 2024-04-19 6:02 ` Arnd Bergmann
2024-04-19 6:25 ` Ravi Gunasekaran
2025-02-26 6:14 ` Malladi, Meghana
2025-02-26 6:18 ` Malladi, Meghana
2 siblings, 1 reply; 14+ messages in thread
From: Arnd Bergmann @ 2024-04-19 6:02 UTC (permalink / raw)
To: Ravi Gunasekaran, Catalin Marinas, Will Deacon, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo
Cc: Bjorn Andersson, Krzysztof Kozlowski, linux-arm-kernel,
linux-kernel, srk
On Fri, Apr 19, 2024, at 08:00, Ravi Gunasekaran wrote:
> HSR is a redundancy protocol that can be realized with any
> two port ethernet controller.
>
> Many of TI's K3 SoCs support multi port ethernet controller.
> So enable HSR driver inorder to support this protocol on
> such SoCs.
>
> Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
Acked-by: Arnd Bergmann <arnd@arndb.de>
The normal way this gets picked up into mainline is that you
send it to the K3 platform maintainers (added to Cc) and they
send me a pull request or forward the patch to soc@kernel.org.
Arnd
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] arm64: defconfig: Enable HSR driver
2024-04-19 6:02 ` Arnd Bergmann
@ 2024-04-19 6:25 ` Ravi Gunasekaran
2024-04-19 7:02 ` Arnd Bergmann
0 siblings, 1 reply; 14+ messages in thread
From: Ravi Gunasekaran @ 2024-04-19 6:25 UTC (permalink / raw)
To: Arnd Bergmann, Catalin Marinas, Will Deacon, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo
Cc: Bjorn Andersson, Krzysztof Kozlowski, linux-arm-kernel,
linux-kernel, srk, Ravi Gunasekaran
Arnd,
On 4/19/24 11:32 AM, Arnd Bergmann wrote:
> On Fri, Apr 19, 2024, at 08:00, Ravi Gunasekaran wrote:
>> HSR is a redundancy protocol that can be realized with any
>> two port ethernet controller.
>>
>> Many of TI's K3 SoCs support multi port ethernet controller.
>> So enable HSR driver inorder to support this protocol on
>> such SoCs.
>>
>> Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
>
> Acked-by: Arnd Bergmann <arnd@arndb.de>
>
> The normal way this gets picked up into mainline is that you
> send it to the K3 platform maintainers (added to Cc) and they
> send me a pull request or forward the patch to soc@kernel.org.
>
I usually do that for TI specific patches. This one seemed like a
generic one as HSR protocol can be run on any multi port ethernet
controller. So I wasn't sure about including TI maintainers.
But no harm in Cc'ing them. I will follow this advice for future
patches.
> Arnd
--
Regards,
Ravi
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] arm64: defconfig: Enable HSR driver
2024-04-19 6:25 ` Ravi Gunasekaran
@ 2024-04-19 7:02 ` Arnd Bergmann
0 siblings, 0 replies; 14+ messages in thread
From: Arnd Bergmann @ 2024-04-19 7:02 UTC (permalink / raw)
To: Ravi Gunasekaran, Catalin Marinas, Will Deacon, Nishanth Menon,
Vignesh Raghavendra, Tero Kristo
Cc: Bjorn Andersson, Krzysztof Kozlowski, linux-arm-kernel,
linux-kernel, srk
On Fri, Apr 19, 2024, at 08:25, Ravi Gunasekaran wrote:
> On 4/19/24 11:32 AM, Arnd Bergmann wrote:
>> On Fri, Apr 19, 2024, at 08:00, Ravi Gunasekaran wrote:
>>> HSR is a redundancy protocol that can be realized with any
>>> two port ethernet controller.
>>>
>>> Many of TI's K3 SoCs support multi port ethernet controller.
>>> So enable HSR driver inorder to support this protocol on
>>> such SoCs.
>>>
>>> Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
>>
>> Acked-by: Arnd Bergmann <arnd@arndb.de>
>>
>> The normal way this gets picked up into mainline is that you
>> send it to the K3 platform maintainers (added to Cc) and they
>> send me a pull request or forward the patch to soc@kernel.org.
>>
>
> I usually do that for TI specific patches. This one seemed like a
> generic one as HSR protocol can be run on any multi port ethernet
> controller. So I wasn't sure about including TI maintainers.
>
> But no harm in Cc'ing them. I will follow this advice for future
> patches.
Ah, I think I misread this as being much more hardware specific.
In a case like this, going through either a platform maintainer
or sending the patch directly to soc@kernel.org (with the usual
Ccs) is equally fine.
Arnd
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] arm64: defconfig: Enable HSR driver
2024-04-19 6:00 [PATCH] arm64: defconfig: Enable HSR driver Ravi Gunasekaran
2024-04-19 6:02 ` Arnd Bergmann
@ 2025-02-26 6:14 ` Malladi, Meghana
2025-02-26 6:18 ` Malladi, Meghana
2 siblings, 0 replies; 14+ messages in thread
From: Malladi, Meghana @ 2025-02-26 6:14 UTC (permalink / raw)
To: Ravi Gunasekaran, catalin.marinas, will
Cc: quic_bjorande, krzysztof.kozlowski, arnd, linux-arm-kernel,
linux-kernel, srk
Hi all,
Any reason why this patch hasn't been merged yet. Other than re-basing
this to the tip, anything else which needs to be addressed to get this
merged?
Thanks & Regards,
Meghana Malladi.
On 4/19/2024 11:30 AM, Ravi Gunasekaran wrote:
> HSR is a redundancy protocol that can be realized with any
> two port ethernet controller.
>
> Many of TI's K3 SoCs support multi port ethernet controller.
> So enable HSR driver inorder to support this protocol on
> such SoCs.
>
> Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
> ---
> arch/arm64/configs/defconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 11fa4aa40094..750343564f41 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -176,6 +176,7 @@ CONFIG_NET_CLS_FLOWER=m
> CONFIG_NET_CLS_ACT=y
> CONFIG_NET_ACT_GACT=m
> CONFIG_NET_ACT_MIRRED=m
> +CONFIG_HSR=m
> CONFIG_NET_ACT_GATE=m
> CONFIG_QRTR_SMD=m
> CONFIG_QRTR_TUN=m
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] arm64: defconfig: Enable HSR driver
2024-04-19 6:00 [PATCH] arm64: defconfig: Enable HSR driver Ravi Gunasekaran
2024-04-19 6:02 ` Arnd Bergmann
2025-02-26 6:14 ` Malladi, Meghana
@ 2025-02-26 6:18 ` Malladi, Meghana
2025-02-26 7:45 ` Arnd Bergmann
2025-02-26 8:05 ` Krzysztof Kozlowski
2 siblings, 2 replies; 14+ messages in thread
From: Malladi, Meghana @ 2025-02-26 6:18 UTC (permalink / raw)
To: catalin.marinas, will
Cc: quic_bjorande, krzysztof.kozlowski, arnd, linux-arm-kernel,
linux-kernel, srk, Anwar, Md Danish
Hi all,
Apologies in case you are receiving this email for the second time.
Any reason why this patch hasn't been merged yet. Other than re-basing
this to the tip, anything else which needs to be addressed to get this
merged?
Thanks & Regards,
Meghana Malladi.
On 4/19/2024 11:30 AM, Ravi Gunasekaran wrote:
> HSR is a redundancy protocol that can be realized with any
> two port ethernet controller.
>
> Many of TI's K3 SoCs support multi port ethernet controller.
> So enable HSR driver inorder to support this protocol on
> such SoCs.
>
> Signed-off-by: Ravi Gunasekaran <r-gunasekaran@ti.com>
> ---
> arch/arm64/configs/defconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 11fa4aa40094..750343564f41 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -176,6 +176,7 @@ CONFIG_NET_CLS_FLOWER=m
> CONFIG_NET_CLS_ACT=y
> CONFIG_NET_ACT_GACT=m
> CONFIG_NET_ACT_MIRRED=m
> +CONFIG_HSR=m
> CONFIG_NET_ACT_GATE=m
> CONFIG_QRTR_SMD=m
> CONFIG_QRTR_TUN=m
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] arm64: defconfig: Enable HSR driver
2025-02-26 6:18 ` Malladi, Meghana
@ 2025-02-26 7:45 ` Arnd Bergmann
2025-02-26 8:05 ` Krzysztof Kozlowski
1 sibling, 0 replies; 14+ messages in thread
From: Arnd Bergmann @ 2025-02-26 7:45 UTC (permalink / raw)
To: Malladi, Meghana, Catalin Marinas, Will Deacon
Cc: Bjorn Andersson, Krzysztof Kozlowski, linux-arm-kernel,
linux-kernel, srk, MD Danish Anwar
On Wed, Feb 26, 2025, at 07:18, Malladi, Meghana wrote:
> Hi all,
>
> Apologies in case you are receiving this email for the second time.
> Any reason why this patch hasn't been merged yet. Other than re-basing
> this to the tip, anything else which needs to be addressed to get this
> merged?
>
Someone needs to send it to soc@linux.dev so it makes it into patchwork,
ideally it should come from a platform maintainer that needs it to
be enabled.
Arnd
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [PATCH] arm64: defconfig: Enable HSR driver
2025-02-26 6:18 ` Malladi, Meghana
2025-02-26 7:45 ` Arnd Bergmann
@ 2025-02-26 8:05 ` Krzysztof Kozlowski
2025-02-26 10:44 ` [EXTERNAL] " Malladi, Meghana
1 sibling, 1 reply; 14+ messages in thread
From: Krzysztof Kozlowski @ 2025-02-26 8:05 UTC (permalink / raw)
To: Malladi, Meghana, catalin.marinas, will
Cc: quic_bjorande, arnd, linux-arm-kernel, linux-kernel, srk,
Anwar, Md Danish
On 26/02/2025 07:18, Malladi, Meghana wrote:
> Hi all,
>
> Apologies in case you are receiving this email for the second time.
> Any reason why this patch hasn't been merged yet. Other than re-basing
> this to the tip, anything else which needs to be addressed to get this
> merged?
I don't think you sent it to your platform maintainers, so no one would
apply it. Usually people ignore emails they did not receive, although I
am here an exception tracking missing-DT-patchwork things... :)
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [EXTERNAL] Re: [PATCH] arm64: defconfig: Enable HSR driver
2025-02-26 8:05 ` Krzysztof Kozlowski
@ 2025-02-26 10:44 ` Malladi, Meghana
2025-02-26 10:48 ` Krzysztof Kozlowski
0 siblings, 1 reply; 14+ messages in thread
From: Malladi, Meghana @ 2025-02-26 10:44 UTC (permalink / raw)
To: Krzysztof Kozlowski, Arnd Bergmann
Cc: quic_bjorande, arnd, linux-arm-kernel, linux-kernel, srk,
Anwar, Md Danish, catalin.marinas, will
On 2/26/2025 1:35 PM, Krzysztof Kozlowski wrote:
> On 26/02/2025 07: 18, Malladi, Meghana wrote: > Hi all, > > Apologies in
> case you are receiving this email for the second time. > Any reason why
> this patch hasn't been merged yet. Other than re-basing > this to the
> tip, anything
> ZjQcmQRYFpfptBannerStart
> This message was sent from outside of Texas Instruments.
> Do not click links or open attachments unless you recognize the source
> of this email and know the content is safe.
> Report Suspicious
> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!
> updgnZav1ue7gKXOnHOpH5hg8gLmw9Osqsqh-
> KTLRTnBEurV_VetrG7mzU898vB_5xFrYgbRkkuuFMBobM-uhFAmUgccRSk$>
> ZjQcmQRYFpfptBannerEnd
>
> On 26/02/2025 07:18, Malladi, Meghana wrote:
>> Hi all,
>>
>> Apologies in case you are receiving this email for the second time.
>> Any reason why this patch hasn't been merged yet. Other than re-basing
>> this to the tip, anything else which needs to be addressed to get this
>> merged?
>
> I don't think you sent it to your platform maintainers, so no one would
> apply it. Usually people ignore emails they did not receive, although I
> am here an exception tracking missing-DT-patchwork things... :)
>
> Best regards,
> Krzysztof
>
Hello Arnd/Krzysztof,
Thank you for the replies :)
Was having ambiguity since this is not TI specific change. Wanted to
confirm before re-posting. Will post v2 patch including TI platform
maintainers.
Regards,
Meghana
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [EXTERNAL] Re: [PATCH] arm64: defconfig: Enable HSR driver
2025-02-26 10:44 ` [EXTERNAL] " Malladi, Meghana
@ 2025-02-26 10:48 ` Krzysztof Kozlowski
2025-02-26 11:54 ` Malladi, Meghana
0 siblings, 1 reply; 14+ messages in thread
From: Krzysztof Kozlowski @ 2025-02-26 10:48 UTC (permalink / raw)
To: Malladi, Meghana, Arnd Bergmann
Cc: quic_bjorande, linux-arm-kernel, linux-kernel, srk,
Anwar, Md Danish, catalin.marinas, will
On 26/02/2025 11:44, Malladi, Meghana wrote:
>
>
> On 2/26/2025 1:35 PM, Krzysztof Kozlowski wrote:
>> On 26/02/2025 07: 18, Malladi, Meghana wrote: > Hi all, > > Apologies in
>> case you are receiving this email for the second time. > Any reason why
>> this patch hasn't been merged yet. Other than re-basing > this to the
>> tip, anything
>> ZjQcmQRYFpfptBannerStart
>> This message was sent from outside of Texas Instruments.
>> Do not click links or open attachments unless you recognize the source
>> of this email and know the content is safe.
>> Report Suspicious
>> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!
>> updgnZav1ue7gKXOnHOpH5hg8gLmw9Osqsqh-
>> KTLRTnBEurV_VetrG7mzU898vB_5xFrYgbRkkuuFMBobM-uhFAmUgccRSk$>
>> ZjQcmQRYFpfptBannerEnd
>>
>> On 26/02/2025 07:18, Malladi, Meghana wrote:
>>> Hi all,
>>>
>>> Apologies in case you are receiving this email for the second time.
BTW, Arnd when responded with Ack described the process.
>>> Any reason why this patch hasn't been merged yet. Other than re-basing
>>> this to the tip, anything else which needs to be addressed to get this
>>> merged?
>>
>> I don't think you sent it to your platform maintainers, so no one would
>> apply it. Usually people ignore emails they did not receive, although I
>> am here an exception tracking missing-DT-patchwork things... :)
>>
>> Best regards,
>> Krzysztof
>>
>
> Hello Arnd/Krzysztof,
>
> Thank you for the replies :)
> Was having ambiguity since this is not TI specific change. Wanted to
> confirm before re-posting. Will post v2 patch including TI platform
> maintainers.
It is not TI specific? Then which upstream boards benefit from this if
not TI?
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [EXTERNAL] Re: [PATCH] arm64: defconfig: Enable HSR driver
2025-02-26 10:48 ` Krzysztof Kozlowski
@ 2025-02-26 11:54 ` Malladi, Meghana
2025-02-26 12:06 ` Krzysztof Kozlowski
0 siblings, 1 reply; 14+ messages in thread
From: Malladi, Meghana @ 2025-02-26 11:54 UTC (permalink / raw)
To: Krzysztof Kozlowski, Arnd Bergmann
Cc: quic_bjorande, linux-arm-kernel, linux-kernel, srk,
Anwar, Md Danish, catalin.marinas, will
Hi Krzysztof,
On 2/26/2025 4:18 PM, Krzysztof Kozlowski wrote:
> On 26/02/2025 11: 44, Malladi, Meghana wrote: > > > On 2/26/2025 1: 35
> PM, Krzysztof Kozlowski wrote: >> On 26/02/2025 07: 18, Malladi, Meghana
> wrote: > Hi all, > > Apologies in >> case you are receiving this email
> ZjQcmQRYFpfptBannerStart
> This message was sent from outside of Texas Instruments.
> Do not click links or open attachments unless you recognize the source
> of this email and know the content is safe.
> Report Suspicious
> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!
> updq3RaPV007wGXFFrPD30z5a3GGYjpJweyhUcIfUTTaUs-8q0QpC4Yk9azBiFp7cBAf2Knk6fnVs4Fs-zRjTMEn8kmpiGo$>
> ZjQcmQRYFpfptBannerEnd
>
> On 26/02/2025 11:44, Malladi, Meghana wrote:
>>
>>
>> On 2/26/2025 1:35 PM, Krzysztof Kozlowski wrote:
>>> On 26/02/2025 07: 18, Malladi, Meghana wrote: > Hi all, > > Apologies in
>>> case you are receiving this email for the second time. > Any reason why
>>> this patch hasn't been merged yet. Other than re-basing > this to the
>>> tip, anything
>>> ZjQcmQRYFpfptBannerStart
>>> This message was sent from outside of Texas Instruments.
>>> Do not click links or open attachments unless you recognize the source
>>> of this email and know the content is safe.
>>> Report Suspicious
>>> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!
>>> updgnZav1ue7gKXOnHOpH5hg8gLmw9Osqsqh-
>>> KTLRTnBEurV_VetrG7mzU898vB_5xFrYgbRkkuuFMBobM-uhFAmUgccRSk$>
>>> ZjQcmQRYFpfptBannerEnd
>>>
>>> On 26/02/2025 07:18, Malladi, Meghana wrote:
>>>> Hi all,
>>>>
>>>> Apologies in case you are receiving this email for the second time.
>
> BTW, Arnd when responded with Ack described the process.
>
Yes, I wanted to re-confirm before posting it again (just in case).
>>>> Any reason why this patch hasn't been merged yet. Other than re-basing
>>>> this to the tip, anything else which needs to be addressed to get this
>>>> merged?
>>>
>>> I don't think you sent it to your platform maintainers, so no one would
>>> apply it. Usually people ignore emails they did not receive, although I
>>> am here an exception tracking missing-DT-patchwork things... :)
>>>
>>> Best regards,
>>> Krzysztof
>>>
>>
>> Hello Arnd/Krzysztof,
>>
>> Thank you for the replies :)
>> Was having ambiguity since this is not TI specific change. Wanted to
>> confirm before re-posting. Will post v2 patch including TI platform
>> maintainers.
>
> It is not TI specific? Then which upstream boards benefit from this if
> not TI?
>
This change enables generic HSR protocol support in the kernel which is
required for HSR driver support in TI boards. When I said "not TI
specific" - I meant this change is not internal to TI driver. Hope this
clarifies the miscommunication from my earlier mail.
Best Regards,
Meghana
>
>
> Best regards,
> Krzysztof
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [EXTERNAL] Re: [PATCH] arm64: defconfig: Enable HSR driver
2025-02-26 11:54 ` Malladi, Meghana
@ 2025-02-26 12:06 ` Krzysztof Kozlowski
2025-02-27 5:43 ` Malladi, Meghana
0 siblings, 1 reply; 14+ messages in thread
From: Krzysztof Kozlowski @ 2025-02-26 12:06 UTC (permalink / raw)
To: Malladi, Meghana, Arnd Bergmann
Cc: quic_bjorande, linux-arm-kernel, linux-kernel, srk,
Anwar, Md Danish, catalin.marinas, will
On 26/02/2025 12:54, Malladi, Meghana wrote:
> Hi Krzysztof,
>
> On 2/26/2025 4:18 PM, Krzysztof Kozlowski wrote:
>> On 26/02/2025 11: 44, Malladi, Meghana wrote: > > > On 2/26/2025 1: 35
>> PM, Krzysztof Kozlowski wrote: >> On 26/02/2025 07: 18, Malladi, Meghana
>> wrote: > Hi all, > > Apologies in >> case you are receiving this email
>> ZjQcmQRYFpfptBannerStart
>> This message was sent from outside of Texas Instruments.
>> Do not click links or open attachments unless you recognize the source
>> of this email and know the content is safe.
>> Report Suspicious
>> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!
>> updq3RaPV007wGXFFrPD30z5a3GGYjpJweyhUcIfUTTaUs-8q0QpC4Yk9azBiFp7cBAf2Knk6fnVs4Fs-zRjTMEn8kmpiGo$>
>> ZjQcmQRYFpfptBannerEnd
>>
>> On 26/02/2025 11:44, Malladi, Meghana wrote:
>>>
>>>
>>> On 2/26/2025 1:35 PM, Krzysztof Kozlowski wrote:
>>>> On 26/02/2025 07: 18, Malladi, Meghana wrote: > Hi all, > > Apologies in
>>>> case you are receiving this email for the second time. > Any reason why
>>>> this patch hasn't been merged yet. Other than re-basing > this to the
>>>> tip, anything
>>>> ZjQcmQRYFpfptBannerStart
>>>> This message was sent from outside of Texas Instruments.
>>>> Do not click links or open attachments unless you recognize the source
>>>> of this email and know the content is safe.
>>>> Report Suspicious
>>>> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!
>>>> updgnZav1ue7gKXOnHOpH5hg8gLmw9Osqsqh-
>>>> KTLRTnBEurV_VetrG7mzU898vB_5xFrYgbRkkuuFMBobM-uhFAmUgccRSk$>
>>>> ZjQcmQRYFpfptBannerEnd
>>>>
>>>> On 26/02/2025 07:18, Malladi, Meghana wrote:
>>>>> Hi all,
>>>>>
>>>>> Apologies in case you are receiving this email for the second time.
>>
>> BTW, Arnd when responded with Ack described the process.
>>
>
> Yes, I wanted to re-confirm before posting it again (just in case).
>
>>>>> Any reason why this patch hasn't been merged yet. Other than re-basing
>>>>> this to the tip, anything else which needs to be addressed to get this
>>>>> merged?
>>>>
>>>> I don't think you sent it to your platform maintainers, so no one would
>>>> apply it. Usually people ignore emails they did not receive, although I
>>>> am here an exception tracking missing-DT-patchwork things... :)
>>>>
>>>> Best regards,
>>>> Krzysztof
>>>>
>>>
>>> Hello Arnd/Krzysztof,
>>>
>>> Thank you for the replies :)
>>> Was having ambiguity since this is not TI specific change. Wanted to
>>> confirm before re-posting. Will post v2 patch including TI platform
>>> maintainers.
>>
>> It is not TI specific? Then which upstream boards benefit from this if
>> not TI?
>>
>
> This change enables generic HSR protocol support in the kernel which is
> required for HSR driver support in TI boards. When I said "not TI
> specific" - I meant this change is not internal to TI driver. Hope this
> clarifies the miscommunication from my earlier mail.
Yes, sure, but this defconfig change benefits and is sent for certain TI
upstream boards, so it is TI upstream maintainers field. It still fits
the "Submitting Patches for Given SoC" in SoC maintainer profile.
Best regards,
Krzysztof
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [EXTERNAL] Re: [PATCH] arm64: defconfig: Enable HSR driver
2025-02-26 12:06 ` Krzysztof Kozlowski
@ 2025-02-27 5:43 ` Malladi, Meghana
2025-02-27 7:32 ` Arnd Bergmann
0 siblings, 1 reply; 14+ messages in thread
From: Malladi, Meghana @ 2025-02-27 5:43 UTC (permalink / raw)
To: Krzysztof Kozlowski, Arnd Bergmann
Cc: quic_bjorande, linux-arm-kernel, linux-kernel, srk,
Anwar, Md Danish, catalin.marinas, will
On 2/26/2025 5:36 PM, Krzysztof Kozlowski wrote:
> On 26/02/2025 12: 54, Malladi, Meghana wrote: > Hi Krzysztof, > > On
> 2/26/2025 4: 18 PM, Krzysztof Kozlowski wrote: >> On 26/02/2025 11: 44,
> Malladi, Meghana wrote: > > > On 2/26/2025 1: 35 >> PM, Krzysztof Kozlowski
> ZjQcmQRYFpfptBannerStart
> This message was sent from outside of Texas Instruments.
> Do not click links or open attachments unless you recognize the source
> of this email and know the content is safe.
> Report Suspicious
> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!
> updgPX1FFo0b4GXEXdPDfwTBSgIATJM2ZirviWBzo9UAByA7dipk7rbevgVn7EQb6-
> DBhdHMpW7WhdxsrgH8RrHp6HnyASM$>
> ZjQcmQRYFpfptBannerEnd
>
> On 26/02/2025 12:54, Malladi, Meghana wrote:
>> Hi Krzysztof,
>>
>> On 2/26/2025 4:18 PM, Krzysztof Kozlowski wrote:
>>> On 26/02/2025 11: 44, Malladi, Meghana wrote: > > > On 2/26/2025 1: 35
>>> PM, Krzysztof Kozlowski wrote: >> On 26/02/2025 07: 18, Malladi, Meghana
>>> wrote: > Hi all, > > Apologies in >> case you are receiving this email
>>> ZjQcmQRYFpfptBannerStart
>>> This message was sent from outside of Texas Instruments.
>>> Do not click links or open attachments unless you recognize the source
>>> of this email and know the content is safe.
>>> Report Suspicious
>>> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!
>>> updq3RaPV007wGXFFrPD30z5a3GGYjpJweyhUcIfUTTaUs-8q0QpC4Yk9azBiFp7cBAf2Knk6fnVs4Fs-zRjTMEn8kmpiGo$>
>>> ZjQcmQRYFpfptBannerEnd
>>>
>>> On 26/02/2025 11:44, Malladi, Meghana wrote:
>>>>
>>>>
>>>> On 2/26/2025 1:35 PM, Krzysztof Kozlowski wrote:
>>>>> On 26/02/2025 07: 18, Malladi, Meghana wrote: > Hi all, > > Apologies in
>>>>> case you are receiving this email for the second time. > Any reason why
>>>>> this patch hasn't been merged yet. Other than re-basing > this to the
>>>>> tip, anything
>>>>> ZjQcmQRYFpfptBannerStart
>>>>> This message was sent from outside of Texas Instruments.
>>>>> Do not click links or open attachments unless you recognize the source
>>>>> of this email and know the content is safe.
>>>>> Report Suspicious
>>>>> <https://us-phishalarm-ewt.proofpoint.com/EWT/v1/G3vK!
>>>>> updgnZav1ue7gKXOnHOpH5hg8gLmw9Osqsqh-
>>>>> KTLRTnBEurV_VetrG7mzU898vB_5xFrYgbRkkuuFMBobM-uhFAmUgccRSk$>
>>>>> ZjQcmQRYFpfptBannerEnd
>>>>>
>>>>> On 26/02/2025 07:18, Malladi, Meghana wrote:
>>>>>> Hi all,
>>>>>>
>>>>>> Apologies in case you are receiving this email for the second time.
>>>
>>> BTW, Arnd when responded with Ack described the process.
>>>
>>
>> Yes, I wanted to re-confirm before posting it again (just in case).
>>
>>>>>> Any reason why this patch hasn't been merged yet. Other than re-basing
>>>>>> this to the tip, anything else which needs to be addressed to get this
>>>>>> merged?
>>>>>
>>>>> I don't think you sent it to your platform maintainers, so no one would
>>>>> apply it. Usually people ignore emails they did not receive, although I
>>>>> am here an exception tracking missing-DT-patchwork things... :)
>>>>>
>>>>> Best regards,
>>>>> Krzysztof
>>>>>
>>>>
>>>> Hello Arnd/Krzysztof,
>>>>
>>>> Thank you for the replies :)
>>>> Was having ambiguity since this is not TI specific change. Wanted to
>>>> confirm before re-posting. Will post v2 patch including TI platform
>>>> maintainers.
>>>
>>> It is not TI specific? Then which upstream boards benefit from this if
>>> not TI?
>>>
>>
>> This change enables generic HSR protocol support in the kernel which is
>> required for HSR driver support in TI boards. When I said "not TI
>> specific" - I meant this change is not internal to TI driver. Hope this
>> clarifies the miscommunication from my earlier mail.
>
> Yes, sure, but this defconfig change benefits and is sent for certain TI
> upstream boards, so it is TI upstream maintainers field. It still fits
> the "Submitting Patches for Given SoC" in SoC maintainer profile.
>
Okay, I got your point now. So as long as this defconfig benefits TI
SoCs, it should be sent with maintainer profile. I will keep that in
mind for future patches as well. Thanks for clarifying this.
Best Regards,
Meghana
> Best regards,
> Krzysztof
>
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [EXTERNAL] Re: [PATCH] arm64: defconfig: Enable HSR driver
2025-02-27 5:43 ` Malladi, Meghana
@ 2025-02-27 7:32 ` Arnd Bergmann
0 siblings, 0 replies; 14+ messages in thread
From: Arnd Bergmann @ 2025-02-27 7:32 UTC (permalink / raw)
To: Meghana Malladi, Krzysztof Kozlowski
Cc: Bjorn Andersson, linux-arm-kernel, linux-kernel, srk,
MD Danish Anwar, Catalin Marinas, Will Deacon
On Thu, Feb 27, 2025, at 06:43, Malladi, Meghana wrote:
> On 2/26/2025 5:36 PM, Krzysztof Kozlowski wrote:
>> On 26/02/2025 12: 54, Malladi, Meghana wrote: > Hi Krzysztof, > > On
>> 2/26/2025 4: 18 PM, Krzysztof Kozlowski wrote: >> On 26/02/2025 11: 44,
>>
>> Yes, sure, but this defconfig change benefits and is sent for certain TI
>> upstream boards, so it is TI upstream maintainers field. It still fits
>> the "Submitting Patches for Given SoC" in SoC maintainer profile.
>>
>
> Okay, I got your point now. So as long as this defconfig benefits TI
> SoCs, it should be sent with maintainer profile. I will keep that in
> mind for future patches as well. Thanks for clarifying this.
It's mainly for practical reasons: the patches I put into the soc
tree usually come from platform maintainers that I'm already working
with. They understand the process and I generally trust their
judgement of what should go into the kernel or not.
Individual contributors can still send patches to soc@lists.linux.dev
(formerly soc@kernel.org) and I will usually end up merging them, but
that means that the patch itself needs a more elaborate justification
for why it should be merged, and I end up spending a bit more time on
figuring out if I should take it.
Arnd
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2025-02-27 7:34 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2024-04-19 6:00 [PATCH] arm64: defconfig: Enable HSR driver Ravi Gunasekaran
2024-04-19 6:02 ` Arnd Bergmann
2024-04-19 6:25 ` Ravi Gunasekaran
2024-04-19 7:02 ` Arnd Bergmann
2025-02-26 6:14 ` Malladi, Meghana
2025-02-26 6:18 ` Malladi, Meghana
2025-02-26 7:45 ` Arnd Bergmann
2025-02-26 8:05 ` Krzysztof Kozlowski
2025-02-26 10:44 ` [EXTERNAL] " Malladi, Meghana
2025-02-26 10:48 ` Krzysztof Kozlowski
2025-02-26 11:54 ` Malladi, Meghana
2025-02-26 12:06 ` Krzysztof Kozlowski
2025-02-27 5:43 ` Malladi, Meghana
2025-02-27 7:32 ` Arnd Bergmann
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox