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. [...] MBR, Sergey