Re: [PATCH v4 2/2] pinctrl: qcom: Add MSM8952 tlmm pinctrl driver
From: Konrad Dybcio
Date: Mon Sep 07 2026 - 09:45:54 EST
On 9/7/26 3:32 PM, Muzaffer Kadir wrote:
> Hi Konrad,
>
> On Mon, 7 Sep 2026 14:04:59 +0200, Konrad Dybcio wrote:
>> On 04-Sep-26 14:55, Muzaffer Kadir via B4 Relay wrote:
>> I told the robot to take a look again (and again and again
>> a couple more times until it stopped complaining)..
>>
>> The following feedback is meaningful:
>>
>> Hi Muzaffer,
>>
>> `qdss_tracedata_b_groups` advertises GPIOs that cannot actually select
>> the function. Please populate the corresponding PINGROUP slots:
>>
>> GPIO8: mux 11
>> GPIO9: mux 12
>> GPIO12: mux 12
>> GPIO13: mux 12
>
> I will add these to their pins by sending v5.
>
> But also I want to ask,
> does atest_combodac_to_gpio_native
> have any mux on gpio12/13/20/21/29/30 ?Or are they wrongly set ?
Yes they do, the robot pointed that out too but I figured it pulled it
from the internal data and not from the existing driver. They don't
really have much of a use outside validation testing I believe, but
since they're there already:
pin func_idx
12 14
13 14
20 11
21 11
29 10
30 11
Konrad