Re: [PATCH 1/2] i2c: qup: Add device tree bindings information
I agree with Kumar on removing this. If we decide it is something worth keeping, logic to support it doesn't belong in the QUP driver, but in the i2c core. I don't have strong objection here, will remove aliases support. When resubmitting, please test against v3.12-rcX. of_i2c_register_devices() is in the core meanwhile and alias support for of is in the core for a while, too. Check i2c_add_adapter(). signature.asc Description: Digital signature
Re: [PATCHv4 2/3] ARM: msm: Add support for APQ8074 Dragonboard
On Sep 23, 2013, at 10:13 PM, Rohit Vaswani wrote: This patch adds basic board support for APQ8074 Dragonboard which belongs to the Snapdragon 800 family. For now, just support a basic machine with device tree. Signed-off-by: Rohit Vaswani rvasw...@codeaurora.org --- arch/arm/Kconfig.debug | 9 +++ arch/arm/boot/dts/Makefile | 3 ++- arch/arm/boot/dts/qcom-apq8074-dragonboard.dts | 6 + arch/arm/boot/dts/qcom-msm8974.dtsi| 35 ++ arch/arm/include/debug/msm.S | 5 arch/arm/mach-msm/Kconfig | 13 ++ arch/arm/mach-msm/board-dt.c | 9 +++ 7 files changed, 79 insertions(+), 1 deletion(-) create mode 100644 arch/arm/boot/dts/qcom-apq8074-dragonboard.dts create mode 100644 arch/arm/boot/dts/qcom-msm8974.dtsi diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug index e18a6fc..959b2c7 100644 --- a/arch/arm/Kconfig.debug +++ b/arch/arm/Kconfig.debug @@ -357,6 +357,15 @@ choice Say Y here if you want the debug print routines to direct their output to the serial port on MSM 8960 devices. + config DEBUG_MSM8974_UART + bool Kernel low-level debugging messages via MSM 8974 UART + depends on ARCH_MSM8974 + select MSM_HAS_DEBUG_UART_HS + select DEBUG_MSM_UART + help + Say Y here if you want the debug print routines to direct + their output to the serial port on MSM 8974 devices. + A little surprised you didn't pull this and the ARCH_MSM8974 into its own patch outside of this patch being board support. config DEBUG_MVEBU_UART bool Kernel low-level debugging messages via MVEBU UART (old bootloaders) depends on ARCH_MVEBU diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile index 000cf76..e71a3ec 100644 --- a/arch/arm/boot/dts/Makefile +++ b/arch/arm/boot/dts/Makefile @@ -102,7 +102,8 @@ dtb-$(CONFIG_ARCH_KIRKWOOD) += kirkwood-cloudbox.dtb \ kirkwood-openblocks_a6.dtb dtb-$(CONFIG_ARCH_MARCO) += marco-evb.dtb dtb-$(CONFIG_ARCH_MSM) += msm8660-surf.dtb \ - msm8960-cdp.dtb + msm8960-cdp.dtb \ + qcom-apq8074-dragonboard.dtb dtb-$(CONFIG_ARCH_MVEBU) += armada-370-db.dtb \ armada-370-mirabox.dtb \ armada-370-netgear-rn102.dtb \ diff --git a/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts b/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts new file mode 100644 index 000..bb6f3c4 --- /dev/null +++ b/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts @@ -0,0 +1,6 @@ +/include/ qcom-msm8974.dtsi + +/ { + model = Qualcomm APQ8074 Dragonboard; + compatible = qcom,apq8074-dragonboard, qcom,apq8074; +}; diff --git a/arch/arm/boot/dts/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom-msm8974.dtsi new file mode 100644 index 000..f04b643 --- /dev/null +++ b/arch/arm/boot/dts/qcom-msm8974.dtsi @@ -0,0 +1,35 @@ +/dts-v1/; + +/include/ skeleton.dtsi + +/ { + model = Qualcomm MSM8974; + compatible = qcom,msm8974; + interrupt-parent = intc; + + soc: soc { }; We should have a unit address here: soc: soc@FOOBAR { also, split out the curly braces so any future patches do have to muck with that. }; +}; + +soc { + #address-cells = 1; + #size-cells = 1; + ranges; + compatible = simple-bus; + + intc: interrupt-controller@f900 { + compatible = qcom,msm-qgic2; + interrupt-controller; + #interrupt-cells = 3; + reg = 0xf900 0x1000, + 0xf9002000 0x1000; + }; + + timer { + compatible = arm,armv7-timer; + interrupts = 1 2 0xf08, + 1 3 0xf08, + 1 4 0xf08, + 1 1 0xf08; + clock-frequency = 1920; + }; +}; diff --git a/arch/arm/include/debug/msm.S b/arch/arm/include/debug/msm.S index 9166e1b..9d653d4 100644 --- a/arch/arm/include/debug/msm.S +++ b/arch/arm/include/debug/msm.S @@ -46,6 +46,11 @@ #define MSM_DEBUG_UART_PHYS 0x1644 #endif +#ifdef CONFIG_DEBUG_MSM8974_UART +#define MSM_DEBUG_UART_BASE 0xFA71E000 +#define MSM_DEBUG_UART_PHYS 0xF991E000 +#endif + .macro addruart, rp, rv, tmp #ifdef MSM_DEBUG_UART_PHYS ldr \rp, =MSM_DEBUG_UART_PHYS diff --git a/arch/arm/mach-msm/Kconfig b/arch/arm/mach-msm/Kconfig index 2586c28..086bcb9 100644 --- a/arch/arm/mach-msm/Kconfig +++ b/arch/arm/mach-msm/Kconfig @@ -64,6 +64,19 @@ config ARCH_MSM_DT select SPARSE_IRQ select USE_OF +config ARCH_MSM8974 + bool MSM8974 + select ARM_GIC + select CPU_V7 + select HAVE_ARM_ARCH_TIMER + select HAVE_SMP + select MSM_SCM if SMP + select USE_OF +
Re: [PATCH v2 1/1] usb: gadget: f_fs: Add support for SuperSpeed Mode
Hi, (please avoid top-posting) On Tue, Sep 24, 2013 at 03:00:20PM +0530, Manu Gautam wrote: Hi Felipe, I wanted to mention one point with respect to this patch: Below changes in the functionfs.h to add ss_count (super speed descriptors count) in desc_header (which is passed from userspace) make the driver incompatible with existing userspace applications compiled against old header file. Let me know if that is acceptable. We are using this driver with Android for adbd (android debug bridge) and these changes are required to support adb over Super Speed controllers e.g. DWC3 along with changed in adbd to pass SS EP and companion descriptors. Good you mentioned, it saves me the trouble of reviewing this patch :-) It's not acceptable to break userspace ABI at all. If you want SuperSpeed support on function fs, we need to figure out a way to do so without breaking userspace. This might mean adding a separate userspace interface to be used with superspeed. While at that, we might want to add a few bytes of reserved, unused space in our structures for situations where we need to add more data into it, just to make it slightly future proof. -- balbi signature.asc Description: Digital signature
Re: [PATCHv4 2/3] ARM: msm: Add support for APQ8074 Dragonboard
On 9/25/2013 12:49 PM, Kumar Gala wrote: On Sep 23, 2013, at 10:13 PM, Rohit Vaswani wrote: This patch adds basic board support for APQ8074 Dragonboard which belongs to the Snapdragon 800 family. For now, just support a basic machine with device tree. Signed-off-by: Rohit Vaswani rvasw...@codeaurora.org --- arch/arm/Kconfig.debug | 9 +++ arch/arm/boot/dts/Makefile | 3 ++- arch/arm/boot/dts/qcom-apq8074-dragonboard.dts | 6 + arch/arm/boot/dts/qcom-msm8974.dtsi| 35 ++ arch/arm/include/debug/msm.S | 5 arch/arm/mach-msm/Kconfig | 13 ++ arch/arm/mach-msm/board-dt.c | 9 +++ 7 files changed, 79 insertions(+), 1 deletion(-) create mode 100644 arch/arm/boot/dts/qcom-apq8074-dragonboard.dts create mode 100644 arch/arm/boot/dts/qcom-msm8974.dtsi diff --git a/arch/arm/Kconfig.debug b/arch/arm/Kconfig.debug index e18a6fc..959b2c7 100644 --- a/arch/arm/Kconfig.debug +++ b/arch/arm/Kconfig.debug @@ -357,6 +357,15 @@ choice Say Y here if you want the debug print routines to direct their output to the serial port on MSM 8960 devices. + config DEBUG_MSM8974_UART + bool Kernel low-level debugging messages via MSM 8974 UART + depends on ARCH_MSM8974 + select MSM_HAS_DEBUG_UART_HS + select DEBUG_MSM_UART + help + Say Y here if you want the debug print routines to direct + their output to the serial port on MSM 8974 devices. + A little surprised you didn't pull this and the ARCH_MSM8974 into its own patch outside of this patch being board support. Well, its good to have this as part of initial board setup for earlyprintk. config DEBUG_MVEBU_UART bool Kernel low-level debugging messages via MVEBU UART (old bootloaders) depends on ARCH_MVEBU diff --git a/arch/arm/boot/dts/Makefile b/arch/arm/boot/dts/Makefile index 000cf76..e71a3ec 100644 --- a/arch/arm/boot/dts/Makefile +++ b/arch/arm/boot/dts/Makefile @@ -102,7 +102,8 @@ dtb-$(CONFIG_ARCH_KIRKWOOD) += kirkwood-cloudbox.dtb \ kirkwood-openblocks_a6.dtb dtb-$(CONFIG_ARCH_MARCO) += marco-evb.dtb dtb-$(CONFIG_ARCH_MSM) += msm8660-surf.dtb \ - msm8960-cdp.dtb + msm8960-cdp.dtb \ + qcom-apq8074-dragonboard.dtb dtb-$(CONFIG_ARCH_MVEBU) += armada-370-db.dtb \ armada-370-mirabox.dtb \ armada-370-netgear-rn102.dtb \ diff --git a/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts b/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts new file mode 100644 index 000..bb6f3c4 --- /dev/null +++ b/arch/arm/boot/dts/qcom-apq8074-dragonboard.dts @@ -0,0 +1,6 @@ +/include/ qcom-msm8974.dtsi + +/ { + model = Qualcomm APQ8074 Dragonboard; + compatible = qcom,apq8074-dragonboard, qcom,apq8074; +}; diff --git a/arch/arm/boot/dts/qcom-msm8974.dtsi b/arch/arm/boot/dts/qcom-msm8974.dtsi new file mode 100644 index 000..f04b643 --- /dev/null +++ b/arch/arm/boot/dts/qcom-msm8974.dtsi @@ -0,0 +1,35 @@ +/dts-v1/; + +/include/ skeleton.dtsi + +/ { + model = Qualcomm MSM8974; + compatible = qcom,msm8974; + interrupt-parent = intc; + + soc: soc { }; We should have a unit address here: soc: soc@FOOBAR { also, split out the curly braces so any future patches do have to muck with that. }; Im not sure I understand the reasoning behind the unit address for soc ? +}; + +soc { + #address-cells = 1; + #size-cells = 1; + ranges; + compatible = simple-bus; + + intc: interrupt-controller@f900 { + compatible = qcom,msm-qgic2; + interrupt-controller; + #interrupt-cells = 3; + reg = 0xf900 0x1000, + 0xf9002000 0x1000; + }; + + timer { + compatible = arm,armv7-timer; + interrupts = 1 2 0xf08, +1 3 0xf08, +1 4 0xf08, +1 1 0xf08; + clock-frequency = 1920; + }; +}; diff --git a/arch/arm/include/debug/msm.S b/arch/arm/include/debug/msm.S index 9166e1b..9d653d4 100644 --- a/arch/arm/include/debug/msm.S +++ b/arch/arm/include/debug/msm.S @@ -46,6 +46,11 @@ #define MSM_DEBUG_UART_PHYS 0x1644 #endif +#ifdef CONFIG_DEBUG_MSM8974_UART +#define MSM_DEBUG_UART_BASE0xFA71E000 +#define MSM_DEBUG_UART_PHYS0xF991E000 +#endif + .macro addruart, rp, rv, tmp #ifdef MSM_DEBUG_UART_PHYS ldr \rp, =MSM_DEBUG_UART_PHYS diff --git a/arch/arm/mach-msm/Kconfig b/arch/arm/mach-msm/Kconfig index 2586c28..086bcb9 100644 --- a/arch/arm/mach-msm/Kconfig +++ b/arch/arm/mach-msm/Kconfig @@ -64,6 +64,19 @@ config ARCH_MSM_DT select SPARSE_IRQ select USE_OF +config ARCH_MSM8974 +
Re: [PATCH 1/2] i2c: qup: Add device tree bindings information
Hi, On Wed, 2013-09-25 at 18:06 +0200, Wolfram Sang wrote: I agree with Kumar on removing this. If we decide it is something worth keeping, logic to support it doesn't belong in the QUP driver, but in the i2c core. I don't have strong objection here, will remove aliases support. When resubmitting, please test against v3.12-rcX. of_i2c_register_devices() is in the core meanwhile and alias support for of is in the core for a while, too. Check i2c_add_adapter(). Thanks, I will. Regards, Ivan -- 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