Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <3362316a-dc24-e2e2-4baa-69e51703e86e@ti.com>
Date:   Mon, 12 Jun 2023 17:25:44 +0530
MIME-Version: 1.0
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.11.0
Subject: Re: [EXTERNAL] Re: [PATCH] arm64: defconfig: Enable HSR by default
Content-Language: en-US
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        <catalin.marinas@arm.com>, <will@kernel.org>
CC:     <arnd@arndb.de>, <linux-arm-kernel@lists.infradead.org>,
        <linux-kernel@vger.kernel.org>
References: <20230612094340.13865-1-r-gunasekaran@ti.com>
 <ab8dfd0f-fa49-6493-d79d-c35f632373f3@linaro.org>
 <e3e409c5-56b5-570d-e962-a7c0a1c05fa3@ti.com>
 <3ac37e54-0633-37d2-2ba7-f06abcb9a5a7@linaro.org>
From:   Ravi Gunasekaran <r-gunasekaran@ti.com>
In-Reply-To: <3ac37e54-0633-37d2-2ba7-f06abcb9a5a7@linaro.org>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
Precedence: bulk


On 6/12/23 3:21 PM, Krzysztof Kozlowski wrote:
> On 12/06/2023 11:49, Ravi Gunasekaran wrote:
>>
>>
>> On 6/12/23 3:14 PM, Krzysztof Kozlowski wrote:
>>> On 12/06/2023 11:43, Ravi Gunasekaran wrote:
>>>> Enable HSR feature in kernel build.
>>>
>>> Why? Commit msg should answer to this question.
>>
>> I apologize for incomplete description. I will send out a v2.
>>
>> HSR protocol is implemented by the network stack at /net/hsr/*
>> In order to create HSR interface, HSR module needs to be loaded.
> 
> You described what HSR is, but I want to know why do we need it in our
> development arm64 kernel. You might want to enable anything and explain
> what is this "anything" but this does not solve any of the concerns.
> 

TI SoCs such as AM64x, AM65x support multiple ethernet ports which can be
used to setup HSR (a redundancy protocol) node in the network. So enabling
the HSR driver to be built as module. The HSR node can be created using a
pair of ethernet ports in the system.

> Best regards,
> Krzysztof
> 

-- 
Regards,
Ravi