lkml.org 
[lkml]   [2023]   [Nov]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v13 05/10] usb: dwc3: qcom: Refactor IRQ handling in QCOM Glue driver
From
> 
>> I didn't add missing interrupts on sc8280xp because I see that current
>> interrupts present are working fine (I see ADB working and wakeup
>> working as well), but the interrupt vector numbers are off by "1"
>> between hs specifics and DT (both upstream and downstream). Will sort it
>> out and clean that target up later.
>
> Which interrupt numbers are off by one here?
>

My bad, this might be the confusion. The HW specifics say:

Controller-2, power_event irq:

SYS_apcsQgicSPI[812] Vector-number: 843


Usually vector number = 32 + GIC number AFAIK.
By that logic, If vector number is 843, GIC_SPI number is 811 which is
same as DT. Probably the GIC_SPI number is printed wrong. The DT matches
(vector number - 32).

Sorry for mentioning that it is wrong. The DT entries are right and it
is working on upstream.

The missing hs_phy_irq's have been put on the mail thread on this list
before.

Regards,
Krishna,

>> [1]: https://patchwork.kernel.org/project/linux-arm-msm/list/?series=803412
>
> I took a quick look at the series, and it looks like this will
> eventually clean things up a lot. We should probably define a generic
> order for the interrupts with the sometimes optional SS interrupts last.
>
> Side note: It looks like the threading in that series is broken.
> Consider using git-send-email for sending series as it takes care of
> things like that.
>

Usually I do git send-email for the whole out folder where the patches
are present, but linux-usb list is common to all the patches in that
case, even the DT ones. So to avoid that and to send patches to only
relavant mailing lists, I did git send email individually on each patch
which might have caused this issue.

Will make sure this won't happen again.

Regards,
Krishna,

\
 
 \ /
  Last update: 2023-11-24 10:02    [W:0.143 / U:0.080 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site