Re: [PATCH 14/25] drm/exynos: introduce exynos_drm_plane_config structure

2015-11-18 Thread Tobias Jakobi
Hey Marek, Marek Szyprowski wrote: > Hello, > > On 2015-11-17 19:00, Tobias Jakobi wrote: >> Marek Szyprowski wrote: >>> This patch adds common structure for keeping plane configuration and >>> capabilities data. This patch is inspired by similar code developed by >>> Tobias Jakobi. >>> >>>

Re: [PATCH v3 3/3] ASoC: samsung: pass filter function as pointer

2015-11-18 Thread Mark Brown
On Wed, Nov 18, 2015 at 03:26:41PM +0100, Arnd Bergmann wrote: > As we are now passing the filter data as pointers to the drivers, > we can take the final step and also pass the filter function the > same way. I'm keeping this change separate, as there it's less > obvious that this is a net win.

Re: [PATCH] ARM: multi_v7_defconfig: Build ChromeOS EC drivers as modules

2015-11-18 Thread Krzysztof Kozlowski
W dniu 18.11.2015 o 00:58, Javier Martinez Canillas pisze: > Since the multi_v7_defconfig is used to build an image for different > platforms, the options should be enabled as module whenever possible. > > Signed-off-by: Javier Martinez Canillas > > --- > The patch was

Re: [PATCH 14/25] drm/exynos: introduce exynos_drm_plane_config structure

2015-11-18 Thread Marek Szyprowski
Hello, On 2015-11-17 19:00, Tobias Jakobi wrote: Marek Szyprowski wrote: This patch adds common structure for keeping plane configuration and capabilities data. This patch is inspired by similar code developed by Tobias Jakobi. Signed-off-by: Marek Szyprowski ---

Re: [PATCH 02/16 REBASED] dt-bindings: remove deprecated compatible string from exynos-hdmi

2015-11-18 Thread Krzysztof Kozlowski
On 18.11.2015 17:33, Andrzej Hajda wrote: > samsung,exynos5-hdmi compatible was marked as deprecated in Jun 2013. > It was never used since then. > > Signed-off-by: Andrzej Hajda > Reviewed-by: Gustavo Padovan > --- >

Re: [PATCHv2 3/3] ASoC: samsung: pass filter function as pointer

2015-11-18 Thread Arnd Bergmann
On Wednesday 18 November 2015 11:13:18 Krzysztof Kozlowski wrote: > > I also tested entire patchset on Exynos4412/Trats2 board (custom kernel > with audio working) for regressions and it worked fine. However, since > this was not a S3C24xx/S3C64xx board, I don't find that testing > sufficient for

Re: [PATCH v2] SPI: s3c64xx: pass DMA arguments in platform data

2015-11-18 Thread Arnd Bergmann
On Wednesday 18 November 2015 13:43:27 Krzysztof Kozlowski wrote: > On 18.11.2015 00:48, Arnd Bergmann wrote: > > struct platform_device s3c64xx_device_spi0 = { > > @@ -1135,11 +1133,13 @@ void __init s3c64xx_spi0_set_platdata(int > > (*cfg_gpio)(void), int src_clk_nr, > > pd.num_cs =

[PATCH 02/16 REBASED] dt-bindings: remove deprecated compatible string from exynos-hdmi

2015-11-18 Thread Andrzej Hajda
samsung,exynos5-hdmi compatible was marked as deprecated in Jun 2013. It was never used since then. Signed-off-by: Andrzej Hajda Reviewed-by: Gustavo Padovan --- Documentation/devicetree/bindings/display/exynos/exynos_hdmi.txt | 7 +++

[PATCH v3] SPI: s3c64xx: pass DMA arguments in platform data

2015-11-18 Thread Arnd Bergmann
The s3c64xx platform data already contains a pointer to the DMA filter function, but not to the associated data. This simplifies the code and makes it more generic by passing the data along with the filter function like we do for other drivers. Signed-off-by: Arnd Bergmann ---

[RESEND PATCH v2 5/6] ARM: dts: Remove broken-cd from eMMC node in exynos5420-peach-pi

2015-11-18 Thread Javier Martinez Canillas
The eMMC is non-removable so is marked with the non-removable DT property to avoid having to redetect it after a suspend/resume. But it also has the broken-cd property which is wrong since only one of the DT properties for card detection should be used. Also remove the card-detect-delay property

[RESEND PATCH v2 6/6] ARM: dts: Mark eMMC as non-removable in exynos5250-snow-common

2015-11-18 Thread Javier Martinez Canillas
The eMMC is non-removable so mark it using the non-removable DT property to avoid having to redetect it after a suspend/resume. Also remove the card-detect-delay property that is not needed with non-removable. Signed-off-by: Javier Martinez Canillas Reviewed-by: Tomeu

[PATCH v3 1/3] ASoC: samsung: pass DMA channels as pointers

2015-11-18 Thread Arnd Bergmann
ARM64 allmodconfig produces a bunch of warnings when building the samsung ASoC code: sound/soc/samsung/dmaengine.c: In function 'samsung_asoc_init_dma_data': sound/soc/samsung/dmaengine.c:53:32: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]

[PATCH v3 2/3] ASoC: s3c24xx-i2s: pass DMA channels as platform data

2015-11-18 Thread Arnd Bergmann
This is a minor cleanup to make the s3c2412-i2s and s3c24xx-i2s drivers independent of the mach/dma.h header file and to allow removing the dependency on the specific dmaengine driver in the next patch. As a side not, only the s3c24xx-i2s driver seems to still be used, while the definition of the

[PATCH v3 2/3] ASoC: s3c24xx-i2s: pass DMA channels as platform data

2015-11-18 Thread Arnd Bergmann
This is a minor cleanup to make the s3c2412-i2s and s3c24xx-i2s drivers independent of the mach/dma.h header file and to allow removing the dependency on the specific dmaengine driver in the next patch. As a side not, only the s3c24xx-i2s driver seems to still be used, while the definition of the

[RESEND PATCH v2 3/6] ARM: dts: Mark SDIO as non-removable in exynos5250-snow-common

2015-11-18 Thread Javier Martinez Canillas
The Exynos5250 Snow Chromebooks have a Marvell WiFi SDIO chip which can't neither be removed nor be detected. But the node isn't marked as non-removable and instead has the broken-cd DT property. This causes the device to be removed when the system enters into a suspend state and the following

[RESEND PATCH v2 0/6] ARM: dts: Use correct CD properties for SDIO and eMMC in Snow and Peach

2015-11-18 Thread Javier Martinez Canillas
Hello Krzysztof and Kukjin, This is a resend of a series posted a month ago [0] that use the correct card detection properties in the DTS for Exynos Chromebooks but that was never picked. The patches have no changed, the only difference is that I'm including all the Reviewed-by and Tested-by

[RESEND PATCH v2 1/6] ARM: dts: Mark SDIO as non-removable in exynos5800-peach-pi

2015-11-18 Thread Javier Martinez Canillas
The Exynos5800 Peach Pi Chromebook has a Marvell WiFi SDIO chip which can't neither be removed nor be detected. But the node isn't marked as non-removable and instead has the broken-cd DT property. This causes the device to be removed when the system enters into a suspend state and the following

[RESEND PATCH v2 4/6] ARM: dts: Remove broken-cd from eMMC node in exynos5800-peach-pi

2015-11-18 Thread Javier Martinez Canillas
The eMMC is non-removable so is marked with the non-removable DT property to avoid having to redetect it after a suspend/resume. But it also has the broken-cd property which is wrong since only one of the DT properties for card detection should be used. Also remove the card-detect-delay property

[RESEND PATCH v2 2/6] ARM: dts: Mark SDIO as non-removable in exynos5420-peach-pit

2015-11-18 Thread Javier Martinez Canillas
The Exynos5420 Peach Pit Chromebook has a Marvell WiFi SDIO chip which can't neither be removed nor be detected. But the node isn't marked as non-removable and instead has the broken-cd DT property. This causes the device to be removed when the system enters into a suspend state and the following

Re: [RESEND PATCH v2 6/6] ARM: dts: Mark eMMC as non-removable in exynos5250-snow-common

2015-11-18 Thread Javier Martinez Canillas
Hello, On 11/18/2015 10:23 AM, Javier Martinez Canillas wrote: > The eMMC is non-removable so mark it using the non-removable DT > property to avoid having to redetect it after a suspend/resume. > > Also remove the card-detect-delay property that is not needed with > non-removable. > >

Re: [PATCHv2 2/3] ASoC: s3c24xx-i2s: pass DMA channels as platform data

2015-11-18 Thread Arnd Bergmann
On Tuesday 17 November 2015 16:54:16 Arnd Bergmann wrote: > +#include > My randconfig test setup has now found the typo above. Resending the fixed version. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to

[PATCH v3 3/3] ASoC: samsung: pass filter function as pointer

2015-11-18 Thread Arnd Bergmann
As we are now passing the filter data as pointers to the drivers, we can take the final step and also pass the filter function the same way. I'm keeping this change separate, as there it's less obvious that this is a net win. Upsides of this are: - The ASoC drivers are completely independent

[RESEND PATCH v3] dt-bindings: Consolidate SRAM bindings from all vendors

2015-11-18 Thread Krzysztof Kozlowski
SRAM bindings for various SoCs, using the mmio-sram genalloc API, are spread over different places - per SoC vendor. Since all of these are quite similar (they depend on mmio-sram) move them to a common place. Suggested-by: Rob Herring Signed-off-by: Krzysztof Kozlowski

Re: [PATCH v3] SPI: s3c64xx: pass DMA arguments in platform data

2015-11-18 Thread Krzysztof Kozlowski
On 18.11.2015 23:21, Arnd Bergmann wrote: > The s3c64xx platform data already contains a pointer to the > DMA filter function, but not to the associated data. > > This simplifies the code and makes it more generic by > passing the data along with the filter function like > we do for other

Re: [PATCH 2/2] arm64: EXYNOS: Consolidate ARCH_EXYNOS7 symbol into ARCH_EXYNOS

2015-11-18 Thread Krzysztof Kozlowski
On 19.11.2015 13:23, Tomasz Figa wrote: > Hi Krzysztof, > > 2015-11-16 10:36 GMT+09:00 Krzysztof Kozlowski : >> The ARMv8 Exynos family SoCs in Linux kernel are currently: >> - Exynos5433 (controlled by ARCH_EXYNOS), >> - Exynos7 (controlled by ARCH_EXYNOS7). >> >> It

Re: [2/2] arm64: EXYNOS: Consolidate ARCH_EXYNOS7 symbol into ARCH_EXYNOS

2015-11-18 Thread Alim Akhtar
Hi Krzysztof, On 11/16/2015 07:06 AM, Krzysztof Kozlowski wrote: The ARMv8 Exynos family SoCs in Linux kernel are currently: - Exynos5433 (controlled by ARCH_EXYNOS), - Exynos7 (controlled by ARCH_EXYNOS7). It duplicates Kconfig symbols unnecessarily, so consolidate them into one

Re: [1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-18 Thread Alim Akhtar
Hi Krzysztof, On 11/16/2015 07:06 AM, Krzysztof Kozlowski wrote: Currently the Exynos5433 (ARMv8 SoC) clock driver depends on ARCH_EXYNOS so it is built also on ARMv7. This does not bring any kind of benefit. There won't be a single kernel image for ARMv7 and ARMv8 SoCs (like multi_v7 for

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-18 Thread Tomasz Figa
Hi Krzysztof, Good idea, just a couple of nits inline. Other than that: Acked-by: Tomasz Figa 2015-11-16 10:36 GMT+09:00 Krzysztof Kozlowski : > Currently the Exynos5433 (ARMv8 SoC) clock driver depends on ARCH_EXYNOS > so it is built also on

Re: [PATCH 2/2] arm64: EXYNOS: Consolidate ARCH_EXYNOS7 symbol into ARCH_EXYNOS

2015-11-18 Thread Tomasz Figa
Hi Krzysztof, 2015-11-16 10:36 GMT+09:00 Krzysztof Kozlowski : > The ARMv8 Exynos family SoCs in Linux kernel are currently: > - Exynos5433 (controlled by ARCH_EXYNOS), > - Exynos7 (controlled by ARCH_EXYNOS7). > > It duplicates Kconfig symbols unnecessarily, so

Re: [PATCH 1/2] clk: samsung: Don't build ARMv8 clock drivers on ARMv7

2015-11-18 Thread Krzysztof Kozlowski
On 19.11.2015 13:18, Tomasz Figa wrote: > Hi Krzysztof, > > Good idea, just a couple of nits inline. Other than that: > > Acked-by: Tomasz Figa > > 2015-11-16 10:36 GMT+09:00 Krzysztof Kozlowski : >> Currently the Exynos5433 (ARMv8 SoC) clock

[RESEND PATCH v3 3/3] ASoC: samsung: pass filter function as pointer

2015-11-18 Thread Arnd Bergmann
As we are now passing the filter data as pointers to the drivers, we can take the final step and also pass the filter function the same way. I'm keeping this change separate, as there it's less obvious that this is a net win. Upsides of this are: - The ASoC drivers are completely independent