Use dev_err() and dev_dbg()

Instead of our printk() helpers.
Remove IRQF_ONESHOT from irq_flags

And set it when we make the devm_request_threaded_irq() call instead.
Don't negate I2C_M_RD, it's not a boolean
Report short-reads from goodix_i2c_read()
Synchronise the device in goodix_ts_irq_handler()

Rather than using a label in goodix_ts_work_func()
Return the actual error when input_register_device() fails
use if (!foo) instead of if (foo == NULL)
Trust I2C to be correct for goodix_i2c_test()
Remove disable_irq_nosync() call

It's re-enabled moments after.
Move goodix_request_irq() into goodix_ts_probe()

It's a single function call.
Use if (ret < 0) to check for devm_request_threaded_irq() ret
Use cpu_to_be16() in goodix_i2c_read()
Use cpu_to_be16() in goodix_i2c_write()