[PATCH] spi: mediatek: fix spi incorrect endian usage and remove redundant clock

2015-08-18 Thread Leilk Liu
This patch fixes incorrect endian usage, removes redundant clock in prepare_hardware/unprepare_hardware and revises coding styles. Signed-off-by: Leilk Liu leilk@mediatek.com --- Change in this patch: 1. fix incorrect endian usage on big-endian system. 2. delete redundant clock in

Re: [PATCH] spi: mediatek: fix spi incorrect endian usage and remove redundant clock

2015-08-18 Thread Jonas Gorski
Hi, On Tue, Aug 18, 2015 at 12:53 PM, Leilk Liu leilk@mediatek.com wrote: This patch fixes incorrect endian usage, removes redundant clock in prepare_hardware/unprepare_hardware and revises coding styles. Signed-off-by: Leilk Liu leilk@mediatek.com --- Change in this patch: 1.

Re: [PATCH] spi: mediatek: fix spi incorrect endian usage and remove redundant clock

2015-08-18 Thread lei liu
On Tue, 2015-08-18 at 14:19 +0200, Jonas Gorski wrote: Hi, On Tue, Aug 18, 2015 at 12:53 PM, Leilk Liu leilk@mediatek.com wrote: This patch fixes incorrect endian usage, removes redundant clock in prepare_hardware/unprepare_hardware and revises coding styles. Signed-off-by: Leilk

Re: [PATCH] spi: mediatek: fix spi incorrect endian usage and remove redundant clock

2015-08-18 Thread Mark Brown
On Tue, Aug 18, 2015 at 02:19:54PM +0200, Jonas Gorski wrote: The usual philosophy is to have one change per patch, so this might be better as three patches. But this is Mark's call. Since the driver isn't yet in Linus' tree, it might be a-ok to mix style improvements and actual fixes, but as