[PATCH v3 3/3] mailbox: sprd: Add supplementary inbox support

2021-03-19 Thread Orson Zhai
From: Orson Zhai Some sensors connected to Unisoc mailbox will send data very frequently. This makes channel 0 very busy and the messages from other remote cores not able to be handled as soon as possible. It's a trick (un-documented) from Unisoc ASIC designers to resolve this special

[PATCH v3 2/3] dt-bindings: mailbox: Add interrupt-names to SPRD mailbox

2021-03-19 Thread Orson Zhai
From: Orson Zhai We add an optional supp-outbox interrupt support to SPRD mailbox driver with newly added sc9863a support and change to configure interrupts with names in device tree files. Signed-off-by: Orson Zhai --- Documentation/devicetree/bindings/mailbox/sprd-mailbox.yaml | 13

[PATCH v3 1/3] mailbox: sprd: Introduce refcnt when clients requests/free channels

2021-03-19 Thread Orson Zhai
From: Orson Zhai Unisoc mailbox has no way to be enabled/disabled for any single channel. They can only be set to startup or shutdown as a whole device at same time. Add a variable to count references to avoid mailbox FIFO being reset unexpectedly when clients are requesting or freeing channels

[PATCH v3 0/3] Fixes & a new supplementary feature to SPRD mailbox driver

2021-03-19 Thread Orson Zhai
From: Orson Zhai Fix a real problem for SPRD's mailbox driver in patch 1/3. Add supplementary inbox support for newly added sc9863a in patch 3/3 and change dt bindings yaml accordingly in patch 2/3. Changes Log: V3: - Add Reviewed-by tag from Baolin for driver code. - Change interrupt-names

Re: [PATCH v2 2/3] dt-bindings: mailbox: Add interrupt-names to SPRD mailbox

2021-03-08 Thread Orson Zhai
Hi Rob, On Fri, Mar 05, 2021 at 03:22:21PM -0600, Rob Herring wrote: > On Sat, Feb 13, 2021 at 08:21:42PM +0800, Orson Zhai wrote: > > From: Orson Zhai > > > > We add an optional supp-outbox interrupt support to SPRD mailbox driver > > with newly added sc9863a supp

Re: [PATCH v2 0/3] Fixes & a new supplementary feature to SPRD mailbox driver

2021-02-25 Thread Orson Zhai
Hi Baolin, On Sat, Feb 13, 2021 at 08:21:40PM +0800, Orson Zhai wrote: > Fix a real problem fot SPRD's mailbox driver in patch 1/3. > Add supplementary inbox support for newly added sc9863a in patch 3/3 and > change dt bindings yaml accordingly in patch 2/3. > > Changes Log: &

Re: [PATCH v4 2/6] drm/sprd: add Unisoc's drm kms master

2021-02-22 Thread Orson Zhai
oc's display internel > peripherals through libdrm or libkms, it's test ok by modetest > (DRM/KMS test tool) and Android HWComposer. > > Cc: Orson Zhai > Cc: Chunyan Zhang > Signed-off-by: Kevin Tang > > v4: > - Move the devm_drm_dev_alloc to master_ops->bind function. >

[PATCH v2 2/3] dt-bindings: mailbox: Add interrupt-names to SPRD mailbox

2021-02-13 Thread Orson Zhai
From: Orson Zhai We add an optional supp-outbox interrupt support to SPRD mailbox driver with newly added sc9863a support and change to configure interrupts with names in device tree files. Signed-off-by: Orson Zhai --- .../bindings/mailbox/sprd-mailbox.yaml | 18 +++--- 1

[PATCH v2 3/3] mailbox: sprd: Add supplementary inbox support

2021-02-13 Thread Orson Zhai
From: Orson Zhai Some sensors connected to Unisoc mailbox will send data very frequently. This makes channel 0 very busy and the messages from other remote cores not able to be handled as soon as possible. It's a trick (un-documented) from Unisoc ASIC designers to resolve this special

[PATCH v2 1/3] mailbox: sprd: Introduce refcnt when clients requests/free channels

2021-02-13 Thread Orson Zhai
From: Orson Zhai Unisoc mailbox has no way to be enabled/disabled for any single channel. They can only be set to startup or shutdown as a whole device at same time. Add a variable to count references to avoid mailbox FIFO being reset unexpectedly when clients are requesting or freeing channels

[PATCH v2 0/3] Fixes & a new supplementary feature to SPRD mailbox driver

2021-02-13 Thread Orson Zhai
sage. (sc9860 is not supported by supp-inbox) - Add more details to supp-inbox in commit messages. Orson Zhai (3): mailbox: sprd: Introduce refcnt when clients requests/free channels dt-bindings: mailbox: Add interrupt-names to SPRD mailbox mailbox: sprd: Add supplementary inbox support

Re: [PATCH 2/3] mailbox: sprd: Add supplementary inbox support

2021-02-08 Thread Orson Zhai
On Mon, Feb 08, 2021 at 10:27:47PM +0800, Baolin Wang wrote: > On Mon, Feb 8, 2021 at 7:52 PM Orson Zhai wrote: > > > > From: Orson Zhai > > > > Some sensors connected to Unisoc mailbox will send data very frequently. > > This makes channel 0 very busy and the

Re: [PATCH 1/3] mailbox: sprd: Introduce refcnt when clients requests/free channels

2021-02-08 Thread Orson Zhai
On Mon, Feb 08, 2021 at 10:06:47PM +0800, Baolin Wang wrote: > Hi Orson, > > On Mon, Feb 8, 2021 at 7:52 PM Orson Zhai wrote: > > > > From: Orson Zhai > > > > Unisoc mailbox has no way to be enabled/disabled for any single channel. > > They can only b

[PATCH 3/3] dt-bindings: mailbox: Add interrupt-names to SPRD mailbox

2021-02-08 Thread Orson Zhai
From: Orson Zhai We add an optional supp-outbox interrupt support to driver and change to describe interrupts with names for easy configuration in device tree files. Signed-off-by: Orson Zhai --- Documentation/devicetree/bindings/mailbox/sprd-mailbox.yaml | 12 ++-- 1 file changed, 10

[PATCH 1/3] mailbox: sprd: Introduce refcnt when clients requests/free channels

2021-02-08 Thread Orson Zhai
From: Orson Zhai Unisoc mailbox has no way to be enabled/disabled for any single channel. They can only be set to startup or shutdown as a whole device at same time. Add a variable to count references to avoid mailbox FIFO being reset unexpectedly when clients are requesting or freeing channels

[PATCH 2/3] mailbox: sprd: Add supplementary inbox support

2021-02-08 Thread Orson Zhai
From: Orson Zhai Some sensors connected to Unisoc mailbox will send data very frequently. This makes channel 0 very busy and the messages from other remote cores not able to be handled as soon as possible. Then a supplementary inbox is added to the host core side for transferring mass

Re: [PATCH] include: sysfs: Add macro to assign show for RO attributes

2021-01-27 Thread Orson Zhai
On Wed, Jan 27, 2021 at 08:50:28AM +0100, Greg Kroah-Hartman wrote: > On Wed, Jan 27, 2021 at 12:19:22PM +0800, Orson Zhai wrote: > > In some circumstances, multiple __ATTR_RO attributes need to be assigned > > with a single show function. > > > > Add this macro to m

[PATCH] include: sysfs: Add macro to assign show for RO attributes

2021-01-26 Thread Orson Zhai
In some circumstances, multiple __ATTR_RO attributes need to be assigned with a single show function. Add this macro to make life easier with simple code. Signed-off-by: Orson Zhai --- Documentation/filesystems/sysfs.rst | 2 ++ include/linux/sysfs.h | 5 + 2 files changed, 7

Re: [PATCH] mmc: sdhci-sprd: Fix some resource leaks in the remove function

2020-12-19 Thread Orson Zhai
On Fri, Dec 18, 2020 at 9:46 PM Christophe JAILLET wrote: > > Le 17/12/2020 à 23:55, Orson Zhai a écrit : > > + cc: Billows > > > > Hi Christophe, > > On Fri, Dec 18, 2020 at 4:50 AM Christophe JAILLET > > wrote: > >> > >> 'sdhci_remove_ho

Re: [PATCH] mmc: sdhci-sprd: Fix some resource leaks in the remove function

2020-12-17 Thread Orson Zhai
+ cc: Billows Hi Christophe, On Fri, Dec 18, 2020 at 4:50 AM Christophe JAILLET wrote: > > 'sdhci_remove_host()' and 'sdhci_pltfm_free()' should be used in place of > 'mmc_remove_host()' and 'mmc_free_host()'. > > This avoids some resource leaks, is more in line with the error handling > path of

Re: [PATCH] add chan->cl check in mbox_chan_received_data()

2020-12-15 Thread Orson Zhai
x.c > +++ b/drivers/mailbox/mailbox.c > @@ -152,7 +152,7 @@ static enum hrtimer_restart txdone_hrtimer(struct hrtimer > *hrtimer) > void mbox_chan_received_data(struct mbox_chan *chan, void *mssg) > { > /* No buffering the received data */ > - if (chan-&

[RFC PATCH] printk: Change timestamp to triplet as mono, boot and real

2020-08-10 Thread Orson Zhai
://lore.kernel.org/lkml/alpine.DEB.2.20.1711142341130.2221@nanos/ [2] https://lore.kernel.org/lkml/ca+55afz-vvpbow0rvdw4en5yhwauzcmvsyzrqec41qdfodi...@mail.gmail.com/ Cc. Thomas Gleixner Cc. Linus Torvalds Cc. Prarit Bhargava Cc. Petr Mladek Signed-off-by: Orson Zhai Tested-by: Cixi Geng

Re: [PATCH] mfd: sprd: get subdevices from DT instead for SC27XX SPI

2020-06-15 Thread Orson Zhai
On Mon, Jun 15, 2020 at 10:12 PM Baolin Wang wrote: > > On Mon, Jun 15, 2020 at 11:47 AM Chunyan Zhang wrote: > > > > From: Chunyan Zhang > > > > SC27XX-SPI added subdevices according to a pre-defined mfd_cell array, > > no matter these devices were really included on board. So with this > >

Re: [PATCH v4 1/2] mfd: syscon: Support physical regmap bus

2020-05-20 Thread Orson Zhai
On Tue, May 19, 2020 at 9:19 PM Arnd Bergmann wrote: > > On Sat, May 16, 2020 at 12:13 PM Baolin Wang wrote: > > > > Some platforms such as Spreadtrum platform, define a special method to > > update bits of the registers instead of reading and writing, which means > > we should use a physical

Re: [PATCH v4 1/2] mfd: syscon: Support physical regmap bus

2020-05-16 Thread Orson Zhai
On Sat, May 16, 2020 at 6:13 PM Baolin Wang wrote: > > Some platforms such as Spreadtrum platform, define a special method to > update bits of the registers instead of reading and writing, which means > we should use a physical regmap bus to define the reg_update_bits() > operation instead of the

Re: [PATCH V2] dynamic_debug: Add an option to enable dynamic debug for modules only

2020-05-15 Thread Orson Zhai
On Fri, May 15, 2020 at 5:55 PM Petr Mladek wrote: > > On Thu 2020-04-23 00:02:48, Orson Zhai wrote: > > On Wed, Apr 22, 2020 at 10:25 PM Leon Romanovsky wrote: > > > > > > On Wed, Apr 22, 2020 at 09:06:08PM +0800, Orson Zhai wrote: > > > > On Tue, Apr

[RFC] tools: selftests: psock_tpacket: skip un-supported tpacket_v3 test

2017-08-31 Thread Orson Zhai
ed-off-by: Orson Zhai <orson.z...@linaro.org> --- tools/testing/selftests/net/psock_tpacket.c | 17 ++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/tools/testing/selftests/net/psock_tpacket.c b/tools/testing/selftests/net/psock_tpacket.c index 7f6cd9fdacf3..

[RFC] tools: selftests: psock_tpacket: skip un-supported tpacket_v3 test

2017-08-31 Thread Orson Zhai
ed-off-by: Orson Zhai --- tools/testing/selftests/net/psock_tpacket.c | 17 ++--- 1 file changed, 14 insertions(+), 3 deletions(-) diff --git a/tools/testing/selftests/net/psock_tpacket.c b/tools/testing/selftests/net/psock_tpacket.c index 7f6cd9fdacf3..f0cfc18c3726 100644 --- a/tools/t

[PATCH] selftests: memfd: Align STACK_SIZE for ARM AArch64 system

2017-08-10 Thread Orson Zhai
b.de> >Signed-off-by: Shuah Khan <shua...@osg.samsung.com> Signed-off-by: Orson Zhai <orson.z...@linaro.org> --- tools/testing/selftests/memfd/fuse_test.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/testing/selftests/memfd/fuse_test.c b/tools/testing/

[PATCH] selftests: memfd: Align STACK_SIZE for ARM AArch64 system

2017-08-10 Thread Orson Zhai
also be 16-byte aligned, otherwise we would >get an error when calling clone(). > >[1] http://lxr.free-electrons.com/source/arch/arm64/kernel/process.c#L265 > >Signed-off-by: Chunyan Zhang >Acked-by: Arnd Bergmann >Signed-off-by: Shuah Khan Signed-of

Re: [RFC PATCH] char: misc: Init misc->list in a safe way

2017-06-27 Thread Orson Zhai
Hi Greg & Arnd, On Tue, Jun 27, 2017 at 08:29:17AM +0200, Greg Kroah-Hartman wrote: > > A: Because it messes up the order in which people normally read text. > Q: Why is top-posting such a bad thing? > A: Top-posting. > Q: What is the most annoying thing in e-mail? > > A: No. > Q: Should I

Re: [RFC PATCH] char: misc: Init misc->list in a safe way

2017-06-27 Thread Orson Zhai
Hi Greg & Arnd, On Tue, Jun 27, 2017 at 08:29:17AM +0200, Greg Kroah-Hartman wrote: > > A: Because it messes up the order in which people normally read text. > Q: Why is top-posting such a bad thing? > A: Top-posting. > Q: What is the most annoying thing in e-mail? > > A: No. > Q: Should I

[PATCH V4] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-26 Thread Orson Zhai
-by: Orson Zhai <orson.z...@linaro.org> Reviewed-by: Sumit Semwal <sumit.sem...@linaro.org> Tested-by: Sumit Semwal <sumit.sem...@linaro.org> --- tools/testing/selftests/sysctl/common_tests | 22 ++ tools/testing/selftests/sysctl/run_numerictests | 2 +- tools

[PATCH V4] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-26 Thread Orson Zhai
-by: Orson Zhai Reviewed-by: Sumit Semwal Tested-by: Sumit Semwal --- tools/testing/selftests/sysctl/common_tests | 22 ++ tools/testing/selftests/sysctl/run_numerictests | 2 +- tools/testing/selftests/sysctl/run_stringtests | 2 +- 3 files changed, 24 insertions(+), 2

[RFC PATCH] char: misc: Init misc->list in a safe way

2017-06-26 Thread Orson Zhai
re. Move the initializion line out of all error branches to avoid any side effect. Signed-off-by: Zhongping Tan <zhongping@spreadtrum.com> Signed-off-by: Orson Zhai <orson.z...@spreadtrum.com> --- drivers/char/misc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/

[RFC PATCH] char: misc: Init misc->list in a safe way

2017-06-26 Thread Orson Zhai
ror branches to avoid any side effect. Signed-off-by: Zhongping Tan Signed-off-by: Orson Zhai --- drivers/char/misc.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/drivers/char/misc.c b/drivers/char/misc.c index c9cd1ea6844a..876e7d57cc6c 100644 --- a/drivers/char/misc.c ++

Re: [PATCH V3] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-23 Thread Orson Zhai
On 23 June 2017 at 23:55, Shuah Khan <sh...@kernel.org> wrote: > Hi Orson, > > On 06/22/2017 10:24 PM, Orson Zhai wrote: >> Sysctl test will fail in some items if the value of /proc/sys/kernel >> /sysctrl_writes_strict is 0 as the default value in kernel older than v4.5

Re: [PATCH V3] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-23 Thread Orson Zhai
On 23 June 2017 at 23:55, Shuah Khan wrote: > Hi Orson, > > On 06/22/2017 10:24 PM, Orson Zhai wrote: >> Sysctl test will fail in some items if the value of /proc/sys/kernel >> /sysctrl_writes_strict is 0 as the default value in kernel older than v4.5. >> &g

[PATCH V3] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-22 Thread Orson Zhai
. Signed-off-by: Orson Zhai <orson.z...@linaro.org> Reviewed-by: Sumit Semwal <sumit.sem...@linaro.org> Tested-by: Sumit Semwal <sumit.sem...@linaro.org> --- tools/testing/selftests/sysctl/common_tests | 22 ++ tools/testing/selftests/sysctl/run_numerictests | 2

[PATCH V3] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-22 Thread Orson Zhai
. Signed-off-by: Orson Zhai Reviewed-by: Sumit Semwal Tested-by: Sumit Semwal --- tools/testing/selftests/sysctl/common_tests | 22 ++ tools/testing/selftests/sysctl/run_numerictests | 2 +- tools/testing/selftests/sysctl/run_stringtests | 2 +- 3 files changed, 24 insertions

Fwd: [PATCH V2] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-22 Thread Orson Zhai
-by: Orson Zhai <orson.z...@linaro.org> Reviewed-by: Sumit Semwal <sumit.sem...@linaro.org> Tested-by: Sumit Semwal <sumit.sem...@linaro.org> --- tools/testing/selftests/sysctl/common_tests | 22 ++ tools/testing/selftests/sysctl/run_numerictests | 2

Fwd: [PATCH V2] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-22 Thread Orson Zhai
-by: Orson Zhai Reviewed-by: Sumit Semwal Tested-by: Sumit Semwal --- tools/testing/selftests/sysctl/common_tests | 22 ++ tools/testing/selftests/sysctl/run_numerictests | 2 +- tools/testing/selftests/sysctl/run_stringtests | 2 +- 3 files changed, 24 insertions(+), 2

Re: [PATCH] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-20 Thread Orson Zhai
Hi Kees, Thanks for reviewing. I will update in V1 soon. Br, Orson On 20 June 2017 at 03:01, Kees Cook <keesc...@chromium.org> wrote: > On Fri, Jun 16, 2017 at 2:28 AM, Orson Zhai <orson.z...@linaro.org> wrote: >> Sysctl test will fail in some items if the valu

Re: [PATCH] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-20 Thread Orson Zhai
Hi Kees, Thanks for reviewing. I will update in V1 soon. Br, Orson On 20 June 2017 at 03:01, Kees Cook wrote: > On Fri, Jun 16, 2017 at 2:28 AM, Orson Zhai wrote: >> Sysctl test will fail in some items if the value of /proc/sys/kernel >> /sysctrl_writes_strict is 0 as th

Re: [PATCH] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-16 Thread Orson Zhai
On 16 June 2017 at 22:49, Sumit Semwal <sumit.sem...@linaro.org> wrote: > Hi Orson, > > Thanks for the patch. > > On 16 June 2017 at 14:58, Orson Zhai <orson.z...@linaro.org> wrote: >> Sysctl test will fail in some items if the value of /proc/sys/kern

Re: [PATCH] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-16 Thread Orson Zhai
On 16 June 2017 at 22:49, Sumit Semwal wrote: > Hi Orson, > > Thanks for the patch. > > On 16 June 2017 at 14:58, Orson Zhai wrote: >> Sysctl test will fail in some items if the value of /proc/sys/kernel >> /sysctrl_writes_strict is 0 as the default value in kernel

Fwd: [PATCH] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-16 Thread Orson Zhai
. Signed-off-by: Orson Zhai <orson.z...@linaro.org> --- tools/testing/selftests/sysctl/common_tests | 14 ++ tools/testing/selftests/sysctl/run_numerictests | 3 +++ tools/testing/selftests/sysctl/run_stringtests | 3 +++ 3 files changed, 20 insertions(+) diff --git a/tools/t

Fwd: [PATCH] tools/testing/selftests/sysctl: Add pre-check to the value of writes_strict

2017-06-16 Thread Orson Zhai
. Signed-off-by: Orson Zhai --- tools/testing/selftests/sysctl/common_tests | 14 ++ tools/testing/selftests/sysctl/run_numerictests | 3 +++ tools/testing/selftests/sysctl/run_stringtests | 3 +++ 3 files changed, 20 insertions(+) diff --git a/tools/testing/selftests/sysctl

Re: [PATCH] iio: core: Fix suspicious sizeof usage

2017-04-27 Thread Orson Zhai
On Thu, Apr 27, 2017 at 06:32:22AM +0100, Jonathan Cameron wrote: > On 25/04/17 02:16, Orson Zhai wrote: > > Pointer size is variours in different system, say 32bit for 4 and 64bit > > for 8. The 'sizeof(infomask)' may lead to wrong bit numbers. > > > > Signed-

Re: [PATCH] iio: core: Fix suspicious sizeof usage

2017-04-27 Thread Orson Zhai
On Thu, Apr 27, 2017 at 06:32:22AM +0100, Jonathan Cameron wrote: > On 25/04/17 02:16, Orson Zhai wrote: > > Pointer size is variours in different system, say 32bit for 4 and 64bit > > for 8. The 'sizeof(infomask)' may lead to wrong bit numbers. > > > > Signed-o

[PATCH] iio: core: Fix suspicious sizeof usage

2017-04-24 Thread Orson Zhai
Pointer size is variours in different system, say 32bit for 4 and 64bit for 8. The 'sizeof(infomask)' may lead to wrong bit numbers. Signed-off-by: Orson Zhai <orson.z...@spreadtrum.com> --- drivers/iio/industrialio-core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff

[PATCH] iio: core: Fix suspicious sizeof usage

2017-04-24 Thread Orson Zhai
Pointer size is variours in different system, say 32bit for 4 and 64bit for 8. The 'sizeof(infomask)' may lead to wrong bit numbers. Signed-off-by: Orson Zhai --- drivers/iio/industrialio-core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/iio/industrialio

[PATCH] clk: Register clkdev after setup of fixed-rate and fixed-factor clocks

2016-10-11 Thread Orson Zhai
fixed-rate and fixed-factor clock and ignore the error if any. Signed-off-by: Xiaolong Zhang <xiaolong.zh...@spreadtrum.com> Signed-off-by: Orson Zhai <orson.z...@spreadtrum.com> --- drivers/clk/clk-fixed-factor.c | 5 + drivers/clk/clk-fixed-rate.c | 5 + 2 files chan

[PATCH] clk: Register clkdev after setup of fixed-rate and fixed-factor clocks

2016-10-11 Thread Orson Zhai
and ignore the error if any. Signed-off-by: Xiaolong Zhang Signed-off-by: Orson Zhai --- drivers/clk/clk-fixed-factor.c | 5 + drivers/clk/clk-fixed-rate.c | 5 + 2 files changed, 10 insertions(+) diff --git a/drivers/clk/clk-fixed-factor.c b/drivers/clk/clk-fixed-factor.c index a5d402d

Re: [RFC PATCH] mmc: sprd: add MMC host driver for Spreadtrum SoC

2015-07-03 Thread Orson Zhai
some logs in work-queue as "2060.sdio detecting... card type: emmc" -orson If you have a good opinion how to distinguish MMC form SD or SDIO device in host controller probe function not by physical address, please tell us. Thanks! --billows _______ Fr

Re: [RFC PATCH] mmc: sprd: add MMC host driver for Spreadtrum SoC

2015-07-03 Thread Orson Zhai
in work-queue as 2060.sdio detecting... card type: emmc -orson If you have a good opinion how to distinguish MMC form SD or SDIO device in host controller probe function not by physical address, please tell us. Thanks! --billows ___ From: Orson Zhai orsonz

Re: [RFC PATCH] mmc: sprd: add MMC host driver for Spreadtrum SoC

2015-07-02 Thread Orson Zhai
On Thu, Jul 2, 2015 at 11:47 PM, Rob Herring wrote: > On Thu, Jul 2, 2015 at 4:26 AM, Orson Zhai wrote: >> Hi Rob, >> >> Thanks for your quick reply. >> I have one question to your last comment. >> >> On Thu, Jul 2, 2015 at 1:50 AM, Rob Herring wrote: >

Re: [RFC PATCH] mmc: sprd: add MMC host driver for Spreadtrum SoC

2015-07-02 Thread Orson Zhai
C, SD, and >> SDIO types of memory cards. >> >> Signed-off-by: Billows Wu >> Reviewed-by: Orson Zhai >> Signed-off-by: Chunyan Zhang >> --- >> drivers/mmc/host/sprd_sdhost.c | 1270 >> >> dr

Re: [RFC PATCH] mmc: sprd: add MMC host driver for Spreadtrum SoC

2015-07-02 Thread Orson Zhai
MMC host driver is used to support EMMC, SD, and SDIO types of memory cards. Signed-off-by: Billows Wu billows...@spreadtrum.com Reviewed-by: Orson Zhai orson.z...@spreadtrum.com Signed-off-by: Chunyan Zhang chunyan.zh...@spreadtrum.com --- drivers/mmc/host/sprd_sdhost.c | 1270

Re: [RFC PATCH] mmc: sprd: add MMC host driver for Spreadtrum SoC

2015-07-02 Thread Orson Zhai
On Thu, Jul 2, 2015 at 11:47 PM, Rob Herring robherri...@gmail.com wrote: On Thu, Jul 2, 2015 at 4:26 AM, Orson Zhai orsonz...@gmail.com wrote: Hi Rob, Thanks for your quick reply. I have one question to your last comment. On Thu, Jul 2, 2015 at 1:50 AM, Rob Herring robherri...@gmail.com

Re: [PATCH v5 5/5] tty/serial: Add Spreadtrum sc9836-uart driver support

2015-01-20 Thread Orson Zhai
. This patch also replaced the spaces between the macros and their values with the tabs in serial_core.h The locking doesn't look correct. Specific notations below. Signed-off-by: Chunyan Zhang Signed-off-by: Orson Zhai Originally-by: Lanqing Liu --- drivers/tty/serial/Kconfig | 18

Re: [PATCH v5 5/5] tty/serial: Add Spreadtrum sc9836-uart driver support

2015-01-20 Thread Orson Zhai
On Tue, Jan 20, 2015 at 3:37 PM, Lyra Zhang wrote: > Hi, Rob > > I still have a question to be conform, specific describes below: > > On Mon, Jan 19, 2015 at 10:11 PM, Rob Herring wrote: >> On Mon, Jan 19, 2015 at 3:55 AM, Lyra Zhang wrote: >>> On Sat, Jan 17, 2015 at 12:41 AM, Rob Herring

Re: [PATCH v5 5/5] tty/serial: Add Spreadtrum sc9836-uart driver support

2015-01-20 Thread Orson Zhai
On Tue, Jan 20, 2015 at 3:37 PM, Lyra Zhang zhang.l...@gmail.com wrote: Hi, Rob I still have a question to be conform, specific describes below: On Mon, Jan 19, 2015 at 10:11 PM, Rob Herring robherri...@gmail.com wrote: On Mon, Jan 19, 2015 at 3:55 AM, Lyra Zhang zhang.l...@gmail.com wrote:

Re: [PATCH v5 5/5] tty/serial: Add Spreadtrum sc9836-uart driver support

2015-01-20 Thread Orson Zhai
. This patch also replaced the spaces between the macros and their values with the tabs in serial_core.h The locking doesn't look correct. Specific notations below. Signed-off-by: Chunyan Zhang chunyan.zh...@spreadtrum.com Signed-off-by: Orson Zhai orson.z...@spreadtrum.com Originally-by: Lanqing Liu

Re: [PATCH v5 3/5] arm64: dts: Add support for Spreadtrum SC9836 SoC in dts and Makefile

2015-01-17 Thread Orson Zhai
On Fri, Jan 16, 2015 at 10:09 PM, Mark Rutland wrote: > On Fri, Jan 16, 2015 at 12:49:20PM +0000, Orson Zhai wrote: >> Hi, Mark, >> >> On Fri, Jan 16, 2015 at 6:35 PM, Mark Rutland wrote: >[...] >> >> It returns "status busy" after I type t

Re: [PATCH v5 2/5] Documentation: DT: Add bindings for Spreadtrum SoC Platform

2015-01-17 Thread Orson Zhai
On Fri, Jan 16, 2015 at 10:11 PM, Mark Rutland wrote: > On Fri, Jan 16, 2015 at 12:53:16PM +, Lyra Zhang wrote: >> Hi, Mark >> >> >> + >> >> +Required properties: >> >> +- compatible: must be "sprd,sc9836-uart" >> >> +- reg: offset and length of the register set for the device >> >> +-

Re: [PATCH v5 2/5] Documentation: DT: Add bindings for Spreadtrum SoC Platform

2015-01-17 Thread Orson Zhai
On Fri, Jan 16, 2015 at 10:11 PM, Mark Rutland mark.rutl...@arm.com wrote: On Fri, Jan 16, 2015 at 12:53:16PM +, Lyra Zhang wrote: Hi, Mark + +Required properties: +- compatible: must be sprd,sc9836-uart +- reg: offset and length of the register set for the device +- interrupts:

Re: [PATCH v5 3/5] arm64: dts: Add support for Spreadtrum SC9836 SoC in dts and Makefile

2015-01-17 Thread Orson Zhai
On Fri, Jan 16, 2015 at 10:09 PM, Mark Rutland mark.rutl...@arm.com wrote: On Fri, Jan 16, 2015 at 12:49:20PM +, Orson Zhai wrote: Hi, Mark, On Fri, Jan 16, 2015 at 6:35 PM, Mark Rutland mark.rutl...@arm.com wrote: [...] It returns status busy after I type the command below

Re: [PATCH v5 3/5] arm64: dts: Add support for Spreadtrum SC9836 SoC in dts and Makefile

2015-01-16 Thread Orson Zhai
; Sharkl64 platform contains the common nodes of Spreadtrum's arm64-based SoCs. >> >> Signed-off-by: Zhizhou Zhang >> Signed-off-by: Orson Zhai >> Signed-off-by: Chunyan Zhang >> --- >> arch/arm64/boot/dts/Makefile |1 + >> arc

Re: [PATCH v5 3/5] arm64: dts: Add support for Spreadtrum SC9836 SoC in dts and Makefile

2015-01-16 Thread Orson Zhai
. Sharkl64 platform contains the common nodes of Spreadtrum's arm64-based SoCs. Signed-off-by: Zhizhou Zhang zhizhou.zh...@spreadtrum.com Signed-off-by: Orson Zhai orson.z...@spreadtrum.com Signed-off-by: Chunyan Zhang chunyan.zh...@spreadtrum.com --- arch/arm64/boot/dts/Makefile

Re: [PATCH v4 0/5] Add Spreadtrum Sharkl64 Platform support

2014-12-05 Thread Orson Zhai
Hi, Mark, On Fri, Dec 5, 2014 at 8:27 PM, Lyra Zhang wrote: > On Fri, Dec 5, 2014 at 6:40 PM, Mark Rutland wrote: >> Hi, >> >> On Thu, Dec 04, 2014 at 11:34:15AM +, Chunyan Zhang wrote: >>> Spreadtrum is a rapid growing chip vendor providing smart phone total >>> solutions. >>> >>>

Re: [PATCH v4 0/5] Add Spreadtrum Sharkl64 Platform support

2014-12-05 Thread Orson Zhai
Hi, Mark, On Fri, Dec 5, 2014 at 8:27 PM, Lyra Zhang zhang.l...@gmail.com wrote: On Fri, Dec 5, 2014 at 6:40 PM, Mark Rutland mark.rutl...@arm.com wrote: Hi, On Thu, Dec 04, 2014 at 11:34:15AM +, Chunyan Zhang wrote: Spreadtrum is a rapid growing chip vendor providing smart phone total

Re: [PATCH v3 5/5] tty/serial: Add Spreadtrum sc9836-uart driver support

2014-11-28 Thread Orson Zhai
Hi, Alan, Thanks for your comments! Some question for them below, On 2014年11月26日 20:33, One Thousand Gnomes wrote: +213 = /dev/ttySPX0 SPRD serial port 0 + ... +216 = /dev/ttySPX3 SPRD serial port 3 Please use dynamic

Re: [PATCH v3 5/5] tty/serial: Add Spreadtrum sc9836-uart driver support

2014-11-28 Thread Orson Zhai
Hi, Alan, Thanks for your comments! Some question for them below, On 2014年11月26日 20:33, One Thousand Gnomes wrote: +213 = /dev/ttySPX0 SPRD serial port 0 + ... +216 = /dev/ttySPX3 SPRD serial port 3 Please use dynamic

Re: [PATCH v2 5/5] tty/serial: Add earlycon support for Spreadtrum serial driver

2014-10-21 Thread Orson Zhai
On Tue, Oct 21, 2014 at 8:26 PM, One Thousand Gnomes wrote: > On Mon, 20 Oct 2014 18:23:38 +0800 > Orson Zhai wrote: > >> On Sun, Oct 19, 2014 at 5:06 AM, One Thousand Gnomes >> wrote: >> > On Fri, 17 Oct 2014 17:54:25 +0800 >> > Chunyan Zhang wrote: >

Re: [PATCH v2 5/5] tty/serial: Add earlycon support for Spreadtrum serial driver

2014-10-21 Thread Orson Zhai
On Tue, Oct 21, 2014 at 8:26 PM, One Thousand Gnomes gno...@lxorguk.ukuu.org.uk wrote: On Mon, 20 Oct 2014 18:23:38 +0800 Orson Zhai orsonz...@gmail.com wrote: On Sun, Oct 19, 2014 at 5:06 AM, One Thousand Gnomes gno...@lxorguk.ukuu.org.uk wrote: On Fri, 17 Oct 2014 17:54:25 +0800 Chunyan

Re: [PATCH v2 5/5] tty/serial: Add earlycon support for Spreadtrum serial driver

2014-10-20 Thread Orson Zhai
On Sun, Oct 19, 2014 at 5:06 AM, One Thousand Gnomes wrote: > On Fri, 17 Oct 2014 17:54:25 +0800 > Chunyan Zhang wrote: > >> Add serial driver for spreadtrum sharkl platform with earlycon >> support at first. > >> +#define UART_TXD 0x >> +#define UART_RXD 0x0004 >> +#define UART_STS0

Re: [PATCH v2 5/5] tty/serial: Add earlycon support for Spreadtrum serial driver

2014-10-20 Thread Orson Zhai
On Sun, Oct 19, 2014 at 5:06 AM, One Thousand Gnomes gno...@lxorguk.ukuu.org.uk wrote: On Fri, 17 Oct 2014 17:54:25 +0800 Chunyan Zhang chunyan.zh...@spreadtrum.com wrote: Add serial driver for spreadtrum sharkl platform with earlycon support at first. +#define UART_TXD 0x +#define

Re: [PATCH 1/6] Documentation: DT: Add bindings for Spreadtrum serial

2014-10-02 Thread Orson Zhai
On 2014年10月01日 19:21, Mark Rutland wrote: diff --git a/Documentation/devicetree/bindings/serial/of-serial.txt b/Documentation/devicetree/bindings/serial/of-serial.txt >index 7705477..a0bb1d1 100644 >--- a/Documentation/devicetree/bindings/serial/of-serial.txt >+++

Re: [PATCH 1/6] Documentation: DT: Add bindings for Spreadtrum serial

2014-10-02 Thread Orson Zhai
On 2014年10月01日 19:21, Mark Rutland wrote: diff --git a/Documentation/devicetree/bindings/serial/of-serial.txt b/Documentation/devicetree/bindings/serial/of-serial.txt index 7705477..a0bb1d1 100644 --- a/Documentation/devicetree/bindings/serial/of-serial.txt +++