Hi Sergey, Thank you for the review. On Fri, Dec 24, 2021 at 5:54 PM Sergey Shtylyov <s.shtylyov@xxxxxx> wrote: > > On 12/24/21 4:12 PM, Lad Prabhakar wrote: > > > pata_platform_probe() isn't a hotpath, which makes it's questionable to > > use unlikely(). Therefore let's simply drop it. > > > > Signed-off-by: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > > --- > > v2-->v3 > > * New patch > > --- > > drivers/ata/pata_platform.c | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/drivers/ata/pata_platform.c b/drivers/ata/pata_platform.c > > index cb3134bf88eb..29902001e223 100644 > > --- a/drivers/ata/pata_platform.c > > +++ b/drivers/ata/pata_platform.c > > @@ -199,14 +199,14 @@ static int pata_platform_probe(struct platform_device *pdev) > > * Get the I/O base first > > */ > > io_res = platform_get_mem_or_io(pdev, 0); > > - if (unlikely(!io_res)) > > + if (!io_res) > > return -EINVAL; > > > > /* > > * Then the CTL base > > */ > > ctl_res = platform_get_mem_or_io(pdev, 1); > > - if (unlikely(!ctl_res)) > > + if (!ctl_res) > > return -EINVAL; > > I think you should combine this with patch #1. > I'd like to keep the changes separate from patch #1, as it's unrelated. Cheers, Prabhakar