On Tue, Dec 1, 2020 at 2:59 PM Guido Günther <[email protected]> wrote: > This allows downstream supplies and userspace to detect whether external power > is supplied. > > The Librem 5 has the tp65982 in front of bq25980 charge controller. Since > that > is capable of sinking and sourcing power the online property helps to decide > what to do. It also makes upower happy. > > There will be follow up patches providing more properties but these need some > more time to cook and i wanted to check if this is the right way to go?
>From my perspective the patches are okay (after addressing one more comment), FWIW Reviewed-by: Andy Shevchenko <[email protected]> > changes from v3 > - As per review comments from Andy Shevchenko > > https://lore.kernel.org/linux-usb/CAHp75VeLZtm85Y=3qmkpgb332wn05-zr-_mrrwxvnqlhazr...@mail.gmail.com/ > - Use positive conditionals > - Add reviewed by from Heikki Krogerus > > https://lore.kernel.org/linux-usb/[email protected]/T/#u > > https://lore.kernel.org/linux-usb/[email protected]/T/#u > - Fix typc vs typec typo in commit message > > changes from v2 > - As per kernel test robot > https://lore.kernel.org/linux-usb/[email protected]/ > - Flip USB_ROLE_SWITCH and REGMAP_I2C from 'depends on' to 'select' > This matches tcpm and avoids a config symbol recursion which went > unnoticed on my arm64 build but trips up x86_64. > > changes from v1 > - As per review comments from Heikki Krogerus > > https://lore.kernel.org/linux-usb/[email protected]/ > - select POWER_SUPPLY > - use POWER_SUPPLY_USB_TYPE_PD when a PD contract got negotiated > > To: Heikki Krogerus <[email protected]>,Greg Kroah-Hartman > <[email protected]>,[email protected],[email protected],Andy > Shevchenko <[email protected]> > > Guido Günther (2): > usb: typec: tps6598x: Select USB_ROLE_SWITCH and REGMAP_I2C > usb: typec: tps6598x: Export some power supply properties > > drivers/usb/typec/Kconfig | 5 +- > drivers/usb/typec/tps6598x.c | 105 +++++++++++++++++++++++++++++++++++ > 2 files changed, 108 insertions(+), 2 deletions(-) > > -- > 2.29.2 > -- With Best Regards, Andy Shevchenko
