Re: [PATCHv3 2/2] usb: typec: anx7688: Add driver for ANX7688 USB-C HDMI bridge

2024-04-16 Thread Heikki Krogerus
On Tue, Apr 09, 2024 at 01:04:12PM +0200, Pavel Machek wrote: > Hi! > > > > This is driver for ANX7688 USB-C HDMI, with flashing and debugging > > > features removed. ANX7688 is rather criticial piece on PinePhone, > > > there's no display and no battery charging without it. > > > > > > There's

Re: [PATCHv3 2/2] usb: typec: anx7688: Add driver for ANX7688 USB-C HDMI bridge

2024-04-09 Thread Heikki Krogerus
Hi, On Mon, Apr 08, 2024 at 12:54:25PM +0200, Pavel Machek wrote: > From: Ondrej Jirman > > This is driver for ANX7688 USB-C HDMI, with flashing and debugging > features removed. ANX7688 is rather criticial piece on PinePhone, > there's no display and no battery charging without it. > >

Re: [PATCH] usb: typec: ptn36502: switch to DRM_AUX_BRIDGE

2024-03-18 Thread Heikki Krogerus
On Fri, Mar 15, 2024 at 05:04:22PM +0100, Luca Weiss wrote: > Switch to using the new DRM_AUX_BRIDGE helper to create the transparent > DRM bridge device instead of handcoding corresponding functionality. > > Signed-off-by: Luca Weiss Reviewed-by: Heikki Krogerus > ---

Re: [PATCHv2 2/2] usb: typec: anx7688: Add driver for ANX7688 USB-C HDMI bridge

2024-03-12 Thread Heikki Krogerus
Hi Pavel, I'm sorry to keep you waiting. On Fri, Feb 23, 2024 at 10:28:49PM +0100, Pavel Machek wrote: > From: Ondrej Jirman > > This is driver for ANX7688 USB-C HDMI, with flashing and debugging > features removed. ANX7688 is rather criticial piece on PinePhone, > there's no display and no

Re: [PATCH 2/3] usb: typec: ucsi: Add qcm6490-pmic-glink as needing PDOS quirk

2023-12-20 Thread Heikki Krogerus
On Wed, Dec 20, 2023 at 11:02:57AM +0100, Luca Weiss wrote: > The QCM6490 Linux Android firmware needs this workaround as well. Add it > to the list. > > Signed-off-by: Luca Weiss Acked-by: Heikki Krogerus > --- > drivers/usb/typec/ucsi/ucsi_glink.c | 1 + > 1 file

Re: [PATCH v2 2/3] usb: typec: fsa4480: Add support to swap SBU orientation

2023-10-30 Thread Heikki Krogerus
y: Neil Armstrong > Signed-off-by: Luca Weiss Reviewed-by: Heikki Krogerus > --- > drivers/usb/typec/mux/fsa4480.c | 71 > + > 1 file changed, 71 insertions(+) > > diff --git a/drivers/usb/typec/mux/fsa4480.c b/drivers/usb/typec/mux/fsa

Re: [PATCH 2/3] usb: typec: fsa4480: Add support to swap SBU orientation

2023-10-18 Thread Heikki Krogerus
Hi Luca, > > Shouldn't you loop through the endpoints? In any case: > > > > ep = fwnode_graph_get_next_endpoint(dev_fwnode(>client->dev, > > NULL)); > > The docs only mention one endpoint so I'm assuming just next_endpoint is > fine? I'm mostly concerned about what we may have in the

Re: [PATCH 2/3] usb: typec: fsa4480: Add support to swap SBU orientation

2023-10-17 Thread Heikki Krogerus
Hi Luca, On Fri, Oct 13, 2023 at 01:38:06PM +0200, Luca Weiss wrote: > On some hardware designs the AUX+/- lanes are connected reversed to > SBU1/2 compared to the expected design by FSA4480. > > Made more complicated, the otherwise compatible Orient-Chip OCP96011 > expects the lanes to be

Re: [PATCH 2/2] usb: typec: add support for PTN36502 redriver

2023-10-17 Thread Heikki Krogerus
Hi, On Fri, Oct 13, 2023 at 04:24:48PM +0200, Luca Weiss wrote: > Add a driver for the NXP PTN36502 Type-C USB 3.1 Gen 1 and DisplayPort > v1.2 combo redriver. > > Signed-off-by: Luca Weiss Looks OK to me, but couple of nitpicks below. With those fixed: Reviewed-by: Hei

Re: [PATCH] software node: Allow node addition to already existing device

2021-04-14 Thread Heikki Krogerus
On Wed, Apr 14, 2021 at 11:17:55AM +0200, Greg Kroah-Hartman wrote: > On Wed, Apr 14, 2021 at 12:13:35PM +0300, Heikki Krogerus wrote: > > +Greg > > > > Sorry about that. Should I resend this? > > No worries, I can pick it up, thanks > > `b4` really is nice t

Re: [PATCH] software node: Allow node addition to already existing device

2021-04-14 Thread Heikki Krogerus
+Greg Sorry about that. Should I resend this? On Wed, Apr 14, 2021 at 10:54:38AM +0300, Heikki Krogerus wrote: > If the node is added to an already exiting device, the node > needs to be also linked to the device separately. > > This will make sure the reference count is kept in ba

Re: [PATCH] usb: typec: silence a static checker warning

2021-04-14 Thread Heikki Krogerus
ional. Let's > re-arrange the code to silence the warning and make the intent more > clear. > > Signed-off-by: Dan Carpenter Reviewed-by: Heikki Krogerus > --- > drivers/usb/typec/port-mapper.c | 6 -- > 1 file changed, 4 insertions(+), 2 deletions(-) > > d

Re: [PATCH v3 2/3] usb: typec: tcpm: Allow slow charging loops to comply to pSnkStby

2021-04-14 Thread Heikki Krogerus
non-pd link. > > Signed-off-by: Badhri Jagan Sridharan Reviewed-by: Heikki Krogerus > --- > Changes since V2: > * Refactored code based on Heikki's suggestion > --- > drivers/usb/typec/tcpm/tcpm.c | 17 ++--- > 1 file changed, 14 insertions(+), 3 deletions(-

Re: [PATCH v3 1/3] usb: typec: tcpm: Honour pSnkStdby requirement during negotiation

2021-04-14 Thread Heikki Krogerus
D_RESET_SEND @ > 500 ms [rev2 POWER_NEGOTIATION] > [ 169.260222] PD RX, header: 0x566 [1] > [ 169.260227] Setting voltage/current limit 9000 mV 2000 mA > [ 169.261315] set_auto_vbus_discharge_threshold mode:3 pps_active:n > vbus:9000 ret:0 > [ 169.261321] state change SN

[PATCH] software node: Allow node addition to already existing device

2021-04-14 Thread Heikki Krogerus
node: Introduce device_add_software_node()") Signed-off-by: Heikki Krogerus --- drivers/base/swnode.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/drivers/base/swnode.c b/drivers/base/swnode.c index 740333629b420..3cc11b813f28c 100644 --- a/drivers/base/swnode.c +++

Re: [PATCH] ASoC: Intel: Handle device properties with software node API

2021-04-14 Thread Heikki Krogerus
On Tue, Apr 13, 2021 at 10:47:49AM -0500, Pierre-Louis Bossart wrote: > > > On 4/13/21 9:05 AM, Heikki Krogerus wrote: > > On Tue, Apr 13, 2021 at 03:20:45PM +0300, Heikki Krogerus wrote: > > > On Mon, Apr 12, 2021 at 03:36:20PM -0500, Pierre-Louis Bossart wrote:

Re: [PATCH] ASoC: Intel: Handle device properties with software node API

2021-04-13 Thread Heikki Krogerus
On Tue, Apr 13, 2021 at 03:20:45PM +0300, Heikki Krogerus wrote: > On Mon, Apr 12, 2021 at 03:36:20PM -0500, Pierre-Louis Bossart wrote: > > I took the code and split it in two for BYT/CHT (modified to remove devm_) > > and SoundWire parts (added as is). > > > > https

Re: [PATCH] ASoC: Intel: Handle device properties with software node API

2021-04-13 Thread Heikki Krogerus
On Mon, Apr 12, 2021 at 03:36:20PM -0500, Pierre-Louis Bossart wrote: > I took the code and split it in two for BYT/CHT (modified to remove devm_) > and SoundWire parts (added as is). > > https://github.com/thesofproject/linux/pull/2810 > > Both cases result in a refcount error on

Re: [PATCH v2 1/1] devres: Enable trace events

2021-04-13 Thread Heikki Krogerus
On Tue, Apr 13, 2021 at 02:38:01PM +0300, Andy Shevchenko wrote: > In some cases the printf() mechanism is too heavy and can't be used. > For example, when debugging a race condition involving devres API. > When CONFIG_DEBUG_DEVRES is enabled I can't reproduce an issue, and > otherwise it's quite

Re: linux-next: build failure after merge of the usb tree

2021-04-12 Thread Heikki Krogerus
On Mon, Apr 12, 2021 at 02:35:44PM +0200, Greg KH wrote: > On Mon, Apr 12, 2021 at 09:36:55PM +1000, Stephen Rothwell wrote: > > Hi all, > > > > After merging the usb tree, today's linux-next build (x86_64 almodconfig > > modules_install) failed like this: > > > > depmod: ERROR: Cycle detected:

Re: [PATCH 00/12] i2c: Adding support for software nodes

2021-04-09 Thread Heikki Krogerus
On Thu, Apr 08, 2021 at 11:53:23PM +0200, Wolfram Sang wrote: > On Mon, Mar 29, 2021 at 01:50:35PM +0300, Heikki Krogerus wrote: > > Hi, > > > > The old device property API (device_add_properties()) is going to be > > removed. These prepare the i2c subsystem

Re: [PATCH 1/1] usb: typec: tcpm: remove unused static variable 'tcpm_altmode_ops'

2021-04-09 Thread Heikki Krogerus
On Thu, Apr 08, 2021 at 07:14:01PM +0200, Hans de Goede wrote: > Hi, > > On 4/8/21 3:55 PM, Guenter Roeck wrote: > > On 4/8/21 1:28 AM, Heikki Krogerus wrote: > >> On Wed, Apr 07, 2021 at 05:15:40PM +0800, Zhen Lei wrote: > >>> Fixes the following W=1 kernel bu

Re: [PATCH] usb: typec: tcpm: remove useless variable

2021-04-09 Thread Heikki Krogerus
On Fri, Apr 09, 2021 at 05:22:16PM +0800, Jiapeng Chong wrote: > Fix the following gcc warning: > > drivers/usb/typec/tcpm/tcpm.c:2107:39: warning: ‘tcpm_altmode_ops’ > defined but not used [-Wunused-const-variable=]. > > Reported-by: Abaci Robot > Signed-off-by: Jiapeng Chong > --- >

Re: [PATCH 1/1] usb: typec: tcpm: remove unused static variable 'tcpm_altmode_ops'

2021-04-08 Thread Heikki Krogerus
On Thu, Apr 08, 2021 at 11:10:38AM +0200, Hans de Goede wrote: > Hi, > > On 4/7/21 11:15 AM, Zhen Lei wrote: > > Fixes the following W=1 kernel build warning: > > > > drivers/usb/typec/tcpm/tcpm.c:2107:39: warning: ‘tcpm_altmode_ops’ defined > > but not used [-Wunused-const-variable=] > > > >

Re: [PATCH 1/1] usb: typec: tcpm: remove unused static variable 'tcpm_altmode_ops'

2021-04-08 Thread Heikki Krogerus
On Thu, Apr 08, 2021 at 10:38:15AM +0200, Greg Kroah-Hartman wrote: > On Thu, Apr 08, 2021 at 11:28:09AM +0300, Heikki Krogerus wrote: > > On Wed, Apr 07, 2021 at 05:15:40PM +0800, Zhen Lei wrote: > > > Fixes the following W=1 kernel build warning: > > > > > >

Re: [PATCH 1/1] usb: typec: tcpm: remove unused static variable 'tcpm_altmode_ops'

2021-04-08 Thread Heikki Krogerus
ed-off-by: Zhen Lei Oh, I thought this was already fixed. Should this go into the stable trees as well? Acked-by: Heikki Krogerus > --- > drivers/usb/typec/tcpm/tcpm.c | 60 > --- > 1 file changed, 60 deletions(-) > > diff --git a

Re: [PATCH v2 5/6] usb: typec: tcpm: Allow slow charging loops to comply to pSnkStby

2021-04-08 Thread Heikki Krogerus
> > @@ -4047,9 +4053,12 @@ static void run_state_machine(struct tcpm_port *port) > > break; > > case SNK_DISCOVERY: > > if (port->vbus_present) { > > - tcpm_set_current_limit(port, > > -

Re: [PATCH v2 5/6] usb: typec: tcpm: Allow slow charging loops to comply to pSnkStby

2021-04-08 Thread Heikki Krogerus
On Wed, Apr 07, 2021 at 01:07:22PM -0700, Badhri Jagan Sridharan wrote: > When a PD charger advertising Rp-3.0 is connected to a sink port, the > sink port current limit would 3A, during SNK_DISCOVERY, till power > negotiation starts. Once the negotiation starts the power limit needs > to drop

Re: [PATCH v2 4/6] usb: typec: tcpm: Honour pSnkStdby requirement during negotiation

2021-04-08 Thread Heikki Krogerus
On Wed, Apr 07, 2021 at 01:07:21PM -0700, Badhri Jagan Sridharan wrote: > >From PD Spec: > The Sink Shall transition to Sink Standby before a positive or > negative voltage transition of VBUS. During Sink Standby > the Sink Shall reduce its power draw to pSnkStdby. This allows > the Source to

Re: [PATCH v2 3/6] usb: typec: tcpm: update power supply once partner accepts

2021-04-08 Thread Heikki Krogerus
he tcpm_reset_port as power_supply_changed is already > called right after usb_type is changed. > > Fixes: f2a8aa053c176 ("typec: tcpm: Represent source supply through > power_supply") > Signed-off-by: Badhri Jagan Sridharan > Reviewed-by: Adam Thomson Reviewed-by: Heikki Kr

Re: [PATCH v2 2/6] usb: typec: tcpm: Address incorrect values of tcpm psy for pps supply

2021-04-08 Thread Heikki Krogerus
current_limit, supply_voltage gets updated > once local port's tcpm enters SNK_TRANSITION_SINK when the accepted > current_limit and supply_voltage is enforced. > > Fixes: f2a8aa053c176 ("typec: tcpm: Represent source supply through > power_supply") > Signed-off-by: Ba

Re: [PATCH v2 1/6] usb: typec: tcpm: Address incorrect values of tcpm psy for fixed supply

2021-04-08 Thread Heikki Krogerus
pec: tcpm: Represent source supply through > power_supply") > Signed-off-by: Badhri Jagan Sridharan > Reviewed-by: Guenter Roeck > Reviewed-by: Adam Thomson Reviewed-by: Heikki Krogerus > --- > Changes since V1: > * Fixed typo as suggested by Guenter Roeck. >

[PATCH v6 4/4] usb: typec: Link all ports during connector registration

2021-04-07 Thread Heikki Krogerus
The connectors may be registered after the ports, so the "connector" links need to be created for the ports also when ever a new connector gets registered. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/class.c | 9 +++-- drivers/usb/typec/class.h | 4 +-- drivers

[PATCH v6 3/4] usb: Iterator for ports

2021-04-07 Thread Heikki Krogerus
Introducing usb_for_each_port(). It works the same way as usb_for_each_dev(), but instead of going through every USB device in the system, it walks through the USB ports in the system. Acked-by: Alan Stern Signed-off-by: Heikki Krogerus --- drivers/usb/core/usb.c | 46

[PATCH v6 2/4] usb: Link the ports to the connectors they are attached to

2021-04-07 Thread Heikki Krogerus
Creating link to the USB Type-C connector for every new port that is added when possible. Signed-off-by: Heikki Krogerus --- Documentation/ABI/testing/sysfs-bus-usb | 9 + drivers/usb/core/port.c | 3 +++ 2 files changed, 12 insertions(+) diff --git a/Documentation/ABI

[PATCH v6 1/4] usb: typec: Port mapping utility

2021-04-07 Thread Heikki Krogerus
ort object shares the _PLD (Physical Location of Device) with the USB Type-C connector that it's attached to. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/Makefile | 2 +- drivers/usb/typec/class.c | 7 +- drivers/usb/typec/class.h | 9 ++ drivers/usb/typec/por

[PATCH v6 0/4] usb: Linking ports to their Type-C connectors

2021-04-07 Thread Heikki Krogerus
a port is important in general, but it is really important when for example the data or power role of a device needs to swapped. The user probable wants to know which USB device is disconnected if role swap on a USB Type-C connector is executed. Hope these are OK. thanks, Heikki Krogerus (4): usb:

Re: [PATCH v5 3/6] usb: typec: Port mapping utility

2021-04-07 Thread Heikki Krogerus
On Mon, Apr 05, 2021 at 09:09:00AM +0200, Greg Kroah-Hartman wrote: > On Thu, Apr 01, 2021 at 01:58:44PM +0300, Heikki Krogerus wrote: > > Adding functions that can be used to link/unlink ports - > > USB ports, TBT3/USB4 ports, DisplayPorts and so on - to > > the

[PATCH v5 5/6] usb: Iterator for ports

2021-04-01 Thread Heikki Krogerus
Introducing usb_for_each_port(). It works the same way as usb_for_each_dev(), but instead of going through every USB device in the system, it walks through the USB ports in the system. Signed-off-by: Heikki Krogerus Acked-by: Alan Stern --- drivers/usb/core/usb.c | 46

[PATCH v5 6/6] usb: typec: Link all ports during connector registration

2021-04-01 Thread Heikki Krogerus
The connectors may be registered after the ports, so the "connector" links need to be created for the ports also when ever a new connector gets registered. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/class.c | 9 +++-- drivers/usb/typec/class.h | 4 +-- drivers

[PATCH v5 3/6] usb: typec: Port mapping utility

2021-04-01 Thread Heikki Krogerus
ort object shares the _PLD (Physical Location of Device) with the USB Type-C connector that it's attached to. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/Makefile | 2 +- drivers/usb/typec/class.c | 7 +- drivers/usb/typec/class.h | 9 ++ drivers/usb/typec/por

[PATCH v5 2/6] usb: typec: Declare the typec_class static

2021-04-01 Thread Heikki Krogerus
This is only to make the handling of the class consistent with the two other susbsystems - the alt mode bus and the mux class. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/class.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/drivers/usb

[PATCH v5 1/6] usb: typec: Organize the private headers properly

2021-04-01 Thread Heikki Krogerus
Adding a header file for each subsystem - the connector class, alt mode bus and the class for the muxes. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/bus.c | 2 ++ drivers/usb/typec/bus.h | 19 +- drivers/usb/typec/class.c | 69 +++ drivers

[PATCH v5 4/6] usb: Link the ports to the connectors they are attached to

2021-04-01 Thread Heikki Krogerus
Creating link to the USB Type-C connector for every new port that is added when possible. Signed-off-by: Heikki Krogerus --- Documentation/ABI/testing/sysfs-bus-usb | 9 + drivers/usb/core/port.c | 3 +++ 2 files changed, 12 insertions(+) diff --git a/Documentation/ABI

[PATCH v5 0/6] usb: Linking ports to their Type-C connectors

2021-04-01 Thread Heikki Krogerus
user probable wants to know which USB device is disconnected if role swap on a USB Type-C connector is executed. Hope these are OK. thanks, Heikki Krogerus (6): usb: typec: Organize the private headers properly usb: typec: Declare the typec_class static usb: typec: Port mapping utility

Re: [PATCH v4 0/6] usb: Linking ports to their Type-C connectors

2021-04-01 Thread Heikki Krogerus
On Thu, Apr 01, 2021 at 09:53:41AM +0300, Heikki Krogerus wrote: > Hi, > > One more version. I used #ifdef when I should have used #if > IS_DEFINED(). Thanks Guenter for pointing that out. > > I'm sending this version right away because of the holidays. I'm not > changing

[PATCH v4 5/6] usb: Iterator for ports

2021-04-01 Thread Heikki Krogerus
Introducing usb_for_each_port(). It works the same way as usb_for_each_dev(), but instead of going through every USB device in the system, it walks through the USB ports in the system. Signed-off-by: Heikki Krogerus Acked-by: Alan Stern --- drivers/usb/core/usb.c | 46

[PATCH v4 3/6] usb: typec: Port mapping utility

2021-04-01 Thread Heikki Krogerus
ort object shares the _PLD (Physical Location of Device) with the USB Type-C connector that it's attached to. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/Makefile | 2 +- drivers/usb/typec/class.c | 7 +- drivers/usb/typec/class.h | 9 ++ drivers/usb/typec/por

[PATCH v4 6/6] usb: typec: Link all ports during connector registration

2021-04-01 Thread Heikki Krogerus
The connectors may be registered after the ports, so the "connector" links need to be created for the ports also when ever a new connector gets registered. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/class.c | 9 +++-- drivers/usb/typec/class.h | 4 +-- drivers

[PATCH v4 2/6] usb: typec: Declare the typec_class static

2021-04-01 Thread Heikki Krogerus
This is only to make the handling of the class consistent with the two other susbsystems - the alt mode bus and the mux class. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/class.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/drivers/usb

[PATCH v4 4/6] usb: Link the ports to the connectors they are attached to

2021-04-01 Thread Heikki Krogerus
Creating link to the USB Type-C connector for every new port that is added when possible. Signed-off-by: Heikki Krogerus --- Documentation/ABI/testing/sysfs-bus-usb | 9 + drivers/usb/core/port.c | 3 +++ 2 files changed, 12 insertions(+) diff --git a/Documentation/ABI

[PATCH v4 1/6] usb: typec: Organize the private headers properly

2021-04-01 Thread Heikki Krogerus
Adding a header file for each subsystem - the connector class, alt mode bus and the class for the muxes. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/bus.c | 2 ++ drivers/usb/typec/bus.h | 19 +- drivers/usb/typec/class.c | 69 +++ drivers

[PATCH v4 0/6] usb: Linking ports to their Type-C connectors

2021-04-01 Thread Heikki Krogerus
USB Type-C connector is executed. Hope these are OK. thanks, Heikki Krogerus (6): usb: typec: Organize the private headers properly usb: typec: Declare the typec_class static usb: typec: Port mapping utility usb: Link the ports to the connectors they are attached to usb: Iterator for ports

Re: [PATCH v3 5/6] usb: Iterator for ports

2021-04-01 Thread Heikki Krogerus
On Wed, Mar 31, 2021 at 09:41:22AM -0700, Guenter Roeck wrote: > > diff --git a/include/linux/usb.h b/include/linux/usb.h > > index ddd2f5b2a2827..ebcd03d835d04 100644 > > --- a/include/linux/usb.h > > +++ b/include/linux/usb.h > > @@ -882,6 +882,15 @@ extern struct usb_host_interface

Re: [PATCH v2 1/6] software node: Free resources explicitly when swnode_register() fails

2021-03-31 Thread Heikki Krogerus
e logic by freeing resources explicitly when swnode_register() > fails. Currently it happens only in fwnode_create_software_node(). > > Signed-off-by: Andy Shevchenko It all looks OK to me. FWIW, for the whole series: Reviewed-by: Heikki Krogerus > --- > v2: no chan

[PATCH v3 5/6] usb: Iterator for ports

2021-03-31 Thread Heikki Krogerus
Introducing usb_for_each_port(). It works the same way as usb_for_each_dev(), but instead of going through every USB device in the system, it walks through the USB ports in the system. Signed-off-by: Heikki Krogerus Acked-by: Alan Stern --- drivers/usb/core/usb.c | 46

[PATCH v3 6/6] usb: typec: Link all ports during connector registration

2021-03-31 Thread Heikki Krogerus
The connectors may be registered after the ports, so the "connector" links need to be created for the ports also when ever a new connector gets registered. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/class.c | 9 +++-- drivers/usb/typec/class.h | 4 +-- drivers

[PATCH v3 4/6] usb: Link the ports to the connectors they are attached to

2021-03-31 Thread Heikki Krogerus
Creating link to the USB Type-C connector for every new port that is added when possible. Signed-off-by: Heikki Krogerus --- Documentation/ABI/testing/sysfs-bus-usb | 9 + drivers/usb/core/port.c | 3 +++ 2 files changed, 12 insertions(+) diff --git a/Documentation/ABI

[PATCH v3 3/6] usb: typec: Port mapping utility

2021-03-31 Thread Heikki Krogerus
ort object shares the _PLD (Physical Location of Device) with the USB Type-C connector that it's attached to. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/Makefile | 2 +- drivers/usb/typec/class.c | 7 +- drivers/usb/typec/class.h | 9 ++ drivers/usb/typec/por

[PATCH v3 1/6] usb: typec: Organize the private headers properly

2021-03-31 Thread Heikki Krogerus
Adding a header file for each subsystem - the connector class, alt mode bus and the class for the muxes. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/bus.c | 2 ++ drivers/usb/typec/bus.h | 19 +- drivers/usb/typec/class.c | 69 +++ drivers

[PATCH v3 2/6] usb: typec: Declare the typec_class static

2021-03-31 Thread Heikki Krogerus
This is only to make the handling of the class consistent with the two other susbsystems - the alt mode bus and the mux class. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/class.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/drivers/usb

[PATCH v3 0/6] usb: Linking ports to their Type-C connectors

2021-03-31 Thread Heikki Krogerus
now which USB device is disconnected if role swap on a USB Type-C connector is executed. Hope these are OK. thanks, Heikki Krogerus (6): usb: typec: Organize the private headers properly usb: typec: Declare the typec_class static usb: typec: Port mapping utility usb: Link the ports to the connecto

Re: [PATCH 00/12] i2c: Adding support for software nodes

2021-03-31 Thread Heikki Krogerus
On Wed, Mar 31, 2021 at 11:22:32AM +0200, Wolfram Sang wrote: > > > The old device property API (device_add_properties()) is going to be > > removed. These prepare the i2c subsystem and drivers for the change. > > The change is fairly trivial in case of i2c. All we need to do is add > > complete

Re: [RFC PATCH v2 2/2] usb: typec: sama7g5_tcpc: add driver for Microchip sama7g5 tcpc

2021-03-31 Thread Heikki Krogerus
Hi Cristian, On Tue, Mar 30, 2021 at 11:54:42PM +0300, cristian.bir...@microchip.com wrote: > From: Cristian Birsan > > This patch adds initial driver support for the new Microchip USB > Type-C Port Controller (TCPC) embedded in sama7g5 SoC. > > Signed-off-by: Cristian Birsan > --- >

Re: [PATCH v2 5/6] usb: Iterator for ports

2021-03-30 Thread Heikki Krogerus
On Mon, Mar 29, 2021 at 02:49:46PM -0400, Alan Stern wrote: > On Mon, Mar 29, 2021 at 11:44:25AM +0300, Heikki Krogerus wrote: > > Introducing usb_for_each_port(). It works the same way as > > usb_for_each_dev(), but instead of going through every USB > > device in the sys

Re: [PATCH 05/12] ARM: s3c: mini2440: Constify the software node

2021-03-29 Thread Heikki Krogerus
On Mon, Mar 29, 2021 at 12:58:41PM +0200, Krzysztof Kozlowski wrote: > On 29/03/2021 12:50, Heikki Krogerus wrote: > > Additional device properties are always just a part of a > > software fwnode. If the device properties are constant, the > > software node can also b

[PATCH 12/12] i2c: Remove support for dangling device properties

2021-03-29 Thread Heikki Krogerus
>From now on only accepting complete software nodes. Signed-off-by: Heikki Krogerus --- drivers/i2c/i2c-boardinfo.c | 11 --- drivers/i2c/i2c-core-base.c | 15 +-- include/linux/i2c.h | 2 -- 3 files changed, 1 insertion(+), 27 deletions(-) diff --git a/drivers/

[PATCH 03/12] ARM: omap1: osk: Constify the software node

2021-03-29 Thread Heikki Krogerus
Additional device properties are always just a part of a software fwnode. If the device properties are constant, the software node can also be constant. Signed-off-by: Heikki Krogerus Cc: Aaro Koskinen Cc: Tony Lindgren --- arch/arm/mach-omap1/board-osk.c | 6 +- 1 file changed, 5

[PATCH 06/12] platform/x86: intel_cht_int33fe_microb: Constify the software node

2021-03-29 Thread Heikki Krogerus
Additional device properties are always just a part of a software fwnode. If the device properties are constant, the software node can also be constant. Signed-off-by: Heikki Krogerus Cc: Hans de Goede --- drivers/platform/x86/intel_cht_int33fe_microb.c | 6 +- 1 file changed, 5 insertions

[PATCH 11/12] Input: elantech - Prepare a complete software node for the device

2021-03-29 Thread Heikki Krogerus
for the problematic "dangling" device properties from i2c subsystem, i.e. the "properties" member from struct i2c_board_info. The problems caused by them are not related to this driver. Signed-off-by: Heikki Krogerus Cc: Dmitry Torokhov --- drivers/input/mouse/elantech.c | 6 -

[PATCH 05/12] ARM: s3c: mini2440: Constify the software node

2021-03-29 Thread Heikki Krogerus
Additional device properties are always just a part of a software fwnode. If the device properties are constant, the software node can also be constant. Signed-off-by: Heikki Krogerus Cc: Krzysztof Kozlowski --- arch/arm/mach-s3c/mach-mini2440.c | 6 +- 1 file changed, 5 insertions(+), 1

[PATCH 04/12] ARM: pxa: stargate2: Constify the software node

2021-03-29 Thread Heikki Krogerus
Additional device properties are always just a part of a software fwnode. If the device properties are constant, the software node can also be constant. Signed-off-by: Heikki Krogerus Cc: Jonathan Cameron Cc: Daniel Mack Cc: Haojian Zhuang Cc: Robert Jarzmik --- arch/arm/mach-pxa/stargate2

[PATCH 02/12] ARM: davinci: Constify the software nodes

2021-03-29 Thread Heikki Krogerus
Additional device properties are always just a part of a software fwnode. If the device properties are constant, the software node can also be constant. Signed-off-by: Heikki Krogerus Cc: Sekhar Nori Cc: Bartosz Golaszewski --- arch/arm/mach-davinci/board-da830-evm.c| 6 +- arch/arm

[PATCH 07/12] i2c: cht-wc: Constify the software node

2021-03-29 Thread Heikki Krogerus
Additional device properties are always just a part of a software fwnode. If the device properties are constant, the software node can also be constant. Signed-off-by: Heikki Krogerus Cc: Hans de Goede --- drivers/i2c/busses/i2c-cht-wc.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion

[PATCH 10/12] platform/chrome: chromeos_laptop - Prepare complete software nodes

2021-03-29 Thread Heikki Krogerus
The older device property API is going to be removed soon and that will affect also I2C subystem. Supplying complete software nodes instead of only the properties in them for the I2C devices. Signed-off-by: Heikki Krogerus Cc: Benson Leung Cc: Enric Balletbo i Serra --- drivers/platform

[PATCH 08/12] i2c: nvidia-gpu: Constify the software node

2021-03-29 Thread Heikki Krogerus
Additional device properties are always just a part of a software fwnode. If the device properties are constant, the software node can also be constant. Signed-off-by: Heikki Krogerus Cc: Ajay Gupta --- drivers/i2c/busses/i2c-nvidia-gpu.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion

[PATCH 09/12] i2c: icy: Constify the software node

2021-03-29 Thread Heikki Krogerus
Complete software node can now be supplied to the device with struct i2c_board_info. Signed-off-by: Heikki Krogerus Cc: Max Staudt --- drivers/i2c/busses/i2c-icy.c | 32 +--- 1 file changed, 9 insertions(+), 23 deletions(-) diff --git a/drivers/i2c/busses/i2c-icy.c

[PATCH 00/12] i2c: Adding support for software nodes

2021-03-29 Thread Heikki Krogerus
nodes. thanks, Heikki Krogerus (12): i2c: Add support for software nodes ARM: davinci: Constify the software nodes ARM: omap1: osk: Constify the software node ARM: pxa: stargate2: Constify the software node ARM: s3c: mini2440: Constify the software node platform/x86

[PATCH 01/12] i2c: Add support for software nodes

2021-03-29 Thread Heikki Krogerus
This makes it possible for the drivers to assign complete software fwnodes to the devices instead of only the device properties in those nodes. Signed-off-by: Heikki Krogerus --- drivers/i2c/i2c-core-base.c | 15 ++- include/linux/i2c.h | 4 +++- 2 files changed, 17

Re: [PATCH v2 6/6] usb: typec: Link all ports during connector registration

2021-03-29 Thread Heikki Krogerus
> I could make a stub for the usb_for_each_port() function in case > CONFIG_USB is not enable. Would that work? Ah, I think that's what you meant :-) I'll fix it. thaks, -- heikki

Re: [PATCH v2 6/6] usb: typec: Link all ports during connector registration

2021-03-29 Thread Heikki Krogerus
On Mon, Mar 29, 2021 at 10:48:19AM +0200, Greg Kroah-Hartman wrote: > On Mon, Mar 29, 2021 at 11:44:26AM +0300, Heikki Krogerus wrote: > > +#ifdef CONFIG_USB > > This feels odd in a file under drivers/usb/ is it still relevant? Will > this code get built for non-USB systems

[PATCH v2 6/6] usb: typec: Link all ports during connector registration

2021-03-29 Thread Heikki Krogerus
The connectors may be registered after the ports, so the "connector" links need to be created for the ports also when ever a new connector gets registered. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/class.c | 9 +++-- drivers/usb/typec/class.h | 4 +- drivers

[PATCH v2 5/6] usb: Iterator for ports

2021-03-29 Thread Heikki Krogerus
Introducing usb_for_each_port(). It works the same way as usb_for_each_dev(), but instead of going through every USB device in the system, it walks through the USB ports in the system. Signed-off-by: Heikki Krogerus --- drivers/usb/core/usb.c | 46

[PATCH v2 4/6] usb: Link the ports to the connectors they are attached to

2021-03-29 Thread Heikki Krogerus
Creating link to the USB Type-C connector for every new port that is added when possible. Signed-off-by: Heikki Krogerus --- Documentation/ABI/testing/sysfs-bus-usb | 9 + drivers/usb/core/port.c | 3 +++ 2 files changed, 12 insertions(+) diff --git a/Documentation/ABI

[PATCH v2 3/6] usb: typec: Port mapping utility

2021-03-29 Thread Heikki Krogerus
ort object shares the _PLD (Physical Location of Device) with the USB Type-C connector that it's attached to. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/Makefile | 2 +- drivers/usb/typec/class.c | 7 +- drivers/usb/typec/class.h | 9 ++ drivers/usb/typec/por

[PATCH v2 1/6] usb: typec: Organize the private headers properly

2021-03-29 Thread Heikki Krogerus
Adding a header file for each subsystem - the connector class, alt mode bus and the class for the muxes. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/bus.c | 2 ++ drivers/usb/typec/bus.h | 19 +- drivers/usb/typec/class.c | 69 +++ drivers

[PATCH v2 2/6] usb: typec: Declare the typec_class static

2021-03-29 Thread Heikki Krogerus
This is only to make the handling of the class consistent with the two other susbsystems - the alt mode bus and the mux class. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/class.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/drivers/usb

[PATCH v2 0/6] usb: Linking ports to their Type-C connectors

2021-03-29 Thread Heikki Krogerus
. Hope these are OK. thanks, Heikki Krogerus (6): usb: typec: Organize the private headers properly usb: typec: Declare the typec_class static usb: typec: Port mapping utility usb: Link the ports to the connectors they are attached to usb: Iterator for ports usb: typec: Link all po

Re: [PATCH 1/6] usb: Iterator for ports

2021-03-25 Thread Heikki Krogerus
On Thu, Mar 25, 2021 at 04:20:15PM +0100, Greg Kroah-Hartman wrote: > On Thu, Mar 25, 2021 at 05:14:42PM +0200, Heikki Krogerus wrote: > > On Thu, Mar 25, 2021 at 10:41:09AM -0400, Alan Stern wrote: > > > On Thu, Mar 25, 2021 at 03:29:21PM +0300, Heikki Krogerus wrote: &

Re: [PATCH 1/6] usb: Iterator for ports

2021-03-25 Thread Heikki Krogerus
On Thu, Mar 25, 2021 at 05:14:45PM +0200, Heikki Krogerus wrote: > On Thu, Mar 25, 2021 at 10:41:09AM -0400, Alan Stern wrote: > > On Thu, Mar 25, 2021 at 03:29:21PM +0300, Heikki Krogerus wrote: > > > Introducing usb_for_each_port(). It works the same way as > > > usb_

Re: [PATCH 1/6] usb: Iterator for ports

2021-03-25 Thread Heikki Krogerus
On Thu, Mar 25, 2021 at 10:41:09AM -0400, Alan Stern wrote: > On Thu, Mar 25, 2021 at 03:29:21PM +0300, Heikki Krogerus wrote: > > Introducing usb_for_each_port(). It works the same way as > > usb_for_each_dev(), but instead of going through every USB > > device in the sys

[PATCH 6/6] usb: typec: Link all ports during connector registration

2021-03-25 Thread Heikki Krogerus
The connectors may be registered after the ports, so the "connector" links need to be created for the ports also when ever a new connector gets registered. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/class.c | 9 +++-- drivers/usb/typec/class.h | 10 +++--- d

[PATCH 1/6] usb: Iterator for ports

2021-03-25 Thread Heikki Krogerus
Introducing usb_for_each_port(). It works the same way as usb_for_each_dev(), but instead of going through every USB device in the system, it walks through the USB ports in the system. Signed-off-by: Heikki Krogerus --- drivers/usb/core/usb.c | 43

[PATCH 4/6] usb: typec: Port mapping utility

2021-03-25 Thread Heikki Krogerus
ort object shares the _PLD (Physical Location of Device) with the USB Type-C connector that it's attached to. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/Makefile | 1 + drivers/usb/typec/class.c | 7 +- drivers/usb/typec/class.h | 18 +++ drivers/usb/typec/por

[PATCH 3/6] usb: typec: Declare the typec_class static

2021-03-25 Thread Heikki Krogerus
This is only to make the handling of the class consistent with the two other susbsystems - the alt mode bus and the mux class. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/class.c | 24 +--- 1 file changed, 13 insertions(+), 11 deletions(-) diff --git a/drivers/usb

[PATCH 5/6] usb: Link the ports to the connectors they are attached to

2021-03-25 Thread Heikki Krogerus
Creating link to the USB Type-C connector for every new port that is added when possible. Signed-off-by: Heikki Krogerus --- Documentation/ABI/testing/sysfs-bus-usb | 9 + drivers/usb/core/port.c | 3 +++ 2 files changed, 12 insertions(+) diff --git a/Documentation/ABI

[PATCH 2/6] usb: typec: Organize the private headers properly

2021-03-25 Thread Heikki Krogerus
Adding a header file for each subsystem - the connector class, alt mode bus and the class for the muxes. Signed-off-by: Heikki Krogerus --- drivers/usb/typec/bus.c | 2 ++ drivers/usb/typec/bus.h | 19 +- drivers/usb/typec/class.c | 69 +++ drivers

[PATCH 0/6] usb: Linking ports to their Type-C connectors

2021-03-25 Thread Heikki Krogerus
connected to a port is important in general, but it is really important when for example the data or power role of a device needs to swapped. The user probable wants to know which USB device is disconnected if role swap on a USB Type-C connector is executed. Hope these are OK. thanks, Heikki K

Re: [PATCH] usb: typec: Fix a typo

2021-03-25 Thread Heikki Krogerus
On Thu, Mar 25, 2021 at 10:40:23AM +0530, Bhaskar Chowdhury wrote: > > s/Acknowlege/Acknowledge/ > > Signed-off-by: Bhaskar Chowdhury Reviewed-by: Heikki Krogerus > --- > drivers/usb/typec/ucsi/ucsi.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > &g

Re: [PATCH] ASoC: Intel: Handle device properties with software node API

2021-03-23 Thread Heikki Krogerus
On Mon, Mar 22, 2021 at 10:02:40AM -0500, Pierre-Louis Bossart wrote: > > > On 3/22/21 6:06 AM, Heikki Krogerus wrote: > > The function device_add_properties() is going to be removed. > > Replacing it with software node API equivalents. > > > > Signed-off-by

  1   2   3   4   5   6   7   8   9   10   >