[PATCH 0/6] platform/chrome: get rid of BUG_ON()

Tzung-Bi Shih posted 6 patches 3 years, 12 months ago
There is a newer version of this series
drivers/platform/chrome/cros_ec_i2c.c   | 12 ++++++++++--
drivers/platform/chrome/cros_ec_ishtp.c |  4 +++-
drivers/platform/chrome/cros_ec_lpc.c   |  2 ++
drivers/platform/chrome/cros_ec_proto.c | 13 ++++++++-----
drivers/platform/chrome/cros_ec_rpmsg.c |  2 ++
drivers/platform/chrome/cros_ec_spi.c   | 19 ++++++++++++++-----
6 files changed, 39 insertions(+), 13 deletions(-)
[PATCH 0/6] platform/chrome: get rid of BUG_ON()
Posted by Tzung-Bi Shih 3 years, 12 months ago
The series gets rid of BUG_ON()s in drivers/platform/chrome/.  Most of them
can be replaced by returning proper return code.

The 2nd patch makes callers of cros_ec_prepare_tx() to take care of the
return code.

The 3rd patch turns cros_ec_prepare_tx() to return error code if any.

Tzung-Bi Shih (6):
  platform/chrome: cros_ec_proto: drop unneeded BUG_ON() in
    prepare_packet()
  platform/chrome: correct cros_ec_prepare_tx() usage
  platform/chrome: cros_ec_proto: drop BUG_ON() in cros_ec_prepare_tx()
  platform/chrome: cros_ec_proto: drop BUG_ON() in
    cros_ec_get_host_event()
  platform/chrome: cros_ec_i2c: drop BUG_ON() in cros_ec_pkt_xfer_i2c()
  platform/chrome: cros_ec_spi: drop BUG_ON()

 drivers/platform/chrome/cros_ec_i2c.c   | 12 ++++++++++--
 drivers/platform/chrome/cros_ec_ishtp.c |  4 +++-
 drivers/platform/chrome/cros_ec_lpc.c   |  2 ++
 drivers/platform/chrome/cros_ec_proto.c | 13 ++++++++-----
 drivers/platform/chrome/cros_ec_rpmsg.c |  2 ++
 drivers/platform/chrome/cros_ec_spi.c   | 19 ++++++++++++++-----
 6 files changed, 39 insertions(+), 13 deletions(-)

-- 
2.36.0.512.ge40c2bad7a-goog