[no subject]
unsubscribe linux-usb -- To unsubscribe from this list: send the line unsubscribe linux-usb in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/2] MSM: Add USB suport for QSD8x50
On Tue, 2010-12-14 at 09:10 +0530, Pavan Kondeti wrote: Hi Daniel, Are you talking to me? read below, On 12/8/2010 1:37 PM, Pavankumar Kondeti wrote: OTG driver takes care of putting hardware into low power mode. Hence make peripheral and host devices as children of OTG device and let runtime PM takes care of notifying peripheral and host state to OTG device. VBUS power up and shutdown routines are implemented by modem processor. As RPC infrastructure is not available, configure USB in peripheral only mode. Signed-off-by: Pavankumar Kondeti pkond...@codeaurora.org --- This patch depends on [PATCH V3 00/11] Add MSM USB controller driver patch series which is under review. USB driver patches are taken into Greg KH usb-next tree. Is it possible to include this patch for 2.6.38? I don't control Greg's tree, so I don't know what his policies are regarding patch acceptance .. I am asking about board file changes. i.e this patch and the follow up patch not USB driver patches. Thanks, Pavan -- Sent by an consultant of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. -- To unsubscribe from this list: send the line unsubscribe linux-arm-msm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [RFC 1/2] USB: Notify OTG errors to user space via uevents
Hi Greg, On Mon, Dec 06, 2010 at 06:07:50PM +0530, Pavankumar Kondeti wrote: OTG specification mandates no silent failures and all errors should be reported to the user. The spec itself does not give the exact error description. But recommends the error message to be self explanatory. Provide otg_notify_error() utility for USB core and OTG driver to send the error codes to user space. All the error code values are described in include/linux/usb/ch9.h. The user space application can listen to netlink socket and parse the buffer for MODULE=OTG and ERROR=n, where 'n' contains the error code. How are you going to listen to the netlink socket that is already grabbed by libudev? Sorry. I never worked with udev. But I read udev documentation. I thought an external script can be invoked by adding a udev rule when MODULE=OTG is matched and ERROR value can be accessed in the script via env variable. Please, if you really want to do this, create your own netlink socket, don't create new uevent messages that will just confused the existing tools out there, that's ripe for big problems. I have seen examples in drivers sending uevents for notifying docking station status (drivers/acpi/dock.c) and when backlight brightness changes (drivers/video/backlight/backlight.c). So I have taken this approach. -- To unsubscribe from this list: send the line unsubscribe linux-arm-msm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Re: [PATCH 1/5] USB: Add MSM USB Device Controller driver
On Wed, 1 Sep 2010, Pavankumar Kondeti wrote: This patch adds the basic support for the USB Device Controller on Qualcomm MSM family of SOCs. The controller supports upto 16 endpoints including the default endpoint (ep0). All the data transfers are driven by DMA. VBUS line is also connected to PMIC chip. The module controlling PMIC chip notifies about cable connect/disconnect events. Hence, PHY comparators are turned off in low power mode. This driver was originally developed by Google and is available at http://android.git.kernel.org/?p=kernel/experimental.git. ... +static const struct usb_ep_ops msm72k_ep_ops = { +.enable = msm72k_enable, +.disable= msm72k_disable, + +.alloc_request = msm72k_alloc_request, +.free_request = msm72k_free_request, + +.queue = msm72k_queue, +.dequeue= msm72k_dequeue, + +.set_halt = msm72k_set_halt, +.fifo_flush = msm72k_fifo_flush, +}; The .set_wedge method is missing. Thanks for pointing this out. I will implement this method. -- Sent by a consultant of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum. -- To unsubscribe from this list: send the line unsubscribe linux-arm-msm in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html