usb: typec: ucsi: add support for Cypress CCGx

Hi Heikki and Andy
[...]

Quoted 14 lines

Quoted 14 lines

Quoted 1 line

Shouldn't you return -ETIMEDOUT if count == 0?

Yes. Good catch. Does the below fix looks ok?

        do {
                status = ccg_write(uc, CCGX_RAB_INTR_REG, &data, sizeof(data));
                if (status < 0)
                        return status;

                usleep_range(10000, 11000);

                status = ccg_read(uc, CCGX_RAB_INTR_REG, &data, sizeof(data));
                if (status < 0)
                        return status;

Quoted 5 lines

Quoted 3 lines

                if (!data)
                        return 0;
        } while (data && count--);

Doesn't that condition break out of the loop immediately?

How? I didn't get your point? We want to break out when data is zero (interrupt status cleared).