No need to have an id_table

We are ACPI enumerated, so the id_table is not required.
Use get_unaligned_le16 instead of manually parsing the field

We have some nice conveninent functions, use them.

"buf[0] | (buf[1] << 8)" is equivalent to "get_unaligned_le16(&buf[0])"
Remove useless symbol GTP_DEBUG
Do not initialize variables

So the compiler can notice us of uninitialized uses.
Fix some whitespaces damages

checkpatch is much happier
msleep(10) is useless

checkpatch complains about:
WARNING: msleep < 20ms can sleep for up to 20ms;
see Documentation/timers/timers-howto.txt

Use a 20 ms sleep then
Fix 80 columns warnings

Checkpatch is happy now :)
Do not use s32, but native int

fix also useless snprintf
The IRQ flags array is a mess use a static const definitiion
Remove unused defines
Remove useless msleep
Rename gtp_init_panel to gtp_read_config

There is no more initialization of the panel in this function.
Rename it
Obsolete Goodix copyright and set our

The code has been enough rewritten for not having to care about
Goodix authorship IMO.