Re: [PATCH v2 2/2] Input: isa1200 - new driver for Imagis ISA1200

[Svyatoslav Ryhel]

вт, 5 трав. 2026 р. о 13:39 Linus Walleij <linusw@xxxxxxxxxx> пише:
>
> On Thu, Apr 30, 2026 at 11:46 AM Svyatoslav Ryhel <clamor95@xxxxxxxxx> wrote:
> > чт, 30 квіт. 2026 р. о 12:22 Linus Walleij <linusw@xxxxxxxxxx> пише:
>
> > In you original code
> > /*
> > * This is done in the vendor tree with the commment
> > * "Duty 0x64 == nForce 90", and no force feedback happens
> > * unless we do this.
> > */
> > if (isa->clk)
> > regmap_write(isa->map, ISA1200_HCTRL5, 0x64);
> >
> > 0x64 is actually some conversion of duty cycle, you got this accurately.
>
> I wonder if 0x64 = 100 is simply 100% duty cycle?
> So the value in this register can only be 0..100.
>

Assumption is interesting, but it is still just a guess based on
observation of a single hw configuration. For example, Samsung Galaxy
Tab 10.1 (GT-P7501) passes 133 (0x85) into duty register.

> Yours,
> Linus Walleij