2015-12-07 01:00:13

by Krzysztof Kozlowski

[permalink] [raw]
Subject: [PATCH] ARM: multi_v7_defconfig: Enable fan, sensors and audio for Odroid XU3

For Odroid XU3-family enable the:
- PWM fan (to control the CPU fan using thermal subsystem),
- TI INA231 sensors (provide power measurements of big.LITTLE cores,
DRAM and GPU),
- Samsung sound (for Odroid XU3 and Snow as well).

Signed-off-by: Krzysztof Kozlowski <[email protected]>
---
arch/arm/configs/multi_v7_defconfig | 8 ++++++--
1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
index f6a2557b55df..419f9413402c 100644
--- a/arch/arm/configs/multi_v7_defconfig
+++ b/arch/arm/configs/multi_v7_defconfig
@@ -390,7 +390,8 @@ CONFIG_POWER_RESET_RMOBILE=y
CONFIG_SENSORS_LM90=y
CONFIG_SENSORS_LM95245=y
CONFIG_SENSORS_NTC_THERMISTOR=m
-CONFIG_THERMAL=y
+CONFIG_SENSORS_PWM_FAN=m
+CONFIG_SENSORS_INA2XX=m
CONFIG_CPU_THERMAL=y
CONFIG_ROCKCHIP_THERMAL=y
CONFIG_RCAR_THERMAL=y
@@ -521,10 +522,13 @@ CONFIG_SND_HDA_INPUT_BEEP=y
CONFIG_SND_HDA_PATCH_LOADER=y
CONFIG_SND_HDA_CODEC_REALTEK=m
CONFIG_SND_HDA_CODEC_HDMI=m
-CONFIG_SND_USB_AUDIO=y
+CONFIG_SND_USB_AUDIO=m
CONFIG_SND_SOC=m
CONFIG_SND_ATMEL_SOC=m
CONFIG_SND_ATMEL_SOC_WM8904=m
+CONFIG_SND_SOC_SAMSUNG=m
+CONFIG_SND_SOC_SNOW=m
+CONFIG_SND_SOC_ODROIDX2=m
CONFIG_SND_SOC_SH4_FSI=m
CONFIG_SND_SOC_RCAR=m
CONFIG_SND_SOC_RSRC_CARD=m
--
1.9.1


2015-12-07 09:14:50

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH] ARM: multi_v7_defconfig: Enable fan, sensors and audio for Odroid XU3

On Monday 07 December 2015 09:59:54 Krzysztof Kozlowski wrote:
> For Odroid XU3-family enable the:
> - PWM fan (to control the CPU fan using thermal subsystem),
> - TI INA231 sensors (provide power measurements of big.LITTLE cores,
> DRAM and GPU),
> - Samsung sound (for Odroid XU3 and Snow as well).
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>

Looks good. Do you have a samsung/defconfig branch already that you
can put this into, or should be pick it up into arm-soc directly?

My preference is the former, but it would be a bit silly if that
is the only samsung defconfig change we need.

Arnd

2015-12-07 09:38:54

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] ARM: multi_v7_defconfig: Enable fan, sensors and audio for Odroid XU3

On 07.12.2015 18:14, Arnd Bergmann wrote:
> On Monday 07 December 2015 09:59:54 Krzysztof Kozlowski wrote:
>> For Odroid XU3-family enable the:
>> - PWM fan (to control the CPU fan using thermal subsystem),
>> - TI INA231 sensors (provide power measurements of big.LITTLE cores,
>> DRAM and GPU),
>> - Samsung sound (for Odroid XU3 and Snow as well).
>>
>> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>>
>
> Looks good. Do you have a samsung/defconfig branch already that you
> can put this into, or should be pick it up into arm-soc directly?
>
> My preference is the former, but it would be a bit silly if that
> is the only samsung defconfig change we need.

I had a couple of defconfig patches in my queue but I sent them last
week to Kukjin in pull request. He didn't pull it yet. Maybe he could
apply this patch after pulling?

Anyway we can wait for a few days to sort it out. If I don't have
another defconfig patches then I will remind myself with applying it to
arm-soc.

Best regards,
Krzysztof

2015-12-07 09:43:59

by Arnd Bergmann

[permalink] [raw]
Subject: Re: [PATCH] ARM: multi_v7_defconfig: Enable fan, sensors and audio for Odroid XU3

On Monday 07 December 2015 18:38:44 Krzysztof Kozlowski wrote:
> On 07.12.2015 18:14, Arnd Bergmann wrote:
> > On Monday 07 December 2015 09:59:54 Krzysztof Kozlowski wrote:
> >> For Odroid XU3-family enable the:
> >> - PWM fan (to control the CPU fan using thermal subsystem),
> >> - TI INA231 sensors (provide power measurements of big.LITTLE cores,
> >> DRAM and GPU),
> >> - Samsung sound (for Odroid XU3 and Snow as well).
> >>
> >> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> >>
> >
> > Looks good. Do you have a samsung/defconfig branch already that you
> > can put this into, or should be pick it up into arm-soc directly?
> >
> > My preference is the former, but it would be a bit silly if that
> > is the only samsung defconfig change we need.
>
> I had a couple of defconfig patches in my queue but I sent them last
> week to Kukjin in pull request. He didn't pull it yet. Maybe he could
> apply this patch after pulling?
>
> Anyway we can wait for a few days to sort it out. If I don't have
> another defconfig patches then I will remind myself with applying it to
> arm-soc.
>

Ok, sounds good.

Arnd

2015-12-07 12:03:15

by Javier Martinez Canillas

[permalink] [raw]
Subject: Re: [PATCH] ARM: multi_v7_defconfig: Enable fan, sensors and audio for Odroid XU3

Hello Krzysztof,

On 12/06/2015 09:59 PM, Krzysztof Kozlowski wrote:
> For Odroid XU3-family enable the:
> - PWM fan (to control the CPU fan using thermal subsystem),
> - TI INA231 sensors (provide power measurements of big.LITTLE cores,
> DRAM and GPU),
> - Samsung sound (for Odroid XU3 and Snow as well).
>
> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> ---
> arch/arm/configs/multi_v7_defconfig | 8 ++++++--
> 1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/configs/multi_v7_defconfig b/arch/arm/configs/multi_v7_defconfig
> index f6a2557b55df..419f9413402c 100644
> --- a/arch/arm/configs/multi_v7_defconfig
> +++ b/arch/arm/configs/multi_v7_defconfig

Reviewed-by: Javier Martinez Canillas <[email protected]>

Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America

2015-12-17 11:50:26

by Krzysztof Kozlowski

[permalink] [raw]
Subject: Re: [PATCH] ARM: multi_v7_defconfig: Enable fan, sensors and audio for Odroid XU3

2015-12-07 18:43 GMT+09:00 Arnd Bergmann <[email protected]>:
> On Monday 07 December 2015 18:38:44 Krzysztof Kozlowski wrote:
>> On 07.12.2015 18:14, Arnd Bergmann wrote:
>> > On Monday 07 December 2015 09:59:54 Krzysztof Kozlowski wrote:
>> >> For Odroid XU3-family enable the:
>> >> - PWM fan (to control the CPU fan using thermal subsystem),
>> >> - TI INA231 sensors (provide power measurements of big.LITTLE cores,
>> >> DRAM and GPU),
>> >> - Samsung sound (for Odroid XU3 and Snow as well).
>> >>
>> >> Signed-off-by: Krzysztof Kozlowski <[email protected]>
>> >>
>> >
>> > Looks good. Do you have a samsung/defconfig branch already that you
>> > can put this into, or should be pick it up into arm-soc directly?
>> >
>> > My preference is the former, but it would be a bit silly if that
>> > is the only samsung defconfig change we need.
>>
>> I had a couple of defconfig patches in my queue but I sent them last
>> week to Kukjin in pull request. He didn't pull it yet. Maybe he could
>> apply this patch after pulling?
>>
>> Anyway we can wait for a few days to sort it out. If I don't have
>> another defconfig patches then I will remind myself with applying it to
>> arm-soc.
>>
>
> Ok, sounds good.

+Cc [email protected],

I don't have any other defconfig changes in my queue, except of course
stuff which I sent in pull request. I mean the request sent initially
to Kukjin but now redirected with his ack to you guys.

If you don't mind, please apply this patch directly (with Javier's reviewed-by).

Best regards,
Krzysztof

2015-12-22 19:57:07

by Olof Johansson

[permalink] [raw]
Subject: Re: [PATCH] ARM: multi_v7_defconfig: Enable fan, sensors and audio for Odroid XU3

On Thu, Dec 17, 2015 at 08:50:24PM +0900, Krzysztof Kozlowski wrote:
> 2015-12-07 18:43 GMT+09:00 Arnd Bergmann <[email protected]>:
> > On Monday 07 December 2015 18:38:44 Krzysztof Kozlowski wrote:
> >> On 07.12.2015 18:14, Arnd Bergmann wrote:
> >> > On Monday 07 December 2015 09:59:54 Krzysztof Kozlowski wrote:
> >> >> For Odroid XU3-family enable the:
> >> >> - PWM fan (to control the CPU fan using thermal subsystem),
> >> >> - TI INA231 sensors (provide power measurements of big.LITTLE cores,
> >> >> DRAM and GPU),
> >> >> - Samsung sound (for Odroid XU3 and Snow as well).
> >> >>
> >> >> Signed-off-by: Krzysztof Kozlowski <[email protected]>
> >> >>
> >> >
> >> > Looks good. Do you have a samsung/defconfig branch already that you
> >> > can put this into, or should be pick it up into arm-soc directly?
> >> >
> >> > My preference is the former, but it would be a bit silly if that
> >> > is the only samsung defconfig change we need.
> >>
> >> I had a couple of defconfig patches in my queue but I sent them last
> >> week to Kukjin in pull request. He didn't pull it yet. Maybe he could
> >> apply this patch after pulling?
> >>
> >> Anyway we can wait for a few days to sort it out. If I don't have
> >> another defconfig patches then I will remind myself with applying it to
> >> arm-soc.
> >>
> >
> > Ok, sounds good.
>
> +Cc [email protected],
>
> I don't have any other defconfig changes in my queue, except of course
> stuff which I sent in pull request. I mean the request sent initially
> to Kukjin but now redirected with his ack to you guys.
>
> If you don't mind, please apply this patch directly (with Javier's reviewed-by).

Happy to apply, but can you please resend it to us (in patch format is fine)
since I just get this email from the thread in the arm@ folder...


-Olof