10 Apr, 2019
2 commits
-
Disable typec irq when suspend to avoid the threaded irq to access
some resource(e.g. i2c over rpmsg) but those resource is not
available at later phrase, also use IRQ_DISABLE_UNLAZY flag to
mask the irq on irq chip level when irq happens.Suggested-by: Anson Huang
Acked-by: Peter Chen
Signed-off-by: Li Jun
(cherry picked from commit 61869f787fb0ee2f00d0fe9443cb8b487e16e5ec) -
While system suspend, the typec event handling required service
maybe is not available(suspended), so we need freeze those event
handling by using freezable workqueue, e.g while tcpm is handling
PD message but system suspend started.Acked-by: Peter Chen
Signed-off-by: Li Jun
(cherry picked from commit 270d4df94b7c2c773a171fe012fb8ce89196964f)
12 Feb, 2019
38 commits
-
Coverity complains CID 3411292: Improper use of negative valure
(NEGATIVE_RETURNS), as possibly assign a negative value to unsigned
int, fix it by use an int temp value for negative value return.Acked-by: Peter Chen
Signed-off-by: Li Jun -
In case of vbus_never_low, if source setup a typye-c only session, sink
time out on waiting for source capability message and send soft reset,
the source does not response by goodcrc for the soft reset message, we
don't do hard reset, but keep a non-PD typec session, because the source
maybe PD capable, this hard reset will make the source terminate the
session by turn off vbus, if this type-c port is the system power supply,
the whole system will reboot.Acked-by: Peter Chen
Signed-off-by: Li Jun -
Commit 661b7ec2359e ("MLK-17921-2 staging: typec: tcpm: add qos for
PD transfer") introduce qos for tcpm transfer between typec controller
and CPU, but the request and remove are not balanced well, the initial
request should be moved to tcpm_port_register as tcpm_init will be called
for every hard reset, and also move qos hold&release to where port state
changes.Fixes: 661b7ec2359e ("MLK-17921-2 staging: typec: tcpm: add qos for PD transfer")
Acked-by: Peter Chen
Signed-off-by: Li Jun -
TCPCI based typec port controller needs very strict low latency
on i2c transfer, this patch is to improve this by adding qos and
high bus handling in tcpm during PD session setup.Acked-by: Peter Chen
Signed-off-by: Li Jun -
Current typec and PD properties parse can't work well for sink or source
only port, this patch is to improve it.Acked-by: Peter Chen
Signed-off-by: Li Jun -
Enable ion driver on Linux platform for Kernel 4.14.
Signed-off-by: Song Bing
-
In PTN5110 design, DisableSourceVBUS command also disables the sink
enable signal because the EN_SNK can be used to source higher voltage,
and, there is only one TCPC command to disable sourcing voltage without
telling whether to disable 5V or the high voltage, and to keep the
design simple they designed the PTN5110 to disable both. with this
fact, we use the flag drive_vbus to check if the source vbus enable was
issued, if yes we then do vbus source disable, in dead battery case,
we never did vbus source enable, so will not issue vbus source disable
command.Acked-by: Peter Chen
Signed-off-by: Li Jun -
Per tcpci spec, the TCPC_RX_BYTE_CNT is the number of bytes in the
RX_BUFFER_DATA_OBJECTS plus three (for the RX_BUF_FRAME_TYPE and
RX_BUF_HEADER), so after read out the header, we should only read
TCPC_RX_BYTE_CNT-3 bytes for data if this is a data message.Acked-by: Peter Chen
Signed-off-by: Li Jun -
We use vbus force discharge to have a quick vbus off for power role swap,
which works like this: enable vbus force discharge and wait the vbus fall
below vbus low threshold, when reaches, an alarm generated and tcpm can go
forward. but current code do vbus force discharge in any disable source
vbus case, in enable vbus charge case, we firstly disable source vbus and
then turn on vbus sink, in between, vbus force discharge should not be
enabled.Acked-by: Peter Chen
Signed-off-by: Li Jun -
If the vbus is aready on and remote cc state is Rp while typec init, we
think it's a dead battery case, this needs the PD session already setup
by bootloader, so kernel can negotiate a new power session by soft reset,
this patch use the exsiting flag vbus_never_low as boot from dead battery
flag, but update the condition of setting it: not only check vbus, also
the cc status to make sure remote is a power source, if yes, bypass the
vbus sink disable. If the vbus is from local, we will still disable vbus
charge so original code intention is kept.Acked-by: Peter Chen
Signed-off-by: Li Jun -
Otherwise, the probe would be considered successfully, and without
do any destroy jobs, eg, the interrupt has still registered.Acked-by: Jun Li
Signed-off-by: Peter Chen -
Adding fixed sink power settings for sink-disable case, which
is only for PD protocol talk to know the cc orientation if connects
to a PD capable host, HW doesn't really sink any power in this case.Tested-by: Peter Chen
Signed-off-by: Li Jun -
As GPIO normally can be system wakeup source, so enable typec events
as system wakeup source if user requested.Acked-by: Peter Chen
Signed-off-by: Li Jun -
It's missing while do typec related properties read, sink
needs it to check if the provided PDO can match the required
power.Acked-by: Peter Chen
Signed-off-by: Li Jun -
We use vbus low voltage alarm to start vbus discharge to meet
timing requirement on turning off vbus for power swap from
source to sink, per type-C port controller spec(tcpci), the
Voltage Alarms Power status reporting is disabled by default,
so we need enable it at tcpci init.Acked-by: Peter Chen
Signed-off-by: Li Jun -
For vbus change event, we need read the vbus status to clear
the alert. Current code do this in queue work, this has problem
on single core running, the queue work of vbus change may have
no chance to be scheduled as we continue receive the vbus change
event in threaded irq.Acked-by: Peter Chen
Signed-off-by: Li Jun -
As we need to use DRP config for data role, but the power role is source only,
so introduce a property sink-disable to avoid sink vbus command.Signed-off-by: Li Jun
Acked-by: Peter Chen -
This will allow __drm_mode_object_file to be extended to perform
access control checks based on the file in use.v2: Also fix up vboxvideo driver in staging
[airlied: merging early as this is an API change]
Suggested-by: Daniel Vetter
Signed-off-by: Keith Packard
Signed-off-by: Dave Airlie
(cherry picked from commit 418da17214aca5ef5f0b6f7588905ee7df92f98f) -
Change to use the vbus from partner to notify EXTCON_USB.
This is to work around the case of source only typec port
connecting to Host PC via a Rp fixed cable.Acked-by: Peter Chen
Signed-off-by: Li Jun -
Some usb device driver can't know the connect and disconnect to host
if the vbus is always on, if use typec we can rely on cc line status
to know that, so add a notification to let controller driver know
device attach and detach from host.Acked-by: Peter Chen
Signed-off-by: Li Jun -
Some USB3 differential channel switch chips need to do reset before
functional, we add this support here.Signed-off-by: Peter Chen
Acked-by: Li Jun -
The tcpci_irq may visit tcpci->port if unexpected interrupt occurs
(eg, board design issue causes GPIO status is incorrect) and cause
NULL pointer dereference issue.Besides, delete clear TCPC_ALERT and TCPC_ALERT_MASK code which are
already done at tcpci_init.Signed-off-by: Peter Chen
Acked-by: Li Jun -
Instead of fixed pull up super speed mux selection gpio for cc1, use
GPIO_ACTIVE_HIGH/LOW to map the CC1/CC2 orientation via gpiod api, So
for ss-sel-gpios:
GPIO_ACTIVE_HIGH: CC1 GPIO high
GPIO_ACTIVE_LOW : CC1 GPIO lowAcked-by: Peter Chen
Signed-off-by: Li Jun -
As the default data role is device mode if the port is not host,
so set the port data role to be device mode after src detach, this
is to fix the issue of port data role still kept to be host while
the port is open.Acked-by: Peter Chen
Signed-off-by: Li Jun -
Use extcon device for dual role switch on typec port.
Acked-by: Peter Chen
Signed-off-by: Li Jun -
VBus off only means the vbus falls to be below 4v, we can use vbus
force discharge and vbus low alarm to go forward.Reviewed-by: Peter Chen
Signed-off-by: Li Jun -
We should wait at least tErrorRecovery before move to unattached state.
Reviewed-by: Peter Chen
Signed-off-by: Li Jun -
CC line changes on souce side RP value is permitted per standard, in
this case we should move to SNK_ATTACH_WAIT.Reviewed-by: Peter Chen
Signed-off-by: Li Jun -
If we received unsupported BIST command, we should by pass it and ready
to accept following command for test, not unconditionly to set the port
to be unattached state and terminate the test.Reviewed-by: Peter Chen
Signed-off-by: Li Jun -
To early start sink power after source turn off vbus, move the set power
role after vbus is turned off in PR_SWAP_SRC_SNK_SOURCE_OFF, also per
spec, we should do error recovery if the sink side does not turn on
vbus in PD_T_PS_SOURCE_ON.Reviewed-by: Peter Chen
Signed-off-by: Li Jun -
In SNK_HARD_RESET_SINK_ON, if the source disconnected, we should set
state to be unattached state intead of startup sink.Reviewed-by: Peter Chen
Signed-off-by: Li Jun -
Per standard, we should clear the hard reset count to be 0 after souce
sent the source cap.Reviewed-by: Peter Chen
Signed-off-by: Li Jun -
If the power source sent soft reset and the other end accepted, we should
not reset the message id as it was already updated by pd_transmit when
handling the soft reset.Reviewed-by: Peter Chen
Signed-off-by: Li Jun -
Instead of static config, use dt to pass basic PD config for
below settings:
- Source pdo list
- Sink pdo list
- Max sink voltage
- Max sink current
- Sink operating power
- TypeC port type
- TypeC port preferred roleReviewed-by: Peter Chen
Signed-off-by: Li Jun -
According to the polarity of cc line, we need to do USB3 signal mux
selection if it's controlled by software.Reviewed-by: Peter Chen
Signed-off-by: Li Jun -
This patch is to update several PD timings for PD compliance, all
those timing definitions should be configurable via user config,
this can be improved later, for now I just update its values in PD
spec defined range.Reviewed-by: Peter Chen
Signed-off-by: Li Jun -
Per PD spec, the sender response should be in a small range 24~30ms, so
correct the PD_T_SENDER_RESPONSE to be 25.Reviewed-by: Peter Chen
Signed-off-by: Li Jun -
As the sender response timer have very small margin(25~30ms), so
use a hrtimer to handle it.Reviewed-by: Peter Chen
Signed-off-by: Li Jun
