Hi Zengtao,
On 3/21/2018 2:45 PM, Zengtao (B) wrote:
> Hi Minas:
>
>> -Original Message-
>> From: Minas Harutyunyan [mailto:minas.harutyun...@synopsys.com]
>> Sent: Wednesday, March 21, 2018 4:08 PM
>> To: Zengtao (B) ; Minas Harutyunyan
>> ; John Youn ;
>> Felipe Balbi ; Greg Kroah-Hartm
Hi,
On Wed, 2018-03-21 at 13:30 +0200, Roger Quadros wrote:
> Martin,
>
> On 21/03/18 00:01, Martin Blumenstingl wrote:
> > Hi Roger, Hi Chunfeng,
> >
> > On Tue, Mar 20, 2018 at 1:04 PM, Chunfeng Yun
> > wrote:
> >> Hi Martin & Roger:
> >>
> >> On Mon, 2018-03-19 at 17:12 +0100, Martin Blumens
hi,
On Wed, 2018-03-21 at 14:27 +0200, Roger Quadros wrote:
> On 21/03/18 13:50, Chunfeng Yun wrote:
> > Hi Roger,
> >
> > On Wed, 2018-03-21 at 11:05 +0200, Roger Quadros wrote:
> >> Hi Chunfeng,
> >>
> >> On 21/03/18 08:12, Chunfeng Yun wrote:
> >>> The phys has already been initialized when ad
Hi,
Randy Dunlap writes:
> On 03/21/2018 01:25 AM, Stephen Rothwell wrote:
>> Hi all,
>>
>> Changes since 20180320:
>>
>
>
> on i386:
>
> ERROR: "__tracepoint_dwc3_gadget_generic_cmd" [drivers/usb/dwc3/dwc3.ko]
> undefined!
> ERROR: "__tracepoint_dwc3_ep_queue" [drivers/usb/dwc3/dwc3.ko] unde
The phys has already been initialized when add primary hcd,
including usb2 phys and usb3 phys also if exist, so needn't
re-parse "phys" property again.
Signed-off-by: Chunfeng Yun
---
drivers/usb/core/hcd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/usb/core/hcd.
If building a kernel without FTRACE but with TRACING, dwc3.ko fails to
link due to missing trace events. Fix this by using the correct
Kconfig symbol on Makefile.
Reported-by: Randy Dunlap
Signed-off-by: Felipe Balbi
---
drivers/usb/dwc3/Makefile | 2 +-
1 file changed, 1 insertion(+), 1 deleti
Hi,
Joel Pepper writes:
> - Add bFrameIndex as a UVCG_FRAME_ATTR_RO for each frame size.
> - Automatically assign ascending bFrameIndex to each frame in a format.
>
> Before all "bFrameindex" attributes were set to "1" with no way to
> configure the gadget otherwise. This resulted in the host al
Hi,
Benjamin Herrenschmidt writes:
> On Mon, 2018-03-19 at 12:56 +0200, Felipe Balbi wrote:
>> >> do you really need this to be safe? You don't seem to be modifying
>> >> ep_list here.
>> >
>> > Yes, ep->dispose() may do just that. In my Aspeed implementation in
>> > fact that's pretty much the
On Thu, Mar 22, 2018 at 04:39:20PM +0800, Chunfeng Yun wrote:
> The phys has already been initialized when add primary hcd,
> including usb2 phys and usb3 phys also if exist, so needn't
> re-parse "phys" property again.
>
> Signed-off-by: Chunfeng Yun
> ---
> drivers/usb/core/hcd.c | 2 +-
> 1 f
Minas Harutyunyan writes:
> Hi Filipe,
>
> On 3/17/2018 1:08 PM, Minas Harutyunyan wrote:
>> This series fully update existing ISOC DDMA flow which initially based on
>> 2 descriptor chains. Switching between desc chains performing based on BNA
>> interrupt. Because of BNA interrupt few packets c
hi,
On Thu, 2018-03-22 at 10:01 +0100, Greg Kroah-Hartman wrote:
> On Thu, Mar 22, 2018 at 04:39:20PM +0800, Chunfeng Yun wrote:
> > The phys has already been initialized when add primary hcd,
> > including usb2 phys and usb3 phys also if exist, so needn't
> > re-parse "phys" property again.
> >
Hi Minas:
>-Original Message-
>From: Minas Harutyunyan [mailto:minas.harutyun...@synopsys.com]
>Sent: Thursday, March 22, 2018 4:06 PM
>To: Zengtao (B) ; Minas Harutyunyan
>; John Youn ;
>Felipe Balbi ; Greg Kroah-Hartman
>; linux-usb@vger.kernel.org
>Subject: Re: [PATCH v1 2/3] usb: dwc2
On 22 March 2018 03:53, Guenter Roeck wrote:
> > +static void tcpm_pd_ext_msg_request(struct tcpm_port *port,
> > + const struct pd_message *msg)
> > +{
> > + enum pd_ext_msg_type type = pd_header_type_le(msg->header);
> > + unsigned int data_size = pd_ext_header_
devm_regulator_get_optional returns -ENODEV if the regulator isn't
there, so if that's the case we have to make sure not to leave -ENODEV
in the regulator pointer.
Also, make sure we return 0 in that case, but correctly propagate any
other errors. Also propagate the error from _dwc2_hcd_start.
Fi
On 22 March 2018 04:03, Guenter Roeck wrote:
> > static enum pdo_err tcpm_caps_err(struct tcpm_port *port, const u32 *pdo,
> > @@ -1308,6 +1347,26 @@ static enum pdo_err tcpm_caps_err(struct tcpm_port
> *port, const u32 *pdo,
> > pdo_min_voltage(pdo[i - 1]))
The AB8540 was an evolved version of the AB8500, but it was never
mass produced or put into products, only reference designs exist.
The upstream support was never completed and it is unlikely that
this will happen so drop the support for now to simplify
maintenance of the AB8500.
Cc: Loic Pallardy
On 22 March 2018 04:08, Guenter Roeck wrote:
> > +static ssize_t power_supply_show_usb_type(struct device *dev,
> > + enum power_supply_usb_type
> *usb_types,
> > + ssize_t num_usb_types,
> > +
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 tcpm_port *port = power_supply_get_drvdata
Hi Zengtao,
On 3/22/2018 1:36 PM, Zengtao (B) wrote:
> Hi Minas:
>
>> -Original Message-
>> From: Minas Harutyunyan [mailto:minas.harutyun...@synopsys.com]
>> Sent: Thursday, March 22, 2018 4:06 PM
>> To: Zengtao (B) ; Minas Harutyunyan
>> ; John Youn ;
>> Felipe Balbi ; Greg Kroah-Hartma
Hi Filipe,
On 3/22/2018 1:05 PM, Felipe Balbi wrote:
> Minas Harutyunyan writes:
>
>> Hi Filipe,
>>
>> On 3/17/2018 1:08 PM, Minas Harutyunyan wrote:
>>> This series fully update existing ISOC DDMA flow which initially based on
>>> 2 descriptor chains. Switching between desc chains performing ba
Am Donnerstag, 22. März 2018, 10:39:43 CET schrieb Tomeu Vizoso:
> devm_regulator_get_optional returns -ENODEV if the regulator isn't
> there, so if that's the case we have to make sure not to leave -ENODEV
> in the regulator pointer.
>
> Also, make sure we return 0 in that case, but correctly pro
Hi Tomeu,
Am Donnerstag, 22. März 2018, 12:39:13 CET schrieb Heiko Stübner:
> Am Donnerstag, 22. März 2018, 10:39:43 CET schrieb Tomeu Vizoso:
> > devm_regulator_get_optional returns -ENODEV if the regulator isn't
> > there, so if that's the case we have to make sure not to leave -ENODEV
> > in th
Hello Kai-Heng Feng,
This is a semi-automatic email about new static checker warnings.
The patch 027bd6cafd9a: "usb: core: Add "quirks" parameter for
usbcore" from Mar 20, 2018, leads to the following Smatch complaint:
drivers/usb/core/quirks.c:136 quirks_param_set()
error: we previousl
On Thu, Mar 22, 2018 at 05:21:26PM +0800, Chunfeng Yun wrote:
> hi,
> On Thu, 2018-03-22 at 10:01 +0100, Greg Kroah-Hartman wrote:
> > On Thu, Mar 22, 2018 at 04:39:20PM +0800, Chunfeng Yun wrote:
> > > The phys has already been initialized when add primary hcd,
> > > including usb2 phys and usb3
On Wed, Mar 21, 2018 at 01:29:42PM +0800, Zhengjun Xing wrote:
> USB3 hubs don't support global suspend.
>
> USB3 specification 10.10, Enhanced SuperSpeed hubs only support selective
> suspend and resume, they do not support global suspend/resume where the
> hub downstream facing ports states are
On Tue, Mar 20, 2018 at 11:17:13AM +0100, Richard Leitner wrote:
> From: Richard Leitner
>
> For some userspace applications information on the number of
> over-current conditions at specific USB hub ports is relevant.
>
> In our case we have a series of USB hardware (using the cp210x driver)
>
Hi Heiko,
Hi Tomeu,
On 03/22/2018 12:41 PM, Heiko Stübner wrote:
> Hi Tomeu,
>
> Am Donnerstag, 22. März 2018, 12:39:13 CET schrieb Heiko Stübner:
>> Am Donnerstag, 22. März 2018, 10:39:43 CET schrieb Tomeu Vizoso:
>>> devm_regulator_get_optional returns -ENODEV if the regulator isn't
>>> there,
On 22/03/18 14:12, Chunfeng Yun wrote:
> The phys has already been initialized when add primary hcd,
> including usb2 phys and usb3 phys also if exist, so needn't
> re-parse "phys" property again.
>
> Signed-off-by: Chunfeng Yun
Reviewed-by: Roger Quadros
> ---
> V3:
> add the following in
On 22/03/18 10:10, Chunfeng Yun wrote:
> Hi,
> On Wed, 2018-03-21 at 13:30 +0200, Roger Quadros wrote:
>> Martin,
>>
>> On 21/03/18 00:01, Martin Blumenstingl wrote:
>>> Hi Roger, Hi Chunfeng,
>>>
>>> On Tue, Mar 20, 2018 at 1:04 PM, Chunfeng Yun
>>> wrote:
Hi Martin & Roger:
On Mo
On Thu, Mar 22, 2018 at 11:22:24AM +0100, Linus Walleij wrote:
> The AB8540 was an evolved version of the AB8500, but it was never
> mass produced or put into products, only reference designs exist.
> The upstream support was never completed and it is unlikely that
> this will happen so drop the su
On Tue, Mar 20, 2018 at 03:57:01PM +0300, Heikki Krogerus wrote:
> Hi,
>
> This is version 9 of Hans' an my series introducing support for USB muxes, and
> generic device connection handling. The last version have been mostly about
> getting the first patch introducing the device connection handli
On Thu, Mar 22, 2018 at 08:12:50PM +0800, Chunfeng Yun wrote:
> The phys has already been initialized when add primary hcd,
> including usb2 phys and usb3 phys also if exist, so needn't
> re-parse "phys" property again.
Wait, why does the subject say "xhci", when this is in USB core code?
I'll go
devm_regulator_get_optional returns -ENODEV if the regulator isn't
there, so if that's the case we have to make sure not to leave -ENODEV
in the regulator pointer.
Also, make sure we return 0 in that case, but correctly propagate any
other errors. Also propagate the error from _dwc2_hcd_start.
Fi
devm_regulator_get_optional returns -ENODEV if the regulator isn't
there, so if that's the case we have to make sure not to leave -ENODEV
in the regulator pointer.
Also, make sure we return 0 in that case, but correctly propagate any
other errors. Also propagate the error from _dwc2_hcd_start.
Fi
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 tcp
Am Donnerstag, 22. März 2018, 14:14:51 CET schrieb Tomeu Vizoso:
> devm_regulator_get_optional returns -ENODEV if the regulator isn't
> there, so if that's the case we have to make sure not to leave -ENODEV
> in the regulator pointer.
>
> Also, make sure we return 0 in that case, but correctly pro
On 03/22/2018 02:26 PM, Heiko Stübner wrote:
Am Donnerstag, 22. März 2018, 14:14:51 CET schrieb Tomeu Vizoso:
devm_regulator_get_optional returns -ENODEV if the regulator isn't
there, so if that's the case we have to make sure not to leave -ENODEV
in the regulator pointer.
Also, make sure we re
Hi,
Greg Kroah-Hartman writes:
> On Thu, Mar 22, 2018 at 11:22:24AM +0100, Linus Walleij wrote:
>> The AB8540 was an evolved version of the AB8500, but it was never
>> mass produced or put into products, only reference designs exist.
>> The upstream support was never completed and it is unlikely
Hi!
> idle lcd off phy-mapphone-mdm6600ohci-platform
> 153mW 284mW 344mW
>
> So it seems that MDM6600 is currently not yet idling even with it's
> radio turned off, but that's something that is beyond the control of
> this USB PHY driver. This patch does get us to t
On Thu, Mar 22, 2018 at 03:43:15PM +0200, Felipe Balbi wrote:
>
> Hi,
>
> Greg Kroah-Hartman writes:
> > On Thu, Mar 22, 2018 at 11:22:24AM +0100, Linus Walleij wrote:
> >> The AB8540 was an evolved version of the AB8500, but it was never
> >> mass produced or put into products, only reference d
This modem is embedded on dlink dwr-921 router.
The oem configuration states:
T: Bus=02 Lev=01 Prnt=01 Port=00 Cnt=01 Dev#= 2 Spd=480 MxCh= 0
D: Ver= 2.00 Cls=00(>ifc ) Sub=00 Prot=00 MxPS=64 #Cfgs= 1
P: Vendor=1435 ProdID=0918 Rev= 2.32
S: Manufacturer=Android
S: Product=Android
S: Serial
On Thu, Mar 22, 2018 at 03:40:40PM +0100, Giuseppe Lippolis wrote:
> This modem is embedded on dlink dwr-921 router.
> The oem configuration states:
> Signed-off-by: Giuseppe Lippolis
> ---
> drivers/net/usb/qmi_wwan.c | 3 +++
> drivers/usb/serial/option.c | 11 +++
> 2 files changed,
On Wed, 21 Mar 2018, Menion wrote:
> Well, still, my two cents here, is it really that if you have UAS in a
> module and you don't load it but connect an UAS capable device, this
> device is not enumerated rather than downgraded to usb-storage?
Let's be precise. The device _is_ enumerated, becau
Currently we warn the user when the root hub lost power after resume,
but the user cannot do anything about it so it should probably be a
notice.
This will reduce the noise in the console during suspend and resume,
which is already quite significant in many systems.
Signed-off-by: Tomeu Vizoso
-
Would be technically possible that the function uas_use_uas_driver
checks if the uas driver is loaded (not just configured) before
telling the usb-storage to leave the device to the uas module?
Bye
2018-03-22 16:16 GMT+01:00 Alan Stern :
> On Wed, 21 Mar 2018, Menion wrote:
>
>> Well, still, my tw
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 messagescreceived but not
yet handled.
Signed
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 a
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, to represent what was
connected (e.g. USB, USB_
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 to set
the current and voltage and allows for
This commit adds generic ABI information regarding power_supply
properties. This is an initial attempt to try and align the usage
of these properties between drivers. As part of this commit,
common Battery and USB related properties have been listed.
Signed-off-by: Adam Thomson
---
Documentation
Currently there's no error checking of this parameter in the
registration function and it's blindly added to psy class and
subsequently used as is. For example if this is NULL the call
to psy_register_thermal() will try to dereference the pointer
thus causing a kernel dump.
This commit updates the
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: Heikki Krogerus
---
drivers/usb/typec/tcpm.c |
On Thu, 22 Mar 2018, Menion wrote:
> Would be technically possible that the function uas_use_uas_driver
> checks if the uas driver is loaded (not just configured) before
> telling the usb-storage to leave the device to the uas module?
I suppose that might be possible. But it certainly would not
* Pavel Machek [180322 13:57]:
> Hi!
>
> > idle lcd offphy-mapphone-mdm6600ohci-platform
> > 153mW 284mW 344mW
> >
> > So it seems that MDM6600 is currently not yet idling even with it's
> > radio turned off, but that's something that is beyond the con
On Thu, Mar 22, 2018 at 07:31:59PM +0300, Matwey V. Kornilov wrote:
> Hi,
>
> I am running 4.16-rc6 and see the following issue with ASIX
> usb-ethernet dongle running on BeagleBoneBlack (TI AM335x).
Does the issue also happen on older kernels?
> My device is 100Mbps USB 2.0 dongle (ASIX AX88772
2018-03-22 20:11 GMT+03:00 Bin Liu :
> On Thu, Mar 22, 2018 at 07:31:59PM +0300, Matwey V. Kornilov wrote:
>> Hi,
>>
>> I am running 4.16-rc6 and see the following issue with ASIX
>> usb-ethernet dongle running on BeagleBoneBlack (TI AM335x).
>
> Does the issue also happen on older kernels?
>
>> My
On Thu, 22 Mar 2018, Menion wrote:
> You are right.
> But what if usb-storage check if the deferred uas bind does not succeed?
That is not possible, for two reasons. First, the kernel does not keep
track of binding attempts that fail. Second, the kernel can only try
to bind one driver to a devi
Mine was just a question from a non expert in kernel internals
Bye
2018-03-22 18:29 GMT+01:00 Alan Stern :
> On Thu, 22 Mar 2018, Menion wrote:
>
>> You are right.
>> But what if usb-storage check if the deferred uas bind does not succeed?
>
> That is not possible, for two reasons. First, the ker
Hi!
> > user@devuan:/sys/devices/platform/4400.ocp/4a064000.usbhshost$
> > sudo lsusb
> > Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
> > Bus 002 Device 001: ID 1d6b:0001 Linux Foundation 1.1 root hub
> > user@devuan:/sys/devices/platform/4400.ocp/4a064000.usbhshost$
> >
From: Krzysztof Opasiak
commit 749494b6bdbbaf0899aa1c62a1ad74cd747bce47 upstream.
Since commit: ba1582f22231 ("usb: gadget: f_hid: use alloc_ep_req()")
we cannot allocate any requests in bind() as we check if we should
align request buffer based on endpoint descriptor which is assigned
in set_al
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 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 02:53:11PM -0500, Bin Liu wrote:
> From: Krzysztof Opasiak
>
> commit 749494b6bdbbaf0899aa1c62a1ad74cd747bce47 upstream.
>
> Since commit: ba1582f22231 ("usb: gadget: f_hid: use alloc_ep_req()")
> we cannot allocate any requests in bind() as we check if we should
> align
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, 2018-03-22 at 20:28 +0100, Pavel Machek wrote:
> Hi!
>
> > > user@devuan:/sys/devices/platform/4400.ocp/4a064000.usbhshost
> > > $
> > > sudo lsusb
> > > Bus 001 Device 001: ID 1d6b:0002 Linux Foundation 2.0 root hub
> > > Bus 002 Device 001: ID 1d6b:0001 Linux Foundation 1.1 root hub
On Tuesday, March 20, 2018 5:07:35 AM CET Daniel Drake wrote:
> acpi_dev_pm_get_state() is used to determine the range of allowable
> device power states when going into S3 suspend. This is implemented
> by executing the _S3D and _S3W ACPI methods.
>
> Linux follows the ACPI spec behaviour in that
On 2018년 03월 20일 21:57, Heikki Krogerus wrote:
> From: Hans de Goede
>
> The AXP288 BC1.2 charger detection / extcon code may seem like a strange
> place to add code to control the USB role-switch on devices with an AXP288,
> but there are 2 reasons to do this inside the axp288 extcon code:
>
>
On 3/22/2018 8:03 PM, Greg KH wrote:
On Wed, Mar 21, 2018 at 01:29:42PM +0800, Zhengjun Xing wrote:
USB3 hubs don't support global suspend.
USB3 specification 10.10, Enhanced SuperSpeed hubs only support selective
suspend and resume, they do not support global suspend/resume where the
hub dow
Some UDC may want to allocate endpoints dynamically, either because
the HW supports an arbitrary large number or because (like the Aspeed
BMC SoCs), the pool of HW endpoints is shared between multiple gadgets.
The allocation side can be done rather easily using the existing
match_ep() UDC hook.
H
The Aspeed BMC SoCs support a "virtual hub" function. It provides some
HW support for a top-level USB2 hub behind which sit 5 gadget "ports".
This driver adds support for the full functionality, emulating the
hub standard requests and exposing 5 UDC gadget drivers corresponding
to the ports.
The
On Thu, 2018-03-22 at 14:41 +0200, Roger Quadros wrote:
> On 22/03/18 10:10, Chunfeng Yun wrote:
> > Hi,
> > On Wed, 2018-03-21 at 13:30 +0200, Roger Quadros wrote:
> >> Martin,
> >>
> >> On 21/03/18 00:01, Martin Blumenstingl wrote:
> >>> Hi Roger, Hi Chunfeng,
> >>>
> >>> On Tue, Mar 20, 2018 at
On Thu, 2018-03-22 at 13:53 +0100, Greg Kroah-Hartman wrote:
> On Thu, Mar 22, 2018 at 08:12:50PM +0800, Chunfeng Yun wrote:
> > The phys has already been initialized when add primary hcd,
> > including usb2 phys and usb3 phys also if exist, so needn't
> > re-parse "phys" property again.
>
> Wait,
Hi Dan,
Dan Carpenter wrote:
Hello Kai-Heng Feng,
This is a semi-automatic email about new static checker warnings.
I ran Smatch but didn't see the error message:
$ make -j`nproc` CHECK="~/smatch/smatch -p=kernel" C=1 bzImage modules |
tee warns.txt`"
Also, "val" will never get passed a
73 matches
Mail list logo
