Re: [PATCH v4 1/5] PCI: designware: add memory barrier after enabling region

2015-12-09 Thread Russell King - ARM Linux
On Wed, Dec 09, 2015 at 10:10:05AM +0530, Pratyush Anand wrote: > On Tue, Dec 8, 2015 at 2:31 PM, Stanimir Varbanov > wrote: > > > > On 12/03/2015 03:35 PM, Stanimir Varbanov wrote: > > > Add 'write memory' barrier after enable region in PCIE_ATU_CR2 > > > register.

Re: [PATCH v4 1/5] PCI: designware: add memory barrier after enabling region

2015-12-09 Thread Stanimir Varbanov
On 12/09/2015 11:52 AM, Arnd Bergmann wrote: > On Wednesday 09 December 2015 10:10:05 Pratyush Anand wrote: >> On Tue, Dec 8, 2015 at 2:31 PM, Stanimir Varbanov Signed-off-by: Stanimir Varbanov --- drivers/pci/host/pcie-designware.c |5 +

Re: [PATCH v4 1/5] PCI: designware: add memory barrier after enabling region

2015-12-09 Thread Arnd Bergmann
On Wednesday 09 December 2015 10:10:05 Pratyush Anand wrote: > On Tue, Dec 8, 2015 at 2:31 PM, Stanimir Varbanov > > > Signed-off-by: Stanimir Varbanov > > > --- > > > drivers/pci/host/pcie-designware.c |5 + > > > 1 file changed, 5 insertions(+) > > > > > >

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

2015-12-09 Thread Arnd Bergmann
On Tuesday 08 December 2015 16:22:40 John Stultz wrote: > >> diff --git a/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts > >> b/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts > >> index 5183d18..ee5dcb7 100644 > >> --- a/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts > >> +++

[PATCH v3 0/3] ARM: power: qcom_smbb: Add otg regulator

2015-12-09 Thread Tim Bird
Add a regulator to control the OTG chargepath switch. I'm re-sending this (v3) because my v2 submission yesterday failed to include the AUTHOR OF THE DRIVER(!!), Courtney Cavin. Changes since v2 - add the driver author to the To: list! Changes since v1 - switch supply name to have dashes

[PATCH v3 1/3] dt-binding: power: Add otg regulator binding

2015-12-09 Thread Tim Bird
Add a binding for the regulator which controls the OTG chargepath switch. The OTG switch gets its power from pm8941_5vs1, and that should be expressed as a usb-otg-in-supply property in the DT node for the charger driver. The regulator name is "otg". Signed-off-by: Tim Bird

[PATCH v3 2/3] power: qcom_smbb: Add otg regulator for control of vbus

2015-12-09 Thread Tim Bird
Add a regulator to control the OTG chargepath switch. This is used by USB code to control VBUS direction - out for host mode on the OTG port, and in for charging mode. Signed-off-by: Tim Bird --- Changes since v1: - changed name of supply to remove underscores ---

[PATCH v3 3/3] ARM: dts: qcom: add charger otg regulator

2015-12-09 Thread Tim Bird
Add the otg regulator provided by the charger block. Signed-off-by: Tim Bird --- arch/arm/boot/dts/qcom-pm8941.dtsi | 4 1 file changed, 4 insertions(+) diff --git a/arch/arm/boot/dts/qcom-pm8941.dtsi b/arch/arm/boot/dts/qcom-pm8941.dtsi index b0d4439..1b80100

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

2015-12-09 Thread Tim Bird
On 12/08/2015 08:11 PM, Rob Herring wrote: > On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: >> Add a binding for the regulator which controls the OTG chargepath switch. >> The OTG switch gets its power from pm8941_5vs1, and that should be >> expressed as a usb-otg-in-supply property in

Re: [PATCH RFC 01/27] PM / Domains: core changes for multiple states

2015-12-09 Thread Ulf Hansson
On 17 November 2015 at 23:37, Lina Iyer wrote: > From: Axel Haslam > > From: Axel Haslam > > Add the core changes to be able to declare multiple states. > When trying to set a power domain to off, genpd will be able to

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

2015-12-09 Thread Rob Herring
On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: > On 12/08/2015 08:11 PM, Rob Herring wrote: >> On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: >>> Add a binding for the regulator which controls the OTG chargepath switch. >>> The OTG switch gets its power from

Re: [PATCH v2 4/5] arm64: dts: qcom: Alias pm8916 on msm8916 devices

2015-12-09 Thread Frank Rowand
On 11/20/2015 3:31 PM, Stephen Boyd wrote: > Add an alias for pm8916 on msm8916 based SoCs so that the newly > updated dtbTool can find the pmic compatible string and add the > pmic-id element to the QCDT header. > > Signed-off-by: Stephen Boyd > --- >

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

2015-12-09 Thread Mark Brown
On Wed, Dec 09, 2015 at 09:42:09AM -0800, Tim Bird wrote: > On 12/09/2015 06:36 AM, Rob Herring wrote: > > You need a regulator-name property. Also, should should define valid > > values for regulator-min-microvolt and regulator-max-microvolt. > All of those are listed as optional properties in

Re: [PATCH] net: emac: emac gigabit ethernet controller driver

2015-12-09 Thread Fabio Estevam
On Wed, Dec 9, 2015 at 6:09 PM, Timur Tabi wrote: >> +/* set MAC address */ >> +void emac_mac_addr_clear(struct emac_adapter *adpt, u8 *addr) >> +{ >> + u32 sta; >> + >> + /* for example: 00-A0-C6-11-22-33 >> +* 0<-->C6112233, 1<-->00A0. >> +*/ >

Re: [PATCH] net: emac: emac gigabit ethernet controller driver

2015-12-09 Thread David Miller
From: Fabio Estevam Date: Wed, 9 Dec 2015 18:37:35 -0200 > On Wed, Dec 9, 2015 at 6:09 PM, Timur Tabi wrote: > >>> +/* set MAC address */ >>> +void emac_mac_addr_clear(struct emac_adapter *adpt, u8 *addr) >>> +{ >>> + u32 sta; >>> + >>> +

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

2015-12-09 Thread Rob Herring
On Wed, Dec 9, 2015 at 1:59 PM, Bjorn Andersson wrote: > On Wed 09 Dec 06:36 PST 2015, Rob Herring wrote: > >> On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: >> > On 12/08/2015 08:11 PM, Rob Herring wrote: >> >> On Tue, Dec 08, 2015 at

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

2015-12-09 Thread Sebastian Reichel
Hi, On Wed, Dec 09, 2015 at 09:42:09AM -0800, Tim Bird wrote: > Or maybe our power maintainers will chime in with some wisdom. I > can't be the first person to be adding a charge pathway switch to > mainline, so I'm open to doing it the "right way". :-) I don't think there is a standard way for

Re: [PATCH] net: emac: emac gigabit ethernet controller driver

2015-12-09 Thread Gilad Avidov
Thank you Timur for the good review. On Wed, 9 Dec 2015 14:09:27 -0600 Timur Tabi wrote: > So first of all, thanks for posting this. I know it's missing a bunch > of stuff that's necessary for Qualcomm's Server chip, but it's a > start. > > Unfortunately, 6,000 lines is

[PATCH 4/5] usb: ehci-msm: Fix register initialization

2015-12-09 Thread Timur Tabi
From: Jack Pham The default value for the 'transceiver select' field of the PORTSC register may not always be correct. Previously the phy-msm-usb driver would do this for us, but since ehci-msm can now be instantiated standalone without any PHY driver, the register needs to

[PATCH 2/5] usb: ehci-msm: Remove dependency on OTG PHY

2015-12-09 Thread Timur Tabi
From: Jack Pham Currently the EHCI MSM driver has a hard dependency to be created by an OTG layer, namely the phy-msm-usb driver. In some cases or board configurations we want to allow the EHCI host to be instantiated without OTG capability. Instead, relax the dependency on

[PATCH 1/5] usb: ehci-msm: Allow LS devices to work

2015-12-09 Thread Timur Tabi
From: Jack Pham Disable the silicon quirk which is normally enabled for HSIC host mode. This would otherwise prevent low speed devices from enumerating properly. Signed-off-by: Jack Pham Signed-off-by: Timur Tabi ---

[PATCH 5/5] usb: ehci-msm: Register usb shutdown function

2015-12-09 Thread Timur Tabi
From: Azriel Samson Registering usb_hcd_platform_shutdown to be called during shutdown. This is a generic function that performs the generic host stack's shutdown. It ensures that USB operations do not continue while kexec boots a new kernel. Signed-off-by: Azriel Samson

[PATCH 3/5] usb: ehci-msm: Add support for ACPI probing

2015-12-09 Thread Timur Tabi
From: Jack Pham Allow the EHCI MSM driver to probe against an ACPI enumerated device with ID QCOM8040. Signed-off-by: Jack Pham Signed-off-by: Timur Tabi --- drivers/usb/host/ehci-msm.c | 8 1 file changed, 8

Re: [PATCH 4/5] usb: ehci-msm: Fix register initialization

2015-12-09 Thread Jack Pham
Hi Timur, Thanks for sending these patches upstream on my behalf. On Wed, Dec 09, 2015 at 05:41:07PM -0600, Timur Tabi wrote: > Also, update to use the correct value of AHBMODE to allow > data transfers to be posted AHB transactions. This aligns > with the value used in the downstream MSM

Re: [RFC][PATCH] misc: Introduce reboot_reason driver

2015-12-09 Thread John Stultz
On Tue, Dec 8, 2015 at 2:07 PM, Bjorn Andersson wrote: > On Tue 08 Dec 13:29 PST 2015, John Stultz wrote: >> diff --git a/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts >> b/arch/arm/boot/dts/qcom-apq8064-nexus7-flo.dts >> index 5183d18..ee5dcb7 100644 >> ---

Re: [PATCH 4/5] usb: ehci-msm: Fix register initialization

2015-12-09 Thread Timur Tabi
Jack Pham wrote: Andy already sent a patch for just this register change. http://marc.info/?l=linux-usb=144678991912202=2 I see it's already been queued on Greg's usb-next. I think it would be good to rebase. Ok, I will do that in my next patch version. -- Sent by an employee of the Qualcomm

Re: [PATCH] net: emac: emac gigabit ethernet controller driver

2015-12-09 Thread Timur Tabi
Gilad Avidov wrote: pointer math on void* ? what is the size of void ? I'm talking about adding and subtracting pointer values, so u32 pkt_len =((void *)ip_hdr(skb) - skb->data) -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

2015-12-09 Thread Bjorn Andersson
On Wed 09 Dec 06:36 PST 2015, Rob Herring wrote: > On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: > > On 12/08/2015 08:11 PM, Rob Herring wrote: > >> On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: > >>> Add a binding for the regulator which controls the OTG

Re: [PATCH v2 1/3] dt-binding: power: Add otg regulator binding

2015-12-09 Thread Tim Bird
On 12/09/2015 06:36 AM, Rob Herring wrote: > On Wed, Dec 9, 2015 at 6:55 AM, Tim Bird wrote: >> On 12/08/2015 08:11 PM, Rob Herring wrote: >>> On Tue, Dec 08, 2015 at 04:40:16PM -0800, Tim Bird wrote: Add a binding for the regulator which controls the OTG