Received-SPF: pass (google.com: domain of linux-kernel+bounces-195602-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Message-ID: <223d9482-ede8-4f58-b11a-d82f2556c26b@ti.com>
Date: Thu, 30 May 2024 10:31:20 -0500
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH v2 8/8] arm64: defconfig: Enable TI eQEP Driver
From: Judith Mendez <jm@ti.com>
To: Nishanth Menon <nm@ti.com>, <devicetree@vger.kernel.org>
CC: Rob Herring <robh@kernel.org>,
        Krzysztof Kozlowski
	<krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>,
        Catalin Marinas <catalin.marinas@arm.com>,
        Will Deacon <will@kernel.org>,
        William Breathitt Gray <william.gray@linaro.org>,
        David Lechner
	<david@lechnology.com>,
        <linux-arm-kernel@lists.infradead.org>, <linux-kernel@vger.kernel.org>,
        <linux-iio@vger.kernel.org>, Vignesh
 Raghavendra <vigneshr@ti.com>
References: <20240523231516.545085-1-jm@ti.com>
 <20240523231516.545085-9-jm@ti.com>
 <20240524055923.zyfthiwcsbwbjg5k@undecided>
 <ddcab8c0-3772-4134-a3bb-27729e864627@ti.com>
Content-Language: en-US
In-Reply-To: <ddcab8c0-3772-4134-a3bb-27729e864627@ti.com>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: 8bit

Hi Nishanth,

On 5/24/24 9:13 AM, Judith Mendez wrote:
> Hi Nishanth,
> 
> On 5/24/24 12:59 AM, Nishanth Menon wrote:
>> On 18:15-20240523, Judith Mendez wrote:
>>> TI K3 SoC's support eQEP hardware, so enable TI eQEP driver
>>> to be built as a module.
>>
>> All the nodes seem to be only in disabled mode, is there even a single
>> board that is actually using this? if so, why isn't it enabled?
> 
> I will add pinmux and enable in the board level dts file.

Question about the above... If I enable eQEP at the board level,
there will be pinmux conflicts. So, I could:
1. Leave eQEP disabled
2. Enable eQEP and disable main_uart1 at the board level dts file
3. Enable eQEP in an overlay and also disable main_uart1

Let me know which option works for you.

~ Judith

> 
>>
>>>
>>> Signed-off-by: Judith Mendez <jm@ti.com>
>>> ---
>>> Changes since v1:
>>> - No change
>>> ---
>>>   arch/arm64/configs/defconfig | 1 +
>>>   1 file changed, 1 insertion(+)
>>>
>>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
>>> index 2c30d617e1802..23d11a1b20195 100644
>>> --- a/arch/arm64/configs/defconfig
>>> +++ b/arch/arm64/configs/defconfig
>>> @@ -1593,6 +1593,7 @@ CONFIG_INTERCONNECT_QCOM_SM8550=y
>>>   CONFIG_INTERCONNECT_QCOM_SM8650=y
>>>   CONFIG_INTERCONNECT_QCOM_X1E80100=y
>>>   CONFIG_COUNTER=m
>>> +CONFIG_TI_EQEP=m
>>>   CONFIG_RZ_MTU3_CNT=m
>>>   CONFIG_HTE=y
>>>   CONFIG_HTE_TEGRA194=y
>>> -- 
>>> 2.45.1
>>>
>>
> 
>