lkml.org 
[lkml]   [2018]   [Apr]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
Subject[PATCH v8 0/6] typec: tcpm: Add sink side support for PPS
This patch set adds sink side support for the PPS feature introduced in the
USB PD 3.0 specification.

The source PPS supply is represented using the Power Supply framework to provide
access and control APIs for dealing with it's operating voltage and current,
and switching between a standard PDO and PPS APDO operation. During standard PDO
operation the voltage and current is read-only, but for APDO PPS these are
writable as well to allow for control.

It should be noted that the keepalive for PPS is not handled within TCPM. The
expectation is that the external user will be required to ensure re-requests
occur regularly to ensure PPS remains and the source does not hard reset.


Changes in v8:
- Rebase onto latest 'usb-next' commit
(b462e2e0d62a716f7a1b7a7ecea966edc3de45d7) in Greg's USB repo. This picks
up Jun Li's changes to how PDOs are chosen.
- Use of '__maybe_unused' in patch 01 (core support), for new APIs that are not
used until patch 05 (power_supply interface), to avoid warnings
(-Wunused-function) when applying that patch in isolation. '__maybe_unused'
usage is removed in patch 05 as it's no longer necessary.

Changes in v7:
- Further tidy up with bracket usage and unwanted initialisation.
- Stop using else if statement after break.
- Added NULL checking of psy_name after devm_kzalloc().
- Reinstate PD_ROLE_SWAP_TIMEOUT and revert role swap functions to use this.
- Add PD_PPS_CTRL_TIMEOUT for PPS related control functions.

Changes in v6:
- Remove unnecessary use of 'data' variable and associated kzalloc/kfree call
for extended message handling.
- Add patch for error checking psy_desc struct in psy register code.
- Add error checking of usb_type property in psy register code.
- Cosmetic () and initialisation changes as requested by Guenter.
- Removed Acked-by and Reviewed-by, from Heikki and Sebastian respectively, on
patch 04 as there have been changes to error handling with regards to
usb_type, so didn't feel appropriate to keep them.

Changes in v5:
- Rebase on branch with 'Revert "typec: tcpm: Only request matching pdos"' and
header changes already included.
- Update power_supply registration to make power_supply names unique per port,
to avoid errors creating duplicate psy instances. New name uses port
dev name as a suffix.
- Renamed 'connected_type' psy property to 'usb_type', as requested by
maintainer.
- Added initial attempt at generic ABI documentation for common psy class
properties for Battery and USB type supplies.
- Small update to PPS APDO selection code to limit maximum current requested
based on sink maximum allowed current. Have left Heikki's 'Acked-by' tag as
it's a minor change, but can remove if that's not deemed appropriate.

Changes in v4:
- For PD 3.0 definitions patch, make it benign with regards to existing TCPM
code so build isn't broken if this one patch is applied, as suggested by
kbuild robot. Update for dynamic revision is moved to be part of sink side
PPS support patch.
- Use PTR_ERR_OR_ZERO macro to simplify return of devm_tcpm_psy_register()
function, as suggested by kbuild robot.
- Make devm_tcpm_psy_register() static as not used outside this file.

Changes in v3:
- Drop 'RFC' from patch series titles
- Rename PPS related defines to be PPS specific rather than generic APDO titles
- Update source caps logging to only print PPS APDOs, and for others report as
undefined.
- Add ABI documentation for tcpm-source-psy sysfs properties
- Rebase PDO selection on top of 'typec: tcpm: Only request matching pdos'
patch.
- Update capabilities validation introduced in
'typec: tcpm: Validate source and sink caps' to support PPS APDOs.
- Dropped power_supply 'type' property update for PPS addition
- Added 'connected_type' property to power_supply framework, to support
supplies which can report multiple connected types (e.g. USB), as discussed
with Heikki.

Changes in v2:
- Use USB_PD and usb_pd prefixes for macros and inline functions in headers.
- Negotiate spec revision of PD headers during initial contract agreement.
- New headers now use SPDX tags for referencing correct license.

Adam Thomson (6):
typec: tcpm: Add core support for sink side PPS
Documentation: power: Initial effort to document power_supply ABI
power: supply: Add error checking of psy desc during registration
power: supply: Add 'usb_type' property and supporting code
typec: tcpm: Represent source supply through power_supply
typec: tcpm: Add support for sink PPS related messages

Documentation/ABI/testing/sysfs-class-power | 455 +++++++++++++
MAINTAINERS | 1 +
drivers/power/supply/power_supply_core.c | 11 +-
drivers/power/supply/power_supply_sysfs.c | 45 ++
drivers/usb/typec/Kconfig | 1 +
drivers/usb/typec/fusb302/Kconfig | 2 +-
drivers/usb/typec/fusb302/fusb302.c | 63 +-
drivers/usb/typec/tcpm.c | 955 +++++++++++++++++++++++++++-
include/linux/power_supply.h | 16 +
include/linux/usb/pd.h | 4 +-
include/linux/usb/tcpm.h | 1 +
11 files changed, 1466 insertions(+), 88 deletions(-)

--
1.9.1

\
 
 \ /
  Last update: 2018-04-23 16:15    [W:0.111 / U:0.300 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site