Messages in this thread | | | Subject | Re: [PATCH v3 02/10] ata: pata_platform: Drop use of unlikely() in pata_platform_probe | From | Sergey Shtylyov <> | Date | Fri, 24 Dec 2021 20:54:28 +0300 |
| |
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@bp.renesas.com> > --- > 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
| |