# \[PATCH\] ata: remove redundant error print in rb532\_pata\_driver\_probe - Subsystems: LIBATA SUBSYSTEM (Serial and Parallel ATA drivers), THE REST - Maintainers: Damien Le Moal, Niklas Cassel, Linus Torvalds - List: linux-ide - 6 messages, 2 authors - Page: https://ratatoskr.run/linux-ide/2021/01/6231039/t ## \[PATCH\] ata: remove redundant error print in rb532\_pata\_driver\_probe - From: \ - Date: 2021-01-12 02:37:32 - Message: https://ratatoskr.run/linux-ide/2021/01/6231039.md ```text From: Menglong Dong Coccinelle reports a redundant error print in rb532_pata_driver_probe. As 'platform_get_irq' already prints the error message, error print here is redundant and can be removed. Signed-off-by: Menglong Dong --- drivers/ata/pata_rb532_cf.c | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/drivers/ata/pata_rb532_cf.c b/drivers/ata/pata_rb532_cf.c index 479c4b29b856..dcde84f571c4 100644 --- a/drivers/ata/pata_rb532_cf.c +++ b/drivers/ata/pata_rb532_cf.c @@ -115,10 +115,8 @@ static int rb532_pata_driver_probe(struct platform_device *pdev) } irq = platform_get_irq(pdev, 0); - if (irq <= 0) { - dev_err(&pdev->dev, "no IRQ resource found\n"); + if (irq <= 0) return -ENOENT; - } gpiod = devm_gpiod_get(&pdev->dev, NULL, GPIOD_IN); if (IS_ERR(gpiod)) { -- 2.17.1 ``` ## Re: \[PATCH\] ata: remove redundant error print in rb532\_pata\_driver\_probe - From: Sergei Shtylyov \ - Date: 2021-01-12 11:16:24 - Message: https://ratatoskr.run/linux-ide/2021/01/6231471.md ```text Hello! On 1/12/21 5:36 AM, menglong8.dong@gmail.com wrote: > From: Menglong Dong > > Coccinelle reports a redundant error print in rb532_pata_driver_probe. > As 'platform_get_irq' already prints the error message, error print > here is redundant and can be removed. > > Signed-off-by: Menglong Dong > --- > drivers/ata/pata_rb532_cf.c | 4 +--- > 1 file changed, 1 insertion(+), 3 deletions(-) > > diff --git a/drivers/ata/pata_rb532_cf.c b/drivers/ata/pata_rb532_cf.c > index 479c4b29b856..dcde84f571c4 100644 > --- a/drivers/ata/pata_rb532_cf.c > +++ b/drivers/ata/pata_rb532_cf.c > @@ -115,10 +115,8 @@ static int rb532_pata_driver_probe(struct platform_device *pdev) > } > > irq = platform_get_irq(pdev, 0); > - if (irq <= 0) { > - dev_err(&pdev->dev, "no IRQ resource found\n"); > + if (irq <= 0) > return -ENOENT; This still beaks the probe deferral. :-( But that's another problem... [...] MBR, Sergei ``` ## Re: \[PATCH\] ata: remove redundant error print in rb532\_pata\_driver\_probe - From: Menglong Dong \ - Date: 2021-01-13 14:06:09 - Message: https://ratatoskr.run/linux-ide/2021/01/6233438.md ```text Hello, Sergei On Tue, Jan 12, 2021 at 7:15 PM Sergei Shtylyov [ref] wrote: > > Hello! > > On 1/12/21 5:36 AM, menglong8.dong@gmail.com wrote: > [....] > > irq = platform_get_irq(pdev, 0); > > - if (irq <= 0) { > > - dev_err(&pdev->dev, "no IRQ resource found\n"); > > + if (irq <= 0) > > return -ENOENT; > > This still beaks the probe deferral. :-( > But that's another problem... > > [...] > > MBR, Sergei What does this 'MBR' mean? I am a novice~~~ So, is it better to replace 'platform_get_irq' with 'platform_get_irq_optional' here? -- Best Regards Menglong Dong ``` ## Re: \[PATCH\] ata: remove redundant error print in rb532\_pata\_driver\_probe - From: Sergei Shtylyov \ - Date: 2021-01-14 08:31:04 - Message: https://ratatoskr.run/linux-ide/2021/01/6234333.md ```text Hello! On 13.01.2021 17:04, Menglong Dong wrote: [...] >>> irq = platform_get_irq(pdev, 0); >>> - if (irq <= 0) { >>> - dev_err(&pdev->dev, "no IRQ resource found\n"); >>> + if (irq <= 0) >>> return -ENOENT; >> >> This still beaks the probe deferral. :-( >> But that's another problem... >> >> [...] >> >> MBR, Sergei > > What does this 'MBR' mean? I am a novice~~~ Generally speaking, Master Boot Record. But I also use it to send you My Best Regards. :-) > So, is it better to replace 'platform_get_irq' with > 'platform_get_irq_optional' here? No. You should stop overriding the result to -ENOENT and pass the result up the call chain instead. In order to do it, you should only check for (irq < 0). > -- > Best Regards > Menglong Dong MBR, Sergei ``` ## Re: \[PATCH\] ata: remove redundant error print in rb532\_pata\_driver\_probe - From: Menglong Dong \ - Date: 2021-01-14 14:54:57 - Message: https://ratatoskr.run/linux-ide/2021/01/6234749.md ```text On Thu, Jan 14, 2021 at 4:30 PM Sergei Shtylyov [ref] wrote: [...] > > > > What does this 'MBR' mean? I am a novice~~~ > > Generally speaking, Master Boot Record. But I also use it to send you My > Best Regards. :-) Haha~, > > So, is it better to replace 'platform_get_irq' with > > 'platform_get_irq_optional' here? > > No. You should stop overriding the result to -ENOENT and pass the result > up the call chain instead. In order to do it, you should only check for (irq < 0). Well, I didn't even notice this. It does seem to be another problem... --- Best Regards Menglong Dong ``` ## Re: \[PATCH\] ata: remove redundant error print in rb532\_pata\_driver\_probe - From: Sergei Shtylyov \ - Date: 2021-01-14 15:50:34 - Message: https://ratatoskr.run/linux-ide/2021/01/6234836.md ```text On 1/12/21 2:15 PM, Sergei Shtylyov wrote: [...] >> From: Menglong Dong >> >> Coccinelle reports a redundant error print in rb532_pata_driver_probe. >> As 'platform_get_irq' already prints the error message, error print >> here is redundant and can be removed. >> >> Signed-off-by: Menglong Dong >> --- >> drivers/ata/pata_rb532_cf.c | 4 +--- >> 1 file changed, 1 insertion(+), 3 deletions(-) >> >> diff --git a/drivers/ata/pata_rb532_cf.c b/drivers/ata/pata_rb532_cf.c >> index 479c4b29b856..dcde84f571c4 100644 >> --- a/drivers/ata/pata_rb532_cf.c >> +++ b/drivers/ata/pata_rb532_cf.c >> @@ -115,10 +115,8 @@ static int rb532_pata_driver_probe(struct platform_device *pdev) >> } >> >> irq = platform_get_irq(pdev, 0); >> - if (irq <= 0) { >> - dev_err(&pdev->dev, "no IRQ resource found\n"); >> + if (irq <= 0) >> return -ENOENT; > > This still beaks the probe deferral. :-( > But that's another problem... BTW, your patch summary looks quite wrong -- we need the prefix to specify the patch locus, like this: pata_rb532_cf: remove redundant error printing in the probe() method I.e. "ata: " may even be omitted (your choise though). > [...] MBR, Sergei ```