On 10.8.2016 10:08, [email protected] wrote: > From: Zhouyi Zhou <[email protected]> > > return value of class_create should be considered in > module init function. > > Signed-off-by: Zhouyi Zhou <[email protected]> > --- > drivers/char/xilinx_hwicap/xilinx_hwicap.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/char/xilinx_hwicap/xilinx_hwicap.c > b/drivers/char/xilinx_hwicap/xilinx_hwicap.c > index c07dfe5..3877b46 100644 > --- a/drivers/char/xilinx_hwicap/xilinx_hwicap.c > +++ b/drivers/char/xilinx_hwicap/xilinx_hwicap.c > @@ -854,6 +854,9 @@ static int __init hwicap_module_init(void) > int retval; > > icap_class = class_create(THIS_MODULE, "xilinx_config"); > + if (IS_ERR(icap_class)) > + return PTR_ERR(icap_class); > + > mutex_init(&icap_sem); > > devt = MKDEV(XHWICAP_MAJOR, XHWICAP_MINOR); >
Acked-by: Michal Simek <[email protected]> Thanks, Michal
