Hi Aseda:
Thanks for your reply.
On Wed, Jun 26, 2024 at 7:55 AM Aseda Aboagye <aaboagye@xxxxxxxxxxxx> wrote:
>
> Hi Binbin,
>
> On Mon, Jun 24, 2024 at 02:53:59PM +0800, Binbin Zhou wrote:
> > +
> > +static void pixart_reset(struct psmouse *psmouse)
> > +{
> > + ps2_command(&psmouse->ps2dev, NULL, PSMOUSE_CMD_RESET_DIS);
> > + msleep(100);
> > + psmouse_reset(psmouse);
> > +}
>
> What is the reason for the 100ms delay here? Is it to prepare the
> controller for the upcoming reset? If so, could it perhaps be
> defined as a macro?
>
Yes, the 100ms is required for the upcoming reset.
And we just use it in this one place, do we really have to define it as a macro?
Thanks.
Binbin
> Thanks,
>
> --
> Aseda Aboagye
