mirror of
https://github.com/nxp-imx/linux-imx.git
synced 2025-07-19 07:39:54 +02:00
Revert "media: tuners: fix error return code of hybrid_tuner_request_state()"
commite25cc4be46
upstream. This reverts commitb9302fa7ed
. As Fedor Pchelkin pointed out, this commit violates the convention of using the macro return value, which causes errors. For example, in functions tda18271_attach(), xc5000_attach(), simple_tuner_attach(). Link: https://lore.kernel.org/linux-media/20240424202031.syigrtrtipbq5f2l@fpc/ Suggested-by: Fedor Pchelkin <pchelkin@ispras.ru> Signed-off-by: Roman Smirnov <r.smirnov@omp.ru> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
This commit is contained in:
parent
a4c2fbed20
commit
e48edd4762
|
@ -133,10 +133,8 @@ static inline int tuner_i2c_xfer_send_recv(struct tuner_i2c_props *props,
|
|||
} \
|
||||
if (0 == __ret) { \
|
||||
state = kzalloc(sizeof(type), GFP_KERNEL); \
|
||||
if (!state) { \
|
||||
__ret = -ENOMEM; \
|
||||
if (NULL == state) \
|
||||
goto __fail; \
|
||||
} \
|
||||
state->i2c_props.addr = i2caddr; \
|
||||
state->i2c_props.adap = i2cadap; \
|
||||
state->i2c_props.name = devname; \
|
||||
|
|
Loading…
Reference in New Issue
Block a user