On Fri, Sep 07, 2018 at 05:06:12PM +0300, Heikki Krogerus wrote:
> On Fri, Sep 07, 2018 at 06:35:12AM -0700, Guenter Roeck wrote:
> > On 09/07/2018 05:56 AM, Heikki Krogerus wrote:
> > > Moving all the drivers that depend on the Port Controller
> > > Manager under a new
On 09/07/2018 05:56 AM, Heikki Krogerus wrote:
Moving all the drivers that depend on the Port Controller
Manager under a new a new directory drivers/usb/typec/tcpci/
and making Guenter Roeck as the designated reviewer of that
code.
Signed-off-by: Heikki Krogerus
---
Hi guys,
This should be
On 09/07/2018 03:34 AM, Heikki Krogerus wrote:
+Guenter
On Thu, Sep 06, 2018 at 01:26:44PM -0600, Angus Ainslie (Purism) wrote:
If the board is being powered by USB disabling the source and sink
can remove power from the board. Default to source and sink enabled.
Seems to me that might viola
On 08/13/2018 03:36 AM, Heiko Stuebner wrote:
Hi,
I'm currently trying to wrap my head around the new typec subsystem and
also how to do it correctly on Rockchip rk3399 devices.
The issue (and Guenter might know quite a bit about that) is that on
ChromeOS devices the embedded controller hides t
On Wed, Aug 01, 2018 at 11:37:56AM -0700, Tim Harvey wrote:
> On Tue, Jul 31, 2018 at 2:22 AM Adam Thomson
> wrote:
> >
> > On 27 July 2018 17:41, Tim Harvey wrote:
> >
> > Adding Guenter to the thread.
> >
> > > Greetings,
> > >
> > > I have a custom design with a Fairchild FUSB302 Type-C chip dr
Cc: Adam Thomson
Signed-off-by: Hans de Goede
Reviewed-by: Guenter Roeck
---
drivers/i2c/busses/i2c-cht-wc.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/i2c/busses/i2c-cht-wc.c b/drivers/i2c/busses/i2c-cht-wc.c
index 44cffad43701..c4d176f5ed79 100644
---
which now already sets these to 0.
Fixes: f2a8aa053c17 ("typec: tcpm: Represent source supply through...")
Cc: Adam Thomson
Signed-off-by: Hans de Goede
Reviewed-by: Guenter Roeck
---
drivers/usb/typec/tcpm.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --
On 07/01/2018 05:27 AM, Hans de Goede wrote:
The set_current_limit tcpm_dev callback is optional and the tcpm core
will already log the passed in values, so there is no need to have an
empty implementation of this.
Signed-off-by: Hans de Goede
Reviewed-by: Guenter Roeck
---
drivers/usb
On Fri, Jun 29, 2018 at 12:52:34PM +0800, kbuild test robot wrote:
> tree: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git
> usb-next
> head: aaf3f4e925dc2bdc4715142103660285632a245c
> commit: 990da41530b31ed7de29340ce1d78c04bee9670c [36/38] staging: typec:
> tcpci: move tcpci
On 06/26/2018 04:45 PM, Li Jun wrote:
Move TCPCI(Typec port controller interface) driver and rt1711h
driver out of staging.
Signed-off-by: Li Jun
Reviewed-by: Guenter Roeck
---
drivers/staging/Kconfig| 2 --
drivers/staging/Makefile | 1
On 06/26/2018 04:45 PM, Li Jun wrote:
While set polarity, we should keep the disconnected cc line to be
open.
Reviewed-by: Heikki Krogerus
Signed-off-by: Li Jun
Reviewed-by: Guenter Roeck
---
drivers/staging/typec/tcpci.c | 18 ++
1 file changed, 14 insertions(+), 4
On 06/26/2018 04:45 PM, Li Jun wrote:
From: Peter Chen
With that we can clear any pending events and the port is registered
so driver can be ready to handle typec events once we request irq.
Reviewed-by: Heikki Krogerus
Signed-off-by: Peter Chen
Signed-off-by: Li Jun
Reviewed-by: Guenter
On 06/26/2018 04:45 PM, Li Jun wrote:
As tcpm_register_port() and tcpci_register_port() never return
NULL and NULL is not a success in this case, use IS_ERR() to check
the return value of both.
Reviewed-by: Heikki Krogerus
Signed-off-by: Li Jun
Reviewed-by: Guenter Roeck
---
drivers
On 06/26/2018 04:45 PM, Li Jun wrote:
Since we will use config settings via device properties, so
remove the hard code tcpci_tcpc_config.
Reviewed-by: Heikki Krogerus
Signed-off-by: Li Jun
Reviewed-by: Guenter Roeck
---
drivers/staging/typec/tcpci.c | 7 ---
1 file changed, 7
On 06/26/2018 04:45 PM, Li Jun wrote:
Add fwnode handle to get the fwnode so we can get typec configs
it contains.
Suggested-by: Heikki Krogerus
Reviewed-by: Heikki Krogerus
Signed-off-by: Li Jun
Reviewed-by: Guenter Roeck
---
drivers/staging/typec/tcpci.c | 7 +++
drivers/usb
yet.
Reviewed-by: Heikki Krogerus
Signed-off-by: Li Jun
Reviewed-by: Guenter Roeck
---
drivers/staging/typec/tcpci.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/staging/typec/tcpci.c b/drivers/staging/typec/tcpci.c
index 076d97e..dd29288 100644
--- a/drivers/st
to use log buffer,
the index has already updated by the 1st process, so it will get
NULL pointer for updated logbuffer, the error message like below:
tcpci 0-0050: Log buffer index 6 is NULL
Cc: Heikki Krogerus
Cc: Guenter Roeck
Cc: Jun Li
Signed-off-by: Peter Chen
Reviewed-by: Guen
On 06/11/2018 05:29 AM, Heikki Krogerus wrote:
On Mon, May 28, 2018 at 10:52:44AM +0800, Li Jun wrote:
In case of drp toggling, we may need set correct cc value for role control
after attach as it may never been set.
Is this something that should be considered as a fix?
The problem with thi
On 05/27/2018 07:52 PM, Li Jun wrote:
This patch adds support of get typec and power delivery config from
firmware description.
Signed-off-by: Li Jun
Except for the nitpick. on top of Heikki's comments:
Reviewed-by: Guenter Roeck
---
drivers/usb/typec/tcpm.c
On Thu, Jun 7, 2018 at 6:17 AM Dan Carpenter wrote:
>
> The tcpm_set_state() function take msecs not jiffies.
>
Nice catch.
Reviewed-by: Guenter Roeck
> Fixes: f0690a25a140 ("staging: typec: USB Type-C Port Manager (tcpm)")
> Signed-off-by: Dan Carpenter
>
>
was
created, as the code should work properly either way.
Cc: Heikki Krogerus
Cc: Guenter Roeck
Cc: Hans de Goede
Cc: Andy Shevchenko
Cc: Adam Thomson
Signed-off-by: Greg Kroah-Hartman
Reviewed-by: Guenter Roeck
---
v2: changes based on review from Guenter, we still need to keep
On 05/30/2018 03:36 AM, Greg Kroah-Hartman wrote:
On Wed, May 30, 2018 at 01:30:20PM +0300, Heikki Krogerus wrote:
+Guenter
On Tue, May 29, 2018 at 05:30:46PM +0200, Greg Kroah-Hartman wrote:
When calling debugfs functions, there is no need to ever check the
return value. The function can wor
meday...
Yes. I'll send a patch.
Cc: Heikki Krogerus
Signed-off-by: Greg Kroah-Hartman
Revieved-by: Heikki Krogerus
Reviewed-by: Guenter Roeck
---
drivers/usb/typec/tcpm.c | 11 +++
1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/drivers/usb/typec/tcpm.c
On Tue, May 29, 2018 at 06:27:03PM +0200, Greg Kroah-Hartman wrote:
> On Tue, May 29, 2018 at 09:15:30AM -0700, Guenter Roeck wrote:
> > On Tue, May 29, 2018 at 05:30:47PM +0200, Greg Kroah-Hartman wrote:
> > > When calling debugfs functions, there is no need to ever check the
o only work for one device per system,
> otherwise the debugfs directories are going to get confused when a
> device is removed.
>
> Cc: Heikki Krogerus
> Cc: Guenter Roeck
> Cc: Hans de Goede
> Cc: Andy Shevchenko
> Cc: Adam Thomson
> Signed-off-by: Greg Kroah-H
This fixes an issue where the driver can't be re-loaded if
it has been unloaded as the "fusb302" debugfs directory
already exists.
Fixes: 76f0c53d08b9 ("usb: typec: fusb302: Move out of staging")
Signed-off-by: Heikki Krogerus
Reviewed-by: Guenter Roeck
---
driv
g the 'power-supplies' DT property, but in the
> future should also work for ACPI platforms when the relevant support
> is added to the power_supply core.
>
> Signed-off-by: Adam Thomson
> Suggested-by: Heikki Krogerus
Reviewed-by: Guenter Roeck
> ---
> drivers/u
On 05/11/2018 06:18 AM, Heikki Krogerus wrote:
Hi,
This is the third version of my proposal for more complete alternate
mode support. In this version I'm including a proposal for the mux
handling. Basically, I'm proposing that every supported alternate will
have its own mux handle. That should a
eviewed-by: Guenter Roeck
---
drivers/usb/typec/tcpm.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/usb/typec/tcpm.c b/drivers/usb/typec/tcpm.c
index 1ee259bc14a5..66dc0773b9bf 100644
--- a/drivers/usb/typec/tcpm.c
+++ b/drivers/usb/typec/tcpm.c
@@ -4652,6 +4652
On 04/30/2018 05:41 AM, Heikki Krogerus wrote:
Removing the "fusb302" debugfs directory when unloading
the driver. That allows the driver to be loaded more then
ones.
This fixes an issue where the driver, if unloaded, can't be
re-loaded, as the "fusb302" debugfs directory already
exists. While h
On 04/24/2018 04:46 AM, Heikki Krogerus wrote:
On Mon, Apr 23, 2018 at 09:43:57AM -0700, Guenter Roeck wrote:
On Mon, Apr 23, 2018 at 11:03:09AM +0300, Heikki Krogerus wrote:
On Fri, Apr 20, 2018 at 10:26:08AM -0700, Guenter Roeck wrote:
On Wed, Apr 18, 2018 at 03:34:09PM +0300, Heikki
On Mon, Apr 23, 2018 at 11:03:09AM +0300, Heikki Krogerus wrote:
> On Fri, Apr 20, 2018 at 10:26:08AM -0700, Guenter Roeck wrote:
> > On Wed, Apr 18, 2018 at 03:34:09PM +0300, Heikki Krogerus wrote:
> > > If the I2C adapter that the PD controller is attached to
> >
ata));
> + if (ret)
> + return ret;
> +
Sanity check ?
if (data[0] != len)
return -Esomething;
Other than that,
Reviewed-by: Guenter Roeck
> + memcpy(val, &data[1], len);
> + return 0;
> +}
> +
> static inline int tps6598
ble to get the return value of tcpm_pd_select_pdo()
> in patch 1/5, tested on my platform.
> - Add document in fusb302_composite_snk_pdo_array() to explain
> why use deprecated properties in patch 2/5.
>
For the series:
Reviewed-by: Guenter Roeck
> Change for v4:
> - Add Hans's
n
>
> I don't see any problem with this, but I hope Guenter has time to
> check it.
>
I don't see anything wrong, but then the changes are so complex that
I can only hope they were well tested. Unfortunately, I won't have
time to do such testing myself.
A
On 04/20/2018 03:51 AM, Jun Li wrote:
-Original Message-
From: Hans de Goede [mailto:hdego...@redhat.com]
Sent: 2018年4月20日 17:21
To: Jun Li ; li...@roeck-us.net;
heikki.kroge...@linux.intel.com
Cc: gre...@linuxfoundation.org; linux-usb@vger.kernel.org; dl-linux-imx
Subject: Re: [PATCH
On 04/13/2018 08:42 PM, Jun Li wrote:
Hi Guenter,
-Original Message-
From: Guenter Roeck [mailto:groe...@gmail.com] On Behalf Of Guenter Roeck
Sent: 2018年4月14日 11:03
To: Jun Li
Cc: gre...@linuxfoundation.org; robh...@kernel.org;
heikki.kroge...@linux.intel.com; hdego...@redhat.com; rmf
On 04/13/2018 04:31 PM, Jun Li wrote:
-Original Message-
From: Guenter Roeck [mailto:li...@roeck-us.net]
Sent: 2018年4月14日 0:04
To: Jun Li
Cc: gre...@linuxfoundation.org; robh...@kernel.org;
heikki.kroge...@linux.intel.com; hdego...@redhat.com; rmf...@gmail.com;
yueyao@gmail.com
On Fri, Apr 13, 2018 at 10:18:41PM +0800, Li Jun wrote:
> This patch is a combination of commit 57e6f0d7b804
> ("typec: tcpm: Only request matching pdos") and source
> pdo selection optimization based on it, instead of only
> compare between the same pdo type of sink and source,
> we should check s
On Fri, Apr 13, 2018 at 10:18:42PM +0800, Li Jun wrote:
> Since max_snk_* is to be deprecated, so remove max_snk_* by adding a
> variable PDO for sink config.
max_snk or max_sink ?
>
> Reviewed-by: Hans de Goede
> Signed-off-by: Li Jun
> ---
> drivers/usb/typec/fusb302/fusb302.c | 34
On Fri, Apr 13, 2018 at 10:18:43PM +0800, Li Jun wrote:
> Remove max-sink-* properties since they are deprecated.
>
> Reviewed-by: Rob Herring
> Reviewed-by: Hans de Goede
> Signed-off-by: Li Jun
> ---
> Documentation/devicetree/bindings/usb/fcs,fusb302.txt | 6 --
> 1 file changed, 6 dele
On 03/28/2018 09:06 AM, Li Jun wrote:
While set polarity, we should keep the not connecting cc line to be
open.
The more I look at this code, the more I am confused by it.
The original code doesn't touch the CC lines. This function only sets the
polarity.
Is it really appropriate to touch th
On Thu, Mar 29, 2018 at 12:06:15AM +0800, Li Jun wrote:
> In case of drp toggling, we may need set correct cc value for role control
> after attach as it may never been set.
>
Isn't CC set by the lower level driver in this case ? In other words, is it ever
necessary to call back into the low leve
andle messagescreceived but not
yet handled.
Signed-off-by: Adam Thomson
Acked-by: Heikki Krogerus
Reviewed-by: Guenter Roeck
---
drivers/usb/typec/tcpm.c | 143 ---
1 file changed, 134 insertions(+), 9 deletions(-)
diff --git a/drivers/usb/
ypec: tcpm: Represent source supply through power_supply class'
Signed-off-by: Adam Thomson
Reviewed-by: Guenter Roeck
---
drivers/usb/typec/Kconfig | 1 +
drivers/usb/typec/fusb302/Kconfig | 2 +-
drivers/usb/typec/fusb302/fusb302.c | 63 +-
drivers/u
: Heikki Krogerus
Reviewed-by: Guenter Roeck
---
drivers/usb/typec/tcpm.c | 517 ++-
include/linux/usb/pd.h | 4 +-
include/linux/usb/tcpm.h | 1 +
3 files changed, 509 insertions(+), 13 deletions(-)
diff --git a/drivers/usb/typec/tcpm.c b
On Thu, Mar 22, 2018 at 03:52:07PM +, Adam Thomson wrote:
> This commit adds sink side support for Get_Status, Status,
> Get_PPS_Status and PPS_Status handling. As there's the
> potential for a partner to respond with Not_Supported,
> handling of this message is also added. Sending of
> Not_Sup
On Thu, Mar 22, 2018 at 03:52:06PM +, Adam Thomson wrote:
> This commit adds a power_supply class instance to represent a
> PD source's voltage and current properties. This provides an
> interface for reading these properties from user-space or other
> drivers.
>
> For PPS enabled Sources, thi
On Thu, Mar 22, 2018 at 03:52:02PM +, Adam Thomson wrote:
> This commit adds code to handle requesting of PPS APDOs. Switching
> between standard PDOs and APDOs, and re-requesting an APDO to
> modify operating voltage/current will be triggered by an
> external call into TCPM.
>
> Signed-off-by
On 03/22/2018 03:40 AM, Adam Thomson wrote:
On 22 March 2018 04:09, Guenter Roeck wrote:
+static int tcpm_psy_set_prop(struct power_supply *psy,
+enum power_supply_property psp,
+const union power_supply_propval *val)
+{
+ struct
On 03/20/2018 07:33 AM, Adam Thomson wrote:
This commit adds a power_supply class instance to represent a
PD source's voltage and current properties. This provides an
interface for reading these properties from user-space or other
drivers.
For PPS enabled Sources, this also provides write access
On 03/20/2018 07:33 AM, Adam Thomson wrote:
This commit adds the 'usb_type' property to represent USB supplies
which can report a number of different types based on a connection
event.
Examples of this already exist in drivers whereby the existing 'type'
property is updated, based on an event, t
On 03/20/2018 07:33 AM, Adam Thomson wrote:
This commit adds code to handle requesting of PPS APDOs. Switching
between standard PDOs and APDOs, and re-requesting an APDO to
modify operating voltage/current will be triggered by an
external call into TCPM.
Signed-off-by: Adam Thomson
Acked-by: He
On 03/20/2018 07:33 AM, Adam Thomson wrote:
This commit adds sink side support for Get_Status, Status,
Get_PPS_Status and PPS_Status handling. As there's the
potential for a partner to respond with Not_Supported,
handling of this message is also added. Sending of
Not_Supported is added to handle
On Mon, Mar 19, 2018 at 11:49:35AM +0800, ShuFan Lee wrote:
> From: ShuFan Lee
>
> Add device tree binding document for Richtek RT1711H Type-C chip driver
Cc: Rob Herring , devicet...@vger.kernel.org
is missing. Added here but it might make sense to resend.
>
> Signed-off-by: ShuFan Lee
> ---
igned-off-by: ShuFan Lee
No additional feedback on top of Heikki's comments.
As we get more drivers, it might make sense to provide an
API function for rt1711h_check_revision(), or even have
tcpci_register_port() validate vendor and product IDs.
But I think for now we are good
On 03/16/2018 10:52 PM, 李書帆 wrote:
Hi Guenter,
2018-03-17 10:16 GMT+08:00 Guenter Roeck :
On 03/16/2018 10:40 AM, ShuFan Lee wrote:
From: ShuFan Lee
Richtek RT1711H Type-C chip driver that works with
Type-C Port Controller Manager to provide USB PD and
USB Type-C functionalities.
Add
On 03/16/2018 10:40 AM, ShuFan Lee wrote:
From: ShuFan Lee
Richtek RT1711H Type-C chip driver that works with
Type-C Port Controller Manager to provide USB PD and
USB Type-C functionalities.
Add definition of TCPC_CC_STATUS_TOGGLING.
Signed-off-by: ShuFan Lee
Does this patch require DT revi
On Fri, Mar 09, 2018 at 06:19:17PM +0300, Heikki Krogerus wrote:
> Introducing a simple bus for the alternate modes. Bus allows
> binding drivers to the discovered alternate modes the
> partners support.
>
> Signed-off-by: Heikki Krogerus
> ---
> Documentation/ABI/obsolete/sysfs-class-typec | 4
On Fri, Mar 09, 2018 at 06:19:18PM +0300, Heikki Krogerus wrote:
> This adds more complete handling of VDMs and registration of
> partner alternate modes, and introduces callbacks for
> alternate mode operations.
>
> Only DFP role is supported for now.
>
> Signed-off-by: Heikki Krogerus
> ---
>
On Tue, Mar 13, 2018 at 05:34:33PM +0800, Li Jun wrote:
> With that we can clear any pending events and the port is registered
> so driver can be ready to handle typec events once we request irq.
>
> Signed-off-by: Peter Chen
> Signed-off-by: Li Jun
> ---
> drivers/staging/typec/tcpci.c | 15 ++
beginnng of the function. Any thoughts ?
Otherwise
Reviewed-by: Guenter Roeck
> ---
> drivers/staging/typec/tcpci.c | 6 ++
> 1 file changed, 6 insertions(+)
>
> diff --git a/drivers/staging/typec/tcpci.c b/drivers/staging/typec/tcpci.c
> index f5a3bf5..9a230c6 100644
On Tue, Mar 13, 2018 at 05:34:37PM +0800, Li Jun wrote:
> We need regmap_update_bits to avoid touch any other bits when
> enable or disable vconn.
>
> Signed-off-by: Li Jun
Reviewed-by: Guenter Roeck
> ---
> drivers/staging/typec/tcpci.c | 9 +++--
> 1 file change
On Tue, Mar 13, 2018 at 05:34:35PM +0800, Li Jun wrote:
> As DRP port autonomously toggles the Rp/Rd need a start value to
> begin with, so add one parameter for it in tcpm_start_drp_toggling.
>
> Signed-off-by: Li Jun
Reviewed-by: Guenter Roeck
> ---
> drivers/usb
ECTION command
>
> Signed-off-by: ShuFan Lee
I am fine with this version.
Reviewed-by: Guenter Roeck
> ---
> drivers/staging/typec/tcpci.c | 127
> +-
> drivers/staging/typec/tcpci.h | 15 +
> 2 files changed, 116 insertions(+),
>
> In order to allow the port drivers to describe the
> capabilities of the ports more clearly according to the
> newest specifications, introducing separate definitions for
> the data roles.
>
> Signed-off-by: Heikki Krogerus
Reviewed-by: Guenter Roec
o reason why a fixed source_cap cannot be matched against
a variable snk_cap, so at a minimum the commit should be rewritten to
support that.
For now lets revert the "typec: tcpm: Only request matching pdos" commit,
fixing the regression.
Cc: Badhri Jagan Sridharan
Signed-off-by: Hans de
On 03/04/2018 08:16 PM, ShuFan Lee wrote:
From: ShuFan Lee
Handle vendor defined behavior in tcpci_init, tcpci_set_vconn,
tcpci_start_drp_toggling
and export tcpci_irq. More operations can be extended in tcpci_data if needed.
According to TCPCI specification, 4.4.5.2 ROLE_CONTROL,
TCPC shall n
-by: Guenter Roeck
---
Changes in v2:
-Added Heikki's Reviewed-by
---
drivers/usb/typec/Kconfig | 1 +
drivers/usb/typec/fusb302/fusb302.c | 1 -
drivers/usb/typec/tcpm.c| 46 -
include/linux/usb/tcpm.h| 10 ---
reason for taking over control of the muxes for example
is because handling them in the port drivers would be just
boilerplate.
Signed-off-by: Heikki Krogerus
Reviewed-by: Hans de Goede
Signed-off-by: Hans de Goede
Reviewed-by: Guenter Roeck
---
Changes in v2:
-Add IS_ERR_OR_NULL() checks to the
the mux and switch
do _not_ stay in their last mode after a detach.
Reviewed-by: Heikki Krogerus
Signed-off-by: Hans de Goede
Reviewed-by: Guenter Roeck
---
Changes in v2:
-Added Heikki's Reviewed-by
---
drivers/usb/typec/tcpm.c | 22 +++---
include/linux/usb/tcpm.h
On Wed, Feb 21, 2018 at 11:02:23PM +0800, ShuFanLee wrote:
> From: ShuFanLee
>
> Handle vendor defined behavior in tcpci_init, tcpci_set_vconn and export
> tcpci_irq.
> More operations can be extended in tcpci_data if needed.
> According to TCPCI specification, 4.4.5.2 ROLE_CONTROL,
> TCPC shall
On Wed, Feb 14, 2018 at 06:08:29PM +0200, Andy Shevchenko wrote:
> ...instead of open coding file operations followed by custom ->open()
> callbacks per each attribute.
>
> Cc: Heikki Krogerus
> Cc: Guenter Roeck
> Signed-off-by: Andy Shevchenko
Reviewed-by: Guenter Roe
On 02/14/2018 04:45 AM, ShuFanLee wrote:
From: ShuFanLee
Handle vendor defined behavior in tcpci_init, tcpci_set_vconn and export
tcpci_irq.
More operations can be extended in tcpci_data if needed.
According to TCPCI specification, 4.4.5.2 ROLE_CONTROL,
TCPC shall not start DRP toggling until
On Tue, Jan 30, 2018 at 01:21:01PM +, shufan_lee(李書帆) wrote:
> Hi Guenter,
>
> For now, it looks like there are two ways to implement vendor data. It
> would be nice to hear your suggestion.
>
> 1. Set vendor data in the data field of of_device_id.
> If I understand correctly, this would
On 01/30/2018 05:25 AM, Heikki Krogerus wrote:
On Tue, Jan 02, 2018 at 03:50:48PM +, Adam Thomson wrote:
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
acc
On Mon, Jan 29, 2018 at 07:19:06AM +, shufan_lee(李書帆) wrote:
> Hi Guenter,
>
> We try to use the TCPCI driver on RT1711H and here are some questions.
>
> Q1. Is current TCPCI driver written according to TypeC Port Controller
> Interface Specification Revision 1.0 & Version 1.2?
Revision
526612
> *********
>
> -Original Message-----
> From: Guenter Roeck [mailto:groe...@gmail.com] On Behalf Of Guenter Roeck
> Sent: Saturday, January 20, 2018 12:03 AM
> To: Heikki Krogerus
> Cc: shufan_lee(李書帆); 'Jun Li'; ShuFanLee; cy_huang(黃啟原);
>
On Mon, Jan 22, 2018 at 03:31:29PM +0100, Greg Kroah-Hartman wrote:
> On Tue, Jan 02, 2018 at 03:50:48PM +, Adam Thomson wrote:
> > This patch set adds sink side support for the PPS feature introduced in the
> > USB PD 3.0 specification.
>
> Heikki and Guenter, any thoughts on this patch serie
On Fri, Jan 19, 2018 at 11:24:13AM +0200, Heikki Krogerus wrote:
> Hi,
>
> On Fri, Jan 19, 2018 at 09:01:24AM +, shufan_lee(?|) wrote:
> > Hi Heikki,
> >
> > For example, the flow of tcpci_init is a little bit different.
> > In tcpci_init, there are more parameters need to be set for
On 12/18/2017 12:48 AM, Zoltan Boszormenyi wrote:
From: Böszörményi Zoltán
Use the new request_declared_muxed_region() macro to
synchronize access to the I/O port pair 0xcd6 / 0xcd7.
At the same time, remove the long lifetime request_region()
call to reserve these I/O ports, so the sp5100_tco
On Tue, Dec 19, 2017 at 07:11:11AM +0100, Boszormenyi Zoltan wrote:
> 2017-12-18 20:07 keltezéssel, Guenter Roeck írta:
> >On Mon, Dec 18, 2017 at 09:48:38AM +0100, Zoltan Boszormenyi wrote:
> >>From: Böszörményi Zoltán
> >>
> >>In order to make requ
On Mon, Dec 18, 2017 at 09:48:38AM +0100, Zoltan Boszormenyi wrote:
> From: Böszörményi Zoltán
>
> In order to make request_*muxed_region() behave more like
> mutex_lock(), a possible failure case needs to be eliminated.
> When drivers do not properly share the same I/O region, e.g.
> one is usin
On Mon, Dec 18, 2017 at 09:48:37AM +0100, Zoltan Boszormenyi wrote:
> From: Böszörményi Zoltán
>
> Add a new IORESOURCE_ALLOCATED flag that is automatically used
> when alloc_resource() is used internally in kernel/resource.c
> and free_resource() now takes this flag into account.
>
> The core o
ps://chromium.googlesource.com/chromiumos/platform/ec,
which uses 10 as starting point. Nothing we can do about that.
Anyway,
Reviewed-by: Guenter Roeck
> /* ChromeOS specific commands */
> #define VDO_CMD_VERSION VDO_CMD_VENDOR(0)
> --
> 2.15.1
>
--
To unsubscrib
On 12/04/2017 06:04 PM, Pravin Shedge wrote:
These duplicate includes have been found with scripts/checkincludes.pl but
they have been removed manually to avoid removing false positives.
Signed-off-by: Pravin Shedge
Reviewed-by: Guenter Roeck
---
drivers/usb/typec/fusb302/fusb302.c | 1
t real_port initialized.
> The problem occurred on Hikey960 and was reproduced by Guenter Roeck
> on Kevin with chromeos-4.4.
>
> Cc: Guenter Roeck
> Cc: # v4.10+
Fixes: ee8665e28e8d ("xhci: free xhci virtual devices with leaf nodes first")
would probably be bett
'max_snk_mv', 'max_snk_ma' and 'max_snk_mw' ?
Signed-off-by: Badhri Jagan Sridharan
Acked-by: Heikki Krogerus
Reviewed-by: Guenter Roeck
---
Changelog since v1:
- Rebased the patch on top of drivers/usb/type/tcpm.c
- Added duplicate pdo check for variable
On 11/23/2017 03:03 PM, Mats Karrman wrote:
If probe fails, fusb302_debugfs_exit is called making it impossible
to view any logs so use normal dev_err for any error messages during
probe.
Signed-off-by: Mats Karrman
Reviewed-by: Guenter Roeck
---
Changes since v1:
- Driver is no
On 11/23/2017 01:41 PM, Mats Karrman wrote:
If probe fails, fusb302_debugfs_exit is called making it impossible
to view any logs so use normal dev_err for any error messages during
probe.
Signed-off-by: Mats Karrman
---
drivers/staging/typec/fusb302/fusb302.c | 21 +++--
1 fi
gt; Signed-off-by: Adam Thomson
I am not really happy about the code duplication (two calls to
fusb302_pd_read_message() and identical error messages), but
everything else I came up with was even more messy, so
Reviewed-by: Guenter Roeck
> ---
>
> Changes in v3:
> - Always read f
On Thu, Nov 16, 2017 at 04:28:11PM +, Adam Thomson wrote:
> The expectation in the FUSB302 driver is that a TX_SUCCESS event
> should occur after a message has been sent, but before a GCRCSENT
> event is raised to indicate successful receipt of a message from
> the partner. However in some circ
On 11/15/2017 05:01 PM, Badhri Jagan Sridharan wrote:
On Tue, Nov 14, 2017 at 9:05 AM, Guenter Roeck wrote:
[ ... ]
Reviewed-by: Guenter Roeck
/* Though I really dislike this new-world comment style.
* It hurts my eyes and distracts me from the code.
*/
Did I somehow introduce this
fered. If the Sink requires a different voltage this
> will be indicated by its Sink Capabilities message.
>
> Signed-off-by: Badhri Jagan Sridharan
> Acked-by: Heikki Krogerus
Reviewed-by: Guenter Roeck
/* Though I really dislike this new-world comment style.
* It hurts my eyes and d
f partner
> device would only log them.
>
> Signed-off-by: Badhri Jagan Sridharan
> Acked-by: Heikki Krogerus
Reviewed-by: Guenter Roeck
> ---
> Changelog since v1:
> - rebased on top drivers/usb/typec/tcpm.c as suggested by
> gre...@linuxfoundation.org
> - renamed nr
On Tue, Nov 14, 2017 at 05:02:53PM +0200, Heikki Krogerus wrote:
> Hi Guenter,
>
> On Tue, Nov 14, 2017 at 06:48:21AM -0800, Guenter Roeck wrote:
> > On 11/14/2017 05:17 AM, Greg KH wrote:
> > > On Mon, Nov 13, 2017 at 09:29:36PM -0800, Linus Torvalds wrote:
> > &g
On 11/14/2017 05:17 AM, Greg KH wrote:
On Mon, Nov 13, 2017 at 09:29:36PM -0800, Linus Torvalds wrote:
On Mon, Nov 13, 2017 at 8:19 AM, Greg KH wrote:
Other major thing is the typec code that moved out of staging and into
the "real" part of the drivers/usb/ tree, which was nice to see happen.
eal_port initialized.
> The problem occurred on Hikey960 and was reproduced by Guenter Roeck
> on Kevin with chromeos-4.4.
>
> Cc: Guenter Roeck
> Signed-off-by: Fan Ning
> Signed-off-by: Li Rui
> Signed-off-by: yangdi
> Signed-off-by: Yu Chen
>
Thanks a lot for trackin
On Mon, Nov 6, 2017 at 6:00 AM, Mathias Nyman
wrote:
> On 06.11.2017 14:36, Chen Yu wrote:
>>
>>
>>
>> On 2017/11/6 19:32, Greg KH wrote:
A simple process is as below:
xhci_plat_probe()
|
usb_add_hcd()
xhci_plat_remove()
On 11/05/2017 10:41 AM, Mike Looijmans wrote:
On 03-11-17 18:27, Alan Stern wrote:
On Fri, 3 Nov 2017, Mike Looijmans wrote:
Sometimes the USB device gets confused about the state of the initialization and
the connection fails. In particular, the device thinks that it's already set up
and runn
101 - 200 of 505 matches
Mail list logo