On Mon, 22 Aug 2022 at 06:36, Igor Kononenko <[email protected]> wrote: > > The previos v1 [PATCH 1/3] have error of getting `struct kcs_bmc_device` > pointer from `struct platform_device *`. Replaced to retriveing pointer by > `platform_get_drvdata()`
Can we get a v3 with your original commit message added? You had a good write up in v1 but it was dropped in v2. This change looks like the right thing to do. We should get Andrew to review too, as he spends more time with the KCS controllers. The missed irq issue was seen with the other LPC sub drivers because the clock wasn't enabled. We ended up with this patch: https://lore.kernel.org/all/[email protected]/ Do we need to do something similar for KCS? > > Reported-by: kernel test robot <[email protected]> > Signed-off-by: Igor Kononenko <[email protected]> > --- > drivers/char/ipmi/kcs_bmc_aspeed.c | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/drivers/char/ipmi/kcs_bmc_aspeed.c > b/drivers/char/ipmi/kcs_bmc_aspeed.c > index cdc88cde1e9a..8437f3cfe3f4 100644 > --- a/drivers/char/ipmi/kcs_bmc_aspeed.c > +++ b/drivers/char/ipmi/kcs_bmc_aspeed.c > @@ -636,6 +636,14 @@ static int aspeed_kcs_remove(struct platform_device > *pdev) > return 0; > } > > +static void aspeed_kcs_shutdown(struct platform_device *pdev) > +{ > + struct aspeed_kcs_bmc *priv = platform_get_drvdata(pdev); > + struct kcs_bmc_device *kcs_bmc = &priv->kcs_bmc; > + > + aspeed_kcs_irq_mask_update(kcs_bmc, (KCS_BMC_EVENT_TYPE_IBF), 0); > +} > + > static const struct of_device_id ast_kcs_bmc_match[] = { > { .compatible = "aspeed,ast2400-kcs-bmc-v2" }, > { .compatible = "aspeed,ast2500-kcs-bmc-v2" }, > @@ -651,6 +659,7 @@ static struct platform_driver ast_kcs_bmc_driver = { > }, > .probe = aspeed_kcs_probe, > .remove = aspeed_kcs_remove, > + .shutdown = aspeed_kcs_shutdown, > }; > module_platform_driver(ast_kcs_bmc_driver); > > -- > 2.25.1 > _______________________________________________ Openipmi-developer mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/openipmi-developer
