Messages in this thread Patch in this message | | | From | "Cao, Bingbu" <> | Subject | RE: [PATCH v1 4/8] media: ov2740: Remove duplicate check for NULL fwnode | Date | Wed, 27 Jul 2022 09:41:05 +0000 |
| |
Hi, Andy
Thanks for your patch.
Reviewed-by: Bingbu Cao <bingbu.cao@intel.com>
________________________ BRs, Bingbu Cao
> -----Original Message----- > From: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > Sent: Tuesday, July 26, 2022 8:06 PM > To: Andy Shevchenko <andriy.shevchenko@linux.intel.com>; linux- > media@vger.kernel.org; linux-kernel@vger.kernel.org > Cc: Qiu, Tian Shu <tian.shu.qiu@intel.com>; Tu, ShawnX > <shawnx.tu@intel.com>; Cao, Bingbu <bingbu.cao@intel.com>; Mauro Carvalho > Chehab <mchehab@kernel.org> > Subject: [PATCH v1 4/8] media: ov2740: Remove duplicate check for NULL > fwnode > > fwnode API does proper checks and returns correct codes, no need to repeat > it in the caller. > > Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com> > --- > drivers/media/i2c/ov2740.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/media/i2c/ov2740.c b/drivers/media/i2c/ov2740.c index > 9420258c6073..07fe8fda274e 100644 > --- a/drivers/media/i2c/ov2740.c > +++ b/drivers/media/i2c/ov2740.c > @@ -1009,9 +1009,6 @@ static int ov2740_check_hwcfg(struct device *dev) > int ret; > unsigned int i, j; > > - if (!fwnode) > - return -ENXIO; > - > ret = fwnode_property_read_u32(fwnode, "clock-frequency", &mclk); > if (ret) > return ret; > -- > 2.35.1
| |