Re: [PATCH] ipmi: bt-bcm: Use platform_get_irq_optional()

From: Markus Elfring
Date: Fri Apr 03 2020 - 15:03:20 EST


> In this function's reference function 'bt_bmc_probe()',

I have got understanding difficulties for this information.
Would you like to refer to the function âbt_bmc_config_irqâ?


> there are judgments of print message about 'bt_bmc->irq',

Would an other wording be nicer?


> so use platform_get_irq_optional() instead of platform_get_irq()
> to avoid redundant dev_err() message.

Will a separate sentence be helpful?

How do you think about another bit of fine-tuning for the commit message?

Regards,
Markus