Re: [PATCH] powerpc/mpc85xx: fix non-bootcpu cannot up after hibernation resume

2013-05-23 Thread Anton Vorontsov
Hi! On Tue, May 14, 2013 at 08:59:13AM +, Wang Dongsheng-B40534 wrote: > I send to a wrong email address "Anton Vorontsov " > > Add Anton Vorontsov to this email. I don't have any means to test it, but the patch itself looks good and the description makes sense.

Re: [PATCH 3/4 V2] mmc: esdhc: Add quirks to support T4240QDS board

2013-07-12 Thread Anton Vorontsov
On Mon, Jul 08, 2013 at 12:18:39PM -0500, Scott Wood wrote: > On 07/08/2013 02:16:04 AM, Haijun Zhang wrote: > >On T4240QDS board controllers has an unusable ADMA engine, so use > >SDMA instead. > >Also 3.0v is support on T4240QDS board even if the capacity > >detailed only 1.8v > >support. Without

Re: [PATCH 3/4 V2] mmc: esdhc: Add quirks to support T4240QDS board

2013-07-15 Thread Anton Vorontsov
knows better, otherwise read capabilities from the register. Anton > Thanks. > > Regards > Haijun. > > > > -Original Message- > > From: linux-mmc-ow...@vger.kernel.org [mailto:linux-mmc- > > ow...@vger.kernel.org] On Behalf Of Anton Vorontsov > &

Re: [PATCH 2/2] mmc: esdhc: get voltage from dts file

2013-07-26 Thread Anton Vorontsov
On Mon, Jul 22, 2013 at 09:41:34PM -0500, Scott Wood wrote: [...] > >> > +static void esdhc_get_voltage(struct sdhci_host *host, > >> > +struct platform_device *pdev) > >> > +{ > >> > +} > >> > >> Don't duplicate this code. Move it somewhere common and share it. > >[Ha

Re: [PATCH 2/2 V2] mmc: esdhc: get voltage from dts file

2013-07-26 Thread Anton Vorontsov
lse we still read > from capacity or from other provider. > > Signed-off-by: Haijun Zhang > Signed-off-by: Anton Vorontsov Development process nitpick... The code originated from me, but I did not sign off this patch... Per Documentation/SubmittingPatches: The Signed-off-by: tag i

Re: [PATCH] mmc:of_spi: Update the code of getting voltage-ranges

2013-08-08 Thread Anton Vorontsov
s unspecified\n"); > + ocr_mask = mmc_of_parse_voltage(np); > + if (ocr_mask <= 0) '< 0' check for an unsigned type? :) I'd write just !ocr_mask... But other than that the patch looks good to me... Reviewed-by: Anton Vorontsov Thanks! >

Re: [PATCH 1/3 V2] mmc:core: parse voltage from device-tree

2013-08-08 Thread Anton Vorontsov
On Wed, Jul 31, 2013 at 02:25:25PM +0800, Haijun Zhang wrote: > Add function to support get voltage from device-tree. > If there are voltage-range specified in device-tree node, this function > will parse it and return the avail voltage mask. > > Signed-off-by: Haijun Zhang > --- > changes for v2

Re: [PATCH 1/3 V5] mmc:core: parse voltage from device-tree

2013-08-22 Thread Anton Vorontsov
On Wed, Aug 14, 2013 at 01:46:11PM +0800, Haijun Zhang wrote: > Add function to support get voltage from device-tree. > If there are voltage-range specified in device-tree node, this function > will parse it and return the available voltage mask. > > Signed-off-by: Haijun Zhang

Re: [PATCH V3] mmc:of_spi: Update the code of getting voltage-ranges

2013-08-22 Thread Anton Vorontsov
On Mon, Aug 12, 2013 at 09:39:05AM +0800, Haijun Zhang wrote: > Using function mmc_of_parse_voltage() to get voltage-ranges. > > Signed-off-by: Haijun Zhang > --- Acked-by: Anton Vorontsov ___ Linuxppc-dev mailing list Linuxppc-dev@list

Re: [PATCH 3/3 V3] mmc:esdhc: add support to get voltage from device-tree

2013-08-22 Thread Anton Vorontsov
et from host capacity register. > > Signed-off-by: Haijun Zhang Acked-by: Anton Vorontsov ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH 2/3 V3] mmc:sdhc: get voltage from sdhc host

2013-08-22 Thread Anton Vorontsov
ned-off-by: Haijun Zhang > --- Reviewed-by: Anton Vorontsov ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [RFC PATCH 1/7] mmc: sdhci: add quirk for broken 3.0V support

2014-06-20 Thread Anton Vorontsov
On Fri, Jun 20, 2014 at 05:35:22PM +0800, Vincent Yang wrote: > This patch defines a quirk for platforms unable > to enable 3.0V support. > It is a preparation and will be used by Fujitsu > SDHCI controller f_sdh30 driver. > > Signed-off-by: Vincent Yang I don't think you need this patch. Instea

Re: [PATCH] eSDHC: Access Freescale eSDHC registers by 32-bit

2011-09-09 Thread Anton Vorontsov
; Signed-off-by: Kumar Gala > --- The patch looks OK. Acked-by: Anton Vorontsov [...] > +static u8 esdhc_readb(struct sdhci_host *host, int reg) > +{ > + int base = reg & ~0x3; > + int shift = (reg & 0x3) * 8; > + u8 ret = (in_be32(host->ioaddr + base) >

Re: [PATCH 2/3] powerpc/esdhc: add property to disable the CMD23

2012-09-11 Thread Anton Vorontsov
On Tue, Sep 11, 2012 at 12:54:29AM -0700, Anton Vorontsov wrote: > On Tue, Sep 11, 2012 at 03:12:44PM +0800, chang-ming.hu...@freescale.com > wrote: > > From: Jerry Huang > > > > Below SOCs don't support the cmd23 command for MMC card, > > therefore, disable

Re: [PATCH 2/3] powerpc/esdhc: add property to disable the CMD23

2012-09-11 Thread Anton Vorontsov
On Wed, Sep 12, 2012 at 03:19:18AM +, Huang Changming-R66093 wrote: [...] > I don't think it is the best way to do it. For the VVN2.2 or older, > some silicon support this feature (mpc8536 and p2020), but other > silicones don't support it (e.g. p4080, p102x). Though, the current > p5/p4/p3 h

Re: [PATCH] powerpc/esdhc: enable the card insert/remove interrupt

2012-10-23 Thread Anton Vorontsov
e the default card detect to interrupt mode, > if the board can't support this mode, we still use the poll mode. > > Signed-off-by: Jerry Huang > CC: Anton Vorontsov > CC: Chris Ball > --- IIRC, the card detection is broken SOC-revision-wise, not board-wise. So the chang

Re: [PATCH] powerpc/esdhc: enable the card insert/remove interrupt

2012-10-25 Thread Anton Vorontsov
e boards, but left others to suffer. Ideally, the best fix would be to also make the card polling cheap. Anyways, using (d) clause of the "Reviewer's statement of oversight", I can easily give this: Reviewed-by: Anton Vorontsov :) Thanks! [...] > > > IIRC, the

Re: [PATCH] powerpc/esdhc: enable the card insert/remove interrupt

2012-10-26 Thread Anton Vorontsov
Hello Huang, On Fri, Oct 26, 2012 at 02:42:36AM +, Huang Changming-R66093 wrote: > For the current polling mode, driver will send CMD13 to poll the card status > periodically , which will cause too many interrupts. > Once I sent patches to detect the card when using polling mode last year: >

[PATCH v2 0/10] Fixes for common mistakes w/ for_each_process and task->mm

2012-03-24 Thread Anton Vorontsov
o it's better to fix it and move the logic into a common function. Thanks! -- Anton Vorontsov Email: cbouatmai...@gmail.com ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

[PATCH 01/10] cpu: Introduce clear_tasks_mm_cpumask() helper

2012-03-24 Thread Anton Vorontsov
); 2. To catch exited main thread case, we use find_lock_task_mm(), which walks up all threads and returns an appropriate task (with task lock held). Signed-off-by: Anton Vorontsov --- include/linux/cpu.h |1 + kernel/cpu.c| 18 ++ 2 files changed, 19 insert

[PATCH 02/10] arm: Use clear_tasks_mm_cpumask()

2012-03-24 Thread Anton Vorontsov
_lock_task_mm(), which would walk up all threads and returns an appropriate task (with task lock held). clear_tasks_mm_cpumask() has all the issues fixed, so let's use it. Suggested-by: Oleg Nesterov Signed-off-by: Anton Vorontsov --- arch/arm/kernel/smp.c |8 +--- 1 file

[PATCH 03/10] powerpc: Use clear_tasks_mm_cpumask()

2012-03-24 Thread Anton Vorontsov
_lock_task_mm(), which would walk up all threads and returns an appropriate task (with task lock held). clear_tasks_mm_cpumask() has all the issues fixed, so let's use it. Suggested-by: Oleg Nesterov Signed-off-by: Anton Vorontsov --- arch/powerpc/mm/mmu_context_nohash.c | 11 ++-

[PATCH 04/10] sh: Use clear_tasks_mm_cpumask()

2012-03-24 Thread Anton Vorontsov
_lock_task_mm(), which would walk up all threads and returns an appropriate task (with task lock held). clear_tasks_mm_cpumask() has all the issues fixed, so let's use it. Suggested-by: Oleg Nesterov Signed-off-by: Anton Vorontsov --- arch/sh/kernel/smp.c |7 +-- 1 file changed,

[PATCH 05/10] blackfin: A couple of task->mm handling fixes

2012-03-24 Thread Anton Vorontsov
eads and returns an appropriate task (with task lock held). Suggested-by: Oleg Nesterov Signed-off-by: Anton Vorontsov --- arch/blackfin/kernel/trace.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/arch/blackfin/kernel/trace.c b/arch/blackfin/k

[PATCH 06/10] blackfin: Fix possible deadlock in decode_address()

2012-03-24 Thread Anton Vorontsov
ite_ variant of the tasklist lock, read_ one is enough. Suggested-by: Oleg Nesterov Signed-off-by: Anton Vorontsov --- arch/blackfin/kernel/trace.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/blackfin/kernel/trace.c b/arch/blackfin/kernel/trace.c index 5102

[PATCH 07/10] um: Should hold tasklist_lock while traversing processes

2012-03-24 Thread Anton Vorontsov
Traversing the tasks requires holding tasklist_lock, otherwise it is unsafe. p.s. However, I'm not sure that calling os_kill_ptraced_process() in the atomic context is correct. It seem to work, but please take a closer look. Signed-off-by: Anton Vorontsov --- arch/um/kernel/reboot.c |

[PATCH 08/10] um: Fix possible race on task->mm

2012-03-24 Thread Anton Vorontsov
should also use find_lock_task_mm() to check all process' threads for a valid mm, but for uml we'll do it in a separate patch. Signed-off-by: Anton Vorontsov --- arch/um/kernel/reboot.c |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/arch/um/kernel/reboot.c b/arc

[PATCH 09/10] um: Properly check all process' threads for a live mm

2012-03-24 Thread Anton Vorontsov
turns an appropriate task (with task lock held). Suggested-by: Oleg Nesterov Signed-off-by: Anton Vorontsov --- arch/um/kernel/reboot.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/arch/um/kernel/reboot.c b/arch/um/kernel/reboot.c index 1411f4e..3d15243 1006

[PATCH 10/10] oom: Make find_lock_task_mm() sparse-aware

2012-03-24 Thread Anton Vorontsov
e in 'task_in_mem_cgroup' - unexpected unlock p.s. I know Peter Zijlstra detest the __cond_lock() stuff, but untill we have anything better in sparse, let's use it. This particular patch helped me to detect one bug that I myself made during task->mm fixup series. S

Re: [PATCH 10/10] oom: Make find_lock_task_mm() sparse-aware

2012-03-24 Thread Anton Vorontsov
d_locked(l, __ret_value); extern struct task_struct *find_lock_task_mm(struct task_struct *p) __ret_locked_nonnull(&__ret_value->alloc_lock); Thanks, -- Anton Vorontsov Email: cbouatmai...@gmail.com ___ Linuxppc-dev mailing list Linu

[PATCH v2.1 01/10] cpu: Introduce clear_tasks_mm_cpumask() helper

2012-03-24 Thread Anton Vorontsov
this because the function is called after the cpu is taken down and marked offline, so no new tasks will get this cpu set in their mm mask. Signed-off-by: Anton Vorontsov --- On Sat, Mar 24, 2012 at 01:43:41PM +0100, Peter Zijlstra wrote: > On Sat, 2012-03-24 at 14:27 +0400, Anton

Re: [PATCH 07/10] um: Should hold tasklist_lock while traversing processes

2012-03-24 Thread Anton Vorontsov
On Sat, Mar 24, 2012 at 01:48:23PM +0100, Peter Zijlstra wrote: > On Sat, 2012-03-24 at 14:30 +0400, Anton Vorontsov wrote: > > Traversing the tasks requires holding tasklist_lock, otherwise it > > is unsafe. > > No it doesn't, it either requires tasklist_lock

[PATCH v3 0/9] Fixes for common mistakes w/ for_each_process and task->mm

2012-04-23 Thread Anton Vorontsov
ly the rcu read lock, no need for the whole tasklist_lock. Suggested by Peter Zijlstra. In v2: - introduced a small helper in cpu.c: most arches duplicate the same [buggy] code snippet, so it's better to fix it and move the logic into a common function. Thanks, -- Anton Vor

[PATCH 1/9] cpu: Introduce clear_tasks_mm_cpumask() helper

2012-04-23 Thread Anton Vorontsov
he rcu read lock. We can do this because the function is called after the cpu is taken down and marked offline, so no new tasks will get this cpu set in their mm mask. Signed-off-by: Anton Vorontsov --- include/linux/cpu.h |1 + kernel/cpu.c| 26 ++ 2 files

[PATCH 2/9] arm: Use clear_tasks_mm_cpumask()

2012-04-23 Thread Anton Vorontsov
ock held). clear_tasks_mm_cpumask() has this issue fixed, so let's use it. Suggested-by: Oleg Nesterov Signed-off-by: Anton Vorontsov --- arch/arm/kernel/smp.c |8 +--- 1 file changed, 1 insertion(+), 7 deletions(-) diff --git a/arch/arm/kernel/smp.c b/arch/arm/kernel/smp.c index add

[PATCH 3/9] powerpc: Use clear_tasks_mm_cpumask()

2012-04-23 Thread Anton Vorontsov
_lock_task_mm(), which would walk up all threads and returns an appropriate task (with task lock held). clear_tasks_mm_cpumask() has all the issues fixed, so let's use it. Suggested-by: Oleg Nesterov Signed-off-by: Anton Vorontsov --- arch/powerpc/mm/mmu_context_nohash.c | 11 ++-

[PATCH 4/9] sh: Use clear_tasks_mm_cpumask()

2012-04-23 Thread Anton Vorontsov
ock held). clear_tasks_mm_cpumask() has the issue fixed, so let's use it. Suggested-by: Oleg Nesterov Signed-off-by: Anton Vorontsov --- arch/sh/kernel/smp.c |7 +-- 1 file changed, 1 insertion(+), 6 deletions(-) diff --git a/arch/sh/kernel/smp.c b/arch/sh/kernel/smp.c index eaebdf6..4

[PATCH 5/9] blackfin: A couple of task->mm handling fixes

2012-04-23 Thread Anton Vorontsov
eads and returns an appropriate task (with task lock held). Suggested-by: Oleg Nesterov Signed-off-by: Anton Vorontsov --- arch/blackfin/kernel/trace.c | 26 +- 1 file changed, 13 insertions(+), 13 deletions(-) diff --git a/arch/blackfin/kernel/trace.c b/arch/blackfin/k

[PATCH 6/9] blackfin: Fix possible deadlock in decode_address()

2012-04-23 Thread Anton Vorontsov
ite_ variant of the tasklist lock, read_ one is enough. Suggested-by: Oleg Nesterov Signed-off-by: Anton Vorontsov --- arch/blackfin/kernel/trace.c |6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/arch/blackfin/kernel/trace.c b/arch/blackfin/kernel/trace.c index d08f

[PATCH 7/9] um: Should hold tasklist_lock while traversing processes

2012-04-23 Thread Anton Vorontsov
Traversing the tasks requires holding tasklist_lock, otherwise it is unsafe. p.s. However, I'm not sure that calling os_kill_ptraced_process() in the atomic context is correct. It seem to work, but please take a closer look. Signed-off-by: Anton Vorontsov --- arch/um/kernel/reboot.c |

[PATCH 8/9] um: Fix possible race on task->mm

2012-04-23 Thread Anton Vorontsov
should also use find_lock_task_mm() to check all process' threads for a valid mm, but for uml we'll do it in a separate patch. Signed-off-by: Anton Vorontsov --- arch/um/kernel/reboot.c |7 +-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/arch/um/kernel/reboot.c b/arc

[PATCH 9/9] um: Properly check all process' threads for a live mm

2012-04-23 Thread Anton Vorontsov
turns an appropriate task (with task lock held). Suggested-by: Oleg Nesterov Signed-off-by: Anton Vorontsov --- arch/um/kernel/reboot.c | 13 +++-- 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/arch/um/kernel/reboot.c b/arch/um/kernel/reboot.c index 1411f4e..3d15243 1006

Re: [PATCH 7/9] um: Should hold tasklist_lock while traversing processes

2012-04-23 Thread Anton Vorontsov
On Mon, Apr 23, 2012 at 04:57:54PM +0200, Richard Weinberger wrote: > On 23.04.2012 09:09, Anton Vorontsov wrote: > >Traversing the tasks requires holding tasklist_lock, otherwise it > >is unsafe. > > > >p.s. However, I'm not sure that calling os_kill_ptraced_proce

Re: [PATCH 1/9] cpu: Introduce clear_tasks_mm_cpumask() helper

2012-05-04 Thread Anton Vorontsov
cpumask_clear_cpu(cpu, mm_cpumask(t->mm)); > > + task_unlock(t); > > + } > > + rcu_read_unlock(); > > +} > > It is good that this code exists under CONFIG_HOTPLUG_CPU. Did you > check that everything works correctly with CONFIG_HOTPLUG_CPU=n? Yeah,

[PATCH] cpu: Document clear_tasks_mm_cpumask()

2012-05-04 Thread Anton Vorontsov
This patch adds more comments on clear_tasks_mm_cpumask, plus adds a runtime check: the function is only suitable for offlined CPUs, and if called inappropriately, the kernel should scream aloud. Suggested-by: Andrew Morton Suggested-by: Peter Zijlstra Signed-off-by: Anton Vorontsov --- On

Re: [PATCH] spi_mpc8xxx: issue with using definition of pram in Device Tree

2010-09-24 Thread Anton Vorontsov
o muram data regions 0x0..0x2000 and 0x9000..0x9100. Note that we actually don't want "data" regions, and the only reason why that worked is that sysdev/cpm_common.c maps muram(0)..muram(max). Thanks, -- Anton Vorontsov email: cbouatmai...@gmail.com irc://irc.freenode.net/bd2 ___

Re: [PATCH v2 1/3] powerpc/fsl_soc.c: remove FSL USB platform code

2010-09-28 Thread Anton Vorontsov
Gala > Cc: Grant Likely > --- This is not bisectable. You have to merge 1/3 and 2/3. Thanks, -- Anton Vorontsov email: cbouatmai...@gmail.com irc://irc.freenode.net/bd2 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozla

Re: [PATCH v3 6/7] mtd: m25p80: add a read function to read page by page

2010-09-30 Thread Anton Vorontsov
ers don't work, then > > it is the SPI master driver that is buggy. > > By the way, does this fix your problem? > > https://patchwork.kernel.org/patch/184752/ It shouldn't. AFAIK, eSPI is PIO-only controller, and the overrun fix is for the DMA mode. Thanks, p.s.

Re: [PATCH v3 3/7] eSPI: add eSPI controller support

2010-10-01 Thread Anton Vorontsov
atic const struct of_device_id of_fsl_espi_match[] = { > + { .compatible = "fsl,mpc8536-espi" }, > + {} > +}; > +MODULE_DEVICE_TABLE(of, of_fsl_espi_match); > + > +static struct of_platform_driver fsl_espi_driver = { > + .driver = { > + .name = "fsl_espi", > + .owner = THIS_MODULE, > + .of_match_table = of_fsl_espi_match, > + }, > + .probe = of_fsl_espi_probe, > + .remove = __devexit_p(of_fsl_espi_remove), > +}; > + > +static int __init fsl_espi_init(void) > +{ > + return of_register_platform_driver(&fsl_espi_driver); > +} > +module_init(fsl_espi_init); > + > +static void __exit fsl_espi_exit(void) > +{ > + of_unregister_platform_driver(&fsl_espi_driver); > +} > +module_exit(fsl_espi_exit); > + > +MODULE_AUTHOR("Mingkai Hu"); > +MODULE_DESCRIPTION("Enhanced Freescale SPI Driver"); This sounds like that this is an enhanced version of the Freescale SPI driver, which it is not. ;-) > +MODULE_LICENSE("GPL"); > diff --git a/drivers/spi/spi_fsl_lib.h b/drivers/spi/spi_fsl_lib.h > index 6ae8949..9c81498 100644 > --- a/drivers/spi/spi_fsl_lib.h > +++ b/drivers/spi/spi_fsl_lib.h > @@ -26,6 +26,7 @@ struct mpc8xxx_spi { > /* rx & tx bufs from the spi_transfer */ > const void *tx; > void *rx; > + int len; I'd place the #ifdef CONFIG_SPI_ESPI, for documentation purposes. Thanks, -- Anton Vorontsov email: cbouatmai...@gmail.com irc://irc.freenode.net/bd2 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH v3 2/7] spi/mpc8xxx: refactor the common code for SPI/eSPI controller

2010-10-01 Thread Anton Vorontsov
spi_mpc8xxx_cs *cs = spi->controller_state; > - __be32 __iomem *mode = &mspi->base->mode; > + struct fsl_spi_reg *reg_base = (struct fsl_spi_reg *)mspi->reg_base; No need for these type casts (the same is for the whole patch). -- Anton Vorontsov email: cbouatmai

Re: [PATCH] edac: mpc85xx: Add support for new MPCxxx/Pxxxx EDAC controllers (fix)

2010-10-06 Thread Anton Vorontsov
to clean up after ourselves. It's already in Linus' tree. Thanks, - - - - commit cd1542c8197fc3c2eb3a8301505d5d9738fab1e4 Author: Anton Vorontsov Date: Tue Aug 10 18:03:21 2010 -0700 edac: mpc85xx: add support for new MPCxxx/P EDAC controllers Simply add proper ID

Re: [PATCH] edac: mpc85xx: Add support for new MPCxxx/Pxxxx EDAC controllers (fix)

2010-10-07 Thread Anton Vorontsov
y L2 cache controller is not applicable (and based on Scott's comment I removed the l2 cache compatible entry for p4080). But I guess memory-controller is somewhat similar to all other 85xx? If it's not, I can surely prepare a patch that removes p4080 entry. Thanks, -- Anton Voronts

Re: [PATCH 1/2] ucc_geth: Do not bring the whole IF down when TX failure.

2010-11-12 Thread Anton Vorontsov
will deadlock. This patch brings ucc_geth in line with > gianfar: > > Don't bring the interface down and up, just reinit controller HW > and PHY. > > Signed-off-by: Joakim Tjernlund Looks sane, thanks! Reviewed-by: Anton Vorontsov

Re: [PATCH 2/2] ucc_geth: Fix deadlock

2010-11-12 Thread Anton Vorontsov
-by: Joakim Tjernlund It's unclear where exactly adjust_link() hangs, but the patch looks as the right thing overall. Thanks! Reviewed-by: Anton Vorontsov > --- > drivers/net/ucc_geth.c | 10 +++--- > 1 files changed, 7 insertions(+), 3 deletions(-) > > diff --git a

Re: [PATCH 2/2] powerpc: pcm030/032: add pagesize to dts

2010-11-15 Thread Anton Vorontsov
compatible = "catalyst,24c32"; > reg = <0x52>; > + pagesize = <32>; I think you'd better drop the pagesize property altogether, and instead make the compatible string more specific (if needed at all. are there any '

Re: [PATCH 2/2] powerpc: pcm030/032: add pagesize to dts

2010-11-15 Thread Anton Vorontsov
ID (no vendor ID matching). So, the current driver may just implement quirks like this: if (of_device_is_compatible(np, "catalyst,24c32")) pagesize = 32; Or, if it's some generic pattern, something like if (of_device_is_compatible_vendor(np, "catalyst")) pagesize /=

Re: [PATCH 2/2] powerpc: pcm030/032: add pagesize to dts

2010-11-16 Thread Anton Vorontsov
eems that there are enough people in "pagesize" camp anyway, I'd say just go ahead with the current approach. :-) It's an additional information, so won't do any harm anyway... Thanks, -- Anton Vorontsov email: cbouatmai...@gmail.com irc://irc.freenode.net/bd2

Re: [PATCH] of_mmc_spi: add card detect irq support

2010-12-28 Thread Anton Vorontsov
hat happened with this patch? Thanks, -- Anton Vorontsov Email: cbouatmai...@gmail.com ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

Re: [PATCH] gianfar: Fall back to software tcp/udp checksum on older controllers

2011-01-27 Thread Anton Vorontsov
as specified by errata */ > + if (((unsigned long)fcb % 0x20) > 0x18) { > + __skb_pull(skb, GMAC_FCB_LEN); > + skb_checksum_help(skb); > + break; > + } > +

Re: [PATCH v2] qe_lib: Set gpio data before changing the direction to output

2009-08-18 Thread Anton Vorontsov
On Tue, Aug 18, 2009 at 05:20:44PM -0400, Michael Barkowski wrote: > This avoids having a short glitch if the desired initial value is not > the same as what was previously in the data register. > > Signed-off-by: Michael Barkowski Acked-by: Anton Vorontsov Thanks! > --- &g

Re: [PATCH v2 0/6] Device table matching for SPI subsystem

2009-08-18 Thread Anton Vorontsov
On Mon, Aug 10, 2009 at 10:35:23AM +0300, Artem Bityutskiy wrote: > On Fri, 2009-07-31 at 04:39 +0400, Anton Vorontsov wrote: > > Andrew, > > > > This new patch set overwrites following patches: > > > > hwmon-lm70-convert-to-device-table-matching.patch > &g

Re: [PATCH 2/6] mtd: m25p80: Convert to device table matching

2009-08-18 Thread Anton Vorontsov
Hi David, Thanks for the review, and sorry for the delayed response. On Mon, Aug 03, 2009 at 07:54:50PM -0700, David Brownell wrote: > On Thursday 30 July 2009, Anton Vorontsov wrote: > > This patch converts the m25p80 driver so that now it uses .id_table > > for device matc

[PATCH 1/2] mtd: m25p80: Rework probing/JEDEC code

2009-08-18 Thread Anton Vorontsov
ct some parts of the chip). Signed-off-by: Anton Vorontsov --- drivers/mtd/devices/m25p80.c | 69 - 1 files changed, 40 insertions(+), 29 deletions(-) diff --git a/drivers/mtd/devices/m25p80.c b/drivers/mtd/devices/m25p80.c index 0d74b38..b75e319 1006

[PATCH 2/2] mtd: m25p80: Add support for CAT25xxx serial EEPROMs

2009-08-18 Thread Anton Vorontsov
to the mtd subsystem. Signed-off-by: Anton Vorontsov --- drivers/mtd/devices/m25p80.c | 172 -- 1 files changed, 97 insertions(+), 75 deletions(-) diff --git a/drivers/mtd/devices/m25p80.c b/drivers/mtd/devices/m25p80.c index b75e319..8930266 100644 ---

[PATCH v2 0/8] spi_mpc8xxx: Add support for DMA transfers

2009-08-18 Thread Anton Vorontsov
bs for non-QE/CPM builds", so we could merge that patch via powerpc tree? Thanks! -- Anton Vorontsov email: cbouatmai...@gmail.com irc://irc.freenode.net/bd2 ___ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev

[PATCH 1/8] powerpc/cpm: Remove SPI defines and spi structs

2009-08-18 Thread Anton Vorontsov
When cpm2.h included into spi_mpc8xxx driver, the SPI defines in the header conflict with defines in the driver. We don't need them in the header file, so remove them. Plus remove "struct spi", we'll use a better version in the driver. Signed-off-by: Anton Vorontsov ---

[PATCH 2/8] powerpc/qe&cpm2: Avoid redefinitions in CPM2 and QE headers

2009-08-18 Thread Anton Vorontsov
where we'll place common USB structs that are used by QE/CPM UDC and QE Host drivers (FHCI). And as for the structs in qe.h and cpm2.h, just prefix them with qe_ and cpm_. Signed-off-by: Anton Vorontsov --- arch/powerpc/include/asm/cpm.h| 22 ++ arch/powerpc/i

[PATCH 3/8] powerpc/cpm: Move CPMFCR_* defines into cpm.h

2009-08-18 Thread Anton Vorontsov
The bits are generic to CPM devices, so let's move them to the common header file, so drivers won't need to privately reintroduce another bunch of the same bits (as we can't include cpm2.h header together with cpm1.h). Signed-off-by: Anton Vorontsov --- arch/powerpc/include/as

[PATCH 4/8] powerpc/qe&cpm: Implement static inline stubs for non-QE/CPM builds

2009-08-18 Thread Anton Vorontsov
This is needed to avoid ugly #ifdefs in drivers. Also update fsl_qe_udc driver so that now it doesn't define its own versions that cause build breakage when the generic stubs are used. Signed-off-by: Anton Vorontsov --- arch/powerpc/include/asm/cpm.h |

[PATCH 5/8] spi_mpc8xxx: Fix uninitialized variable

2009-08-18 Thread Anton Vorontsov
This patch fixes the following warning: CC drivers/spi/spi_mpc8xxx.o spi_mpc8xxx.c: In function 'of_mpc8xxx_spi_probe': spi_mpc8xxx.c:681: warning: 'ret' may be used uninitialized in this function Signed-off-by: Anton Vorontsov --- drivers/spi/spi_mpc8xxx.c |

[PATCH 6/8] spi_mpc8xxx: Factor out SPI mode change steps into a call

2009-08-18 Thread Anton Vorontsov
We'll add more steps soon, so get rid of the duplication. Signed-off-by: Anton Vorontsov --- drivers/spi/spi_mpc8xxx.c | 56 +++- 1 files changed, 24 insertions(+), 32 deletions(-) diff --git a/drivers/spi/spi_mpc8xxx.c b/drivers/spi/spi_mpc8

[PATCH 7/8] spi_mpc8xxx: Turn qe_mode into flags

2009-08-18 Thread Anton Vorontsov
Soon there will be more flags introduced in subsequent patches, so let's turn qe_mode into flags. Also introduce mpc8xxx_spi_strmode() and print current SPI mode. Signed-off-by: Anton Vorontsov --- drivers/spi/spi_mpc8xxx.c | 30 +++--- include/linux/fsl_devi

[PATCH 8/8] spi_mpc8xxx: Add support for QE DMA mode and CPM1/CPM2 chips

2009-08-18 Thread Anton Vorontsov
+ * CPM SPI and QE buffer descriptors mode support: + * Copyright (c) 2009 MontaVista Software, Inc. + * Author: Anton Vorontsov + * * This program is free software; you can redistribute it and/or modify it * under the terms of the GNU General Public License as published by the

[PATCH] USB: fsl_qe_udc: Add fsl,mpc8323-qe-usb compatible entry

2009-08-18 Thread Anton Vorontsov
Current bindings specify that "fsl,mpc8323-qe-usb" compatible entry should be used as a base match for QE UDCs, so update the driver to comply with the bindings. Signed-off-by: Anton Vorontsov --- On Thu, Apr 02, 2009 at 02:17:11PM -0500, Scott Wood wrote: > Anton Vorontsov wro

Re: [PATCH v2] qe_lib: Set gpio data before changing the direction to output

2009-08-18 Thread Anton Vorontsov
On Tue, Aug 18, 2009 at 05:33:00PM -0500, Timur Tabi wrote: > Anton Vorontsov wrote: > > On Tue, Aug 18, 2009 at 05:20:44PM -0400, Michael Barkowski wrote: > >> This avoids having a short glitch if the desired initial value is not > >> the same as what was prev

[PATCH 1/3] powerpc/82xx: Fix BCSR bits for MPC8272ADS boards

2009-08-18 Thread Anton Vorontsov
2ads.h. Signed-off-by: Anton Vorontsov --- arch/powerpc/platforms/82xx/mpc8272_ads.c |8 +++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/platforms/82xx/mpc8272_ads.c b/arch/powerpc/platforms/82xx/mpc8272_ads.c index 8054c68..67e2184 100644 --- a/arch/po

[PATCH 2/3] powerpc/82xx: Add CPM USB Gadget support for MPC8272ADS boards

2009-08-18 Thread Anton Vorontsov
- Add usb node; - Configure pins and clocks; - Enable USB function in BCSR. The support was successfully tested using serial and ethernet gadget drivers. Signed-off-by: Anton Vorontsov --- arch/powerpc/boot/dts/mpc8272ads.dts |8 arch/powerpc/platforms/82xx/mpc8272_ads.c

[PATCH 3/3] powerpc/85xx: Add QE USB support for MPC8569E-MDS boards

2009-08-18 Thread Anton Vorontsov
s and schematics are correct, though we don't have the pilot boards to actually test it. Signed-off-by: Anton Vorontsov --- arch/powerpc/boot/dts/mpc8569mds.dts | 45 + arch/powerpc/platforms/85xx/mpc85xx_mds.c |4 ++ 2 files changed, 49 insert

[PATCH v2] powerpc/85xx: Add eSDHC support for MPC8536DS boards

2009-08-18 Thread Anton Vorontsov
ce it will fixup clock-frequency automatically. Signed-off-by: Anton Vorontsov --- On Tue, Aug 11, 2009 at 08:48:32AM -0500, Kumar Gala wrote: > On Aug 7, 2009, at 2:58 PM, Anton Vorontsov wrote: > >This patch simply adds sdhci node to the device tree. > > > >We specify clock-fr

Re: [PATCH 4/8] powerpc/qe&cpm: Implement static inline stubs for non-QE/CPM builds

2009-08-18 Thread Anton Vorontsov
On Tue, Aug 18, 2009 at 03:27:47PM -0700, Greg KH wrote: > On Wed, Aug 19, 2009 at 02:04:18AM +0400, Anton Vorontsov wrote: > > This is needed to avoid ugly #ifdefs in drivers. Also update fsl_qe_udc > > driver so that now it doesn't define its own versions that cause build &

Re: [PATCH v2] powerpc/85xx: Add eSDHC support for MPC8536DS boards

2009-08-18 Thread Anton Vorontsov
On Tue, Aug 18, 2009 at 08:24:17PM -0500, Kumar Gala wrote: > > On Aug 18, 2009, at 6:38 PM, Anton Vorontsov wrote: > > >This patch simply adds sdhci node to the device tree. > > > >We specify clock-frequency manually, so that eSDHC will work without > >upgra

Re: [PATCH v2] qe_lib: Set gpio data before changing the direction to output

2009-08-19 Thread Anton Vorontsov
On Wed, Aug 19, 2009 at 09:30:20AM -0400, Michael Barkowski wrote: > Anton Vorontsov wrote: > > On Tue, Aug 18, 2009 at 05:33:00PM -0500, Timur Tabi wrote: > >> Anton Vorontsov wrote: > >>> On Tue, Aug 18, 2009 at 05:20:44PM -0400, Michael Barkowski wrote: > >&

[PATCH 1/5] powerpc/qe: Increase MAX_QE_RISC to 4

2009-08-27 Thread Anton Vorontsov
MPC8569 CPUs have four QE RISCs, so we need to increase MAX_QE_RISC constant, otherwise qe_upload_firmware() fails at sanity checking. Signed-off-by: Anton Vorontsov --- arch/powerpc/sysdev/qe_lib/qe.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/sysdev

[PATCH 3/5] powerpc/qe: Add qe_upload_firmware() stub for non-QE builds

2009-08-27 Thread Anton Vorontsov
This is needed to avoid #ifdefs in MPC85xx suspend/resume code. Signed-off-by: Anton Vorontsov --- arch/powerpc/include/asm/qe.h |7 +++ 1 files changed, 7 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/include/asm/qe.h b/arch/powerpc/include/asm/qe.h index 791c67a..e8232bb

[PATCH 2/5] powerpc/qe: Make qe_reset() code path safe for repeated invocation

2009-08-27 Thread Anton Vorontsov
with previously allocated muram offset; Signed-off-by: Anton Vorontsov --- arch/powerpc/sysdev/cpm_common.c |3 +++ arch/powerpc/sysdev/qe_lib/qe.c | 10 ++ 2 files changed, 9 insertions(+), 4 deletions(-) diff --git a/arch/powerpc/sysdev/cpm_common.c b/arch/powerpc/sysdev/cpm_common.

[PATCH 4/5] powerpc/85xx: Add suspend/resume support

2009-08-27 Thread Anton Vorontsov
QE. So far we don't support Deep Sleep mode as found in newer MPC85xx CPUs (i.e. MPC8536). It can be relatively easy implemented though, and for it we reserve 'mem' suspend type. Signed-off-by: Anton Vorontsov --- arch/powerpc/Kconfig |2 +- arch/powerpc

[PATCH 5/5] powerpc/85xx: Add power management support for MPC85xxMDS boards

2009-08-27 Thread Anton Vorontsov
- Add power management controller nodes; - Add interrupts for RTC nodes, the RTC interrupt may be used as a wakeup source; Signed-off-by: Anton Vorontsov --- arch/powerpc/boot/dts/mpc8568mds.dts | 15 +-- arch/powerpc/boot/dts/mpc8569mds.dts | 13 - arch

[PATCH 1/5] ucc_geth: Fix NULL pointer dereference in uec_get_ethtool_stats()

2009-08-27 Thread Anton Vorontsov
s+0xf8/0x23c Call Trace: [ef87bcd0] [c025597c] ethtool_get_stats+0xd4/0x23c (unreliable) [ef87bd00] [c025706c] dev_ethtool+0xfe8/0x11bc [ef87be00] [c0252b5c] dev_ioctl+0x454/0x6a8 ... ---[ end trace 77fff1162a9586b0 ]--- Segmentation fault This patch fixes the issue. Signe

[PATCH 2/5] powerpc/qe: Implement qe_alive_during_sleep() helper function

2009-08-27 Thread Anton Vorontsov
, which is a sign that QE turns off during sleep. Signed-off-by: Anton Vorontsov --- arch/powerpc/include/asm/qe.h |1 + arch/powerpc/sysdev/qe_lib/qe.c | 13 + 2 files changed, 14 insertions(+), 0 deletions(-) diff --git a/arch/powerpc/include/asm/qe.h b/arch/powerpc/include/asm

[PATCH 3/5] ucc_geth: Factor out MAC initialization steps into a call

2009-08-27 Thread Anton Vorontsov
This patch factors out MAC initialization into ucc_geth_init_mac() function that we'll use for suspend/resume. Signed-off-by: Anton Vorontsov --- drivers/net/ucc_geth.c | 87 --- 1 files changed, 52 insertions(+), 35 deletions(-) diff --

[PATCH 5/5] ucc_geth: Implement suspend/resume and Wake-On-LAN support

2009-08-27 Thread Anton Vorontsov
reinitialize UCC structures. Signed-off-by: Anton Vorontsov --- drivers/net/ucc_geth.c | 85 drivers/net/ucc_geth.h |1 + drivers/net/ucc_geth_ethtool.c | 40 +++ 3 files changed, 126 insertions(+), 0 deletions(-) diff

[PATCH 4/5] ucc_geth: Remove UGETH_MAGIC_PACKET Kconfig symbol and code

2009-08-27 Thread Anton Vorontsov
This patch removes currently unused UGETH_MAGIC_PACKET Kconfig symbol and code, i.e. magic_packet_detection_{enable,disable} functions. The two functions each contain just two steps that we'll place into suspend/resume code path under CONFIG_PM. Signed-off-by: Anton Vorontsov --- driver

[PATCH] rtc: Set wakeup capability for I2C and SPI RTC drivers

2009-08-27 Thread Anton Vorontsov
So there is no guarantee that wakeup will actually work, and so I think there is no point in being pedantic wrt checking IRQ wakeup capability. Signed-off-by: Anton Vorontsov --- drivers/rtc/rtc-ds1305.c |2 ++ drivers/rtc/rtc-ds1307.c |2 ++ drivers/rtc/rtc-ds1374.c |2 ++ 3 files chan

Re: [PATCH] rtc: Set wakeup capability for I2C and SPI RTC drivers

2009-08-27 Thread Anton Vorontsov
up will actually work, > > Yes there is ... it's only **exceptional** cases where it can't > work. Your patch would make it routine that those flags be > unreliable; pretty nasty. It's not exceptional at all, you really can't tell if device's wakeup event will

Re: [PATCH] rtc: Set wakeup capability for I2C and SPI RTC drivers

2009-08-27 Thread Anton Vorontsov
On Fri, Aug 28, 2009 at 03:19:25AM +0400, Anton Vorontsov wrote: [...] > > That is why platform code should device_init_wakeup() and > > drivers should check device_can_wakeup(dev) ... > > They should (and do) check may_wakeup() (i.e. should_wakeup) before > suspe

Re: [PATCH 2/5] powerpc/qe: Make qe_reset() code path safe for repeated invocation

2009-08-28 Thread Anton Vorontsov
hink its set. Technically, no. If qe_sdma_init() fails, kernel will panic: void __init qe_reset(void) { ... if (qe_sdma_init()) panic("sdma init failed!"); } But I see your point, it isn't obvious and may lead to a bug if we'll decide

Re: [PATCH 4/5] powerpc/85xx: Add suspend/resume support

2009-08-28 Thread Anton Vorontsov
On Fri, Aug 28, 2009 at 12:38:51AM -0500, Kumar Gala wrote: > > On Aug 27, 2009, at 12:30 PM, Anton Vorontsov wrote: > > >This patch adds suspend/resume support for MPC8540-compatible and > >MPC8569 CPUs. > > > >MPC8540-compatible PMCs are trivial: we just wri

Re: [PATCH v2] powerpc/85xx: Add eSDHC support for MPC8536DS boards

2009-08-28 Thread Anton Vorontsov
On Fri, Aug 28, 2009 at 07:02:51PM +0800, Hu Mingkai-B21284 wrote: > > On Tue, Aug 18, 2009 at 08:24:17PM -0500, Kumar Gala wrote: > > > > > > On Aug 18, 2009, at 6:38 PM, Anton Vorontsov wrote: > > > > > > >This patch simply adds sdhci node to the

[PATCH v2 0/5] Suspend/resume support for some 83xx/85xx/86xx boards

2009-08-30 Thread Anton Vorontsov
sl_pmc.c; - New patch that adds suspend/resume for MPC8610HPCD; - New patch that adds suspend/resume for 83xx QE boards; - Some fixes in "Make qe_reset() code path safe for repeated invocation" patch. Thanks, -- Anton Vorontsov email:

[PATCH 1/5] powerpc/qe: Make qe_reset() code path safe for repeated invocation

2009-08-30 Thread Anton Vorontsov
with previously allocated muram offset; - Remove __init attributes from qe_reset() and cpm_muram_init(); Signed-off-by: Anton Vorontsov --- arch/powerpc/include/asm/qe.h|2 +- arch/powerpc/sysdev/cpm_common.c |5 - arch/powerpc/sysdev/qe_lib/qe.c | 12 +++- 3 files changed

  1   2   3   4   5   6   7   8   9   10   >