diff options
| author | Dan Carpenter <dan.carpenter@oracle.com> | 2020-12-03 11:40:48 +0300 |
|---|---|---|
| committer | Patrik Jakobsson <patrik.r.jakobsson@gmail.com> | 2020-12-03 10:50:57 +0100 |
| commit | 15ccc39b3aab667c6fa131206f01f31bfbccdf6a (patch) | |
| tree | 7289c9411370744ef5efa186be57ce5b29cec667 /scripts/checkincludes.pl | |
| parent | e4401247070a37c2fce62b2773a4eb7757983938 (diff) | |
| download | linux-15ccc39b3aab667c6fa131206f01f31bfbccdf6a.tar.bz2 | |
gma500: clean up error handling in init
The main problem with this error handling was that it didn't clean up if
i2c_add_numbered_adapter() failed. This code is pretty old, and doesn't
match with today's checkpatch.pl standards so I took the opportunity to
tidy it up a bit. I changed the NULL comparison, and removed the
WARNING message if kzalloc() fails and updated the label names.
Fixes: 1b082ccf5901 ("gma500: Add Oaktrail support")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Patrik Jakobsson <patrik.r.jakobsson@gmail.com>
Link: https://patchwork.freedesktop.org/patch/msgid/X8ikkAqZfnDO2lu6@mwanda
Diffstat (limited to 'scripts/checkincludes.pl')
0 files changed, 0 insertions, 0 deletions