Re: [PATCH] pinctrl: single: Add compatible for ti,am625-padconf

From: Tony Lindgren
Date: Mon Aug 07 2023 - 03:07:36 EST


* Nishanth Menon <nm@xxxxxx> [230805 17:15]:
> On 10:25-20230805, Dhruva Gole wrote:
> > From: Tony Lindgren <tony@xxxxxxxxxxx>
> > +static const struct pcs_soc_data pinctrl_single_am625 = {
> > + .flags = PCS_QUIRK_SHARED_IRQ | PCS_CONTEXT_LOSS_OFF,
> > + .irq_enable_mask = (1 << 29), /* WKUP_EN */
> > + .irq_status_mask = (1 << 30), /* WKUP_EVT */
> > +};
> > +
>
> Why cant we set this in the k3-pinctrl.h and set it once?

Good idea to define the bit offsets k3-pinctrl.h instead of magic numbers
here :)

> The event will not be generated until wakeup daisy chain is triggered
> anyways.

Yup, and having that happen is enough to show the wake-up reason with
grep wakeup /proc/interrupts :)

> Have you looked at all the padconf registers across devices to ensure
> the WKUP_EN/EVT bits are present? daisy chain feature is used elsewhere
> as well.

The lack of bits at least earlier just meant that attempting to use a
wake-up interrupt would just never trigger. Worth checking though.
Dhruva, care to check if some padconf register have reserved bits for
29 and 30 that might be set high by default?

Regards,

Tony