On Tue, Jun 08, 2021 at 05:47:47AM CDT, Andrew Jeffery wrote: >Strengthen the distinction between code that abstracts the >implementation of the KCS behaviours (device drivers) and code that >exploits KCS behaviours (clients). Neither needs to know about the APIs >required by the other, so provide separate headers. > >Signed-off-by: Andrew Jeffery <andrew@xxxxxxxx> >--- > drivers/char/ipmi/kcs_bmc.c | 23 ++++++++++------ > drivers/char/ipmi/kcs_bmc.h | 27 +++++++++---------- > drivers/char/ipmi/kcs_bmc_aspeed.c | 17 ++++++------ > drivers/char/ipmi/kcs_bmc_cdev_ipmi.c | 39 ++++++++++++++++++--------- > drivers/char/ipmi/kcs_bmc_client.h | 27 +++++++++++++++++++ > drivers/char/ipmi/kcs_bmc_device.h | 19 +++++++++++++ > drivers/char/ipmi/kcs_bmc_npcm7xx.c | 17 ++++++------ > 7 files changed, 117 insertions(+), 52 deletions(-) > create mode 100644 drivers/char/ipmi/kcs_bmc_client.h > create mode 100644 drivers/char/ipmi/kcs_bmc_device.h > Reviewed-by: Zev Weiss <zweiss@xxxxxxxxxxx>