Linux Samsung SOC development
 help / color / mirror / Atom feed
* Feedback on hardware support for Odroid U3
@ 2015-06-12 13:53 Thomas Pietrowski
       [not found] ` <557AFEC3.3050806@samsung.com>
  2015-06-13  6:09 ` Krzysztof Kozlowski
  0 siblings, 2 replies; 9+ messages in thread
From: Thomas Pietrowski @ 2015-06-12 13:53 UTC (permalink / raw)
  To: linux-samsung-soc

Hello again,

first after fixing my booting problems I now get different warnings after boot.
- config: http://pastebin.com/MHUu51pF
- log: http://pastebin.com/9YMgMmFL
- repo: arm-soc
I read the last days that you are still working on the clk drivers, so
as these warnings appear to be related to that you might be interested
on them.

Additionally my CPU fan it running at full RPM all the time. I checked
the temperature just to be sure and it says it's 20°C. Is this problem
known or can I help debugging it?

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Feedback on hardware support for Odroid U3
       [not found] ` <557AFEC3.3050806@samsung.com>
@ 2015-06-12 15:53   ` Thomas-Karl Pietrowski
  0 siblings, 0 replies; 9+ messages in thread
From: Thomas-Karl Pietrowski @ 2015-06-12 15:53 UTC (permalink / raw)
  To: Mateusz Krawczuk, linux-samsung-soc

Am Freitag, 12. Juni 2015, 17:46:11 schrieb Mateusz Krawczuk:
> Could you send your config and log using gist.github.com, because
> pastebin.com is forbidden by company IT.
> 
> 
> Mateusz Krawczuk
> Samsung R&D Institute Poland
> 
> W dniu 12.06.2015 o 15:53, Thomas Pietrowski pisze:
> > Hello again,
> > 
> > first after fixing my booting problems I now get different warnings after
> > boot. - config: http://pastebin.com/MHUu51pF
> > - log: http://pastebin.com/9YMgMmFL
> > - repo: arm-soc
> > I read the last days that you are still working on the clk drivers, so
> > as these warnings appear to be related to that you might be interested
> > on them.
> > 
> > Additionally my CPU fan it running at full RPM all the time. I checked
> > the temperature just to be sure and it says it's 20°C. Is this problem
> > known or can I help debugging it?

Of course I can. Didn't know about this restriction.

-config: https://gist.github.com/thopiekar/440059953391ccadc324
-log: https://gist.github.com/thopiekar/061d12802a4346eb783f

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Feedback on hardware support for Odroid U3
  2015-06-12 13:53 Feedback on hardware support for Odroid U3 Thomas Pietrowski
       [not found] ` <557AFEC3.3050806@samsung.com>
@ 2015-06-13  6:09 ` Krzysztof Kozlowski
  2015-06-13  9:11   ` Thomas-Karl Pietrowski
  1 sibling, 1 reply; 9+ messages in thread
From: Krzysztof Kozlowski @ 2015-06-13  6:09 UTC (permalink / raw)
  To: Thomas Pietrowski, linux-samsung-soc

W dniu 12.06.2015 o 22:53, Thomas Pietrowski pisze:
> Hello again,
> 
> first after fixing my booting problems I now get different warnings after boot.
> - config: http://pastebin.com/MHUu51pF
> - log: http://pastebin.com/9YMgMmFL
> - repo: arm-soc
> I read the last days that you are still working on the clk drivers, so
> as these warnings appear to be related to that you might be interested
> on them.

You mean the warnings on clocks (starting from runtime PM and FIMC)? I
hit them also on Trats2. Just need to find a person (or time) who will
fix it...

> Additionally my CPU fan it running at full RPM all the time. I checked
> the temperature just to be sure and it says it's 20°C. Is this problem
> known or can I help debugging it?

After quick look at DTS, I think that on Exynos4412 Odroid family there
is no fan defined. No fan so cooling is done only through CPU
frequencies. The fan probably works on default setting from bootloader
or default register values.

The stock Odroid U3 has only passive cooler.

Probably support for fan could be added... If you would like to take
care of it, go for it! :)

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Feedback on hardware support for Odroid U3
  2015-06-13  6:09 ` Krzysztof Kozlowski
@ 2015-06-13  9:11   ` Thomas-Karl Pietrowski
  2015-06-13  9:58     ` Krzysztof Kozlowski
  0 siblings, 1 reply; 9+ messages in thread
From: Thomas-Karl Pietrowski @ 2015-06-13  9:11 UTC (permalink / raw)
  To: Krzysztof Kozlowski; +Cc: linux-samsung-soc

Am Samstag, 13. Juni 2015, 15:09:32 schrieb Krzysztof Kozlowski:
> W dniu 12.06.2015 o 22:53, Thomas Pietrowski pisze:
> > Hello again,
> > 
> > first after fixing my booting problems I now get different warnings after
> > boot. - config: http://pastebin.com/MHUu51pF
> > - log: http://pastebin.com/9YMgMmFL
> > - repo: arm-soc
> > I read the last days that you are still working on the clk drivers, so
> > as these warnings appear to be related to that you might be interested
> > on them.
> 
> You mean the warnings on clocks (starting from runtime PM and FIMC)? I
> hit them also on Trats2. Just need to find a person (or time) who will
> fix it...
>
Ok then. I just don't remember that I had them when using 3.19.x code by 
Tobias Jakobi before. I'll test his latest code later. The reason I wanted to 
use code from arm-soc is that I thought his changes/patches are already here 
after two linux releases. However, in the past the CPU fan wasn't working at 
all (0 RPM) and /sys/class/thermal/[...]/temp reported more than 70°C (was 
compiling a newer kernel code at this moment). Don't believe it was healthy.

> > Additionally my CPU fan it running at full RPM all the time. I checked
> > the temperature just to be sure and it says it's 20°C. Is this problem
> > known or can I help debugging it?
> 
> After quick look at DTS, I think that on Exynos4412 Odroid family there
> is no fan defined. No fan so cooling is done only through CPU
> frequencies. The fan probably works on default setting from bootloader
> or default register values.
> 
Just something at this point which comes to my mind. Imagine we have the fan 
regulation/driver working. What happens when we simply disconnect the fan and 
the CPU hits higher temperatures? Will it still work on high frequencies or 
will it self-protect by using lower CPU frequencies?
CPU throttling and CPU cooling should be two seperate processes, isn't it?

> The stock Odroid U3 has only passive cooler.
> 
That's true, but they (hardkernel) also sell a optional fan and the socket for 
is is still there when buying the board.
See, 
http://www.hardkernel.com/main/products/prdt_info.php?g_code=G138760358261 for 
the optional fan.
Therefore, it is not a hardware hack or whatever.

> Probably support for fan could be added... If you would like to take
> care of it, go for it! :)
> 
I just browsed Tobias Jakobi's "linux-odroid" repository at github and found 
out, that work is already made on that by people at Samsung :)
https://github.com/tobiasjakobi/linux-odroid/commit/a2236c8f74321357e505be0a054032325a7e3a5d
Hopefully both are also in this mailing list and interested in getting this 
upstream. In the mean time I'll test it based on tobiasjakobi's codebase 
again.

Regards,
Thomas

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Feedback on hardware support for Odroid U3
  2015-06-13  9:11   ` Thomas-Karl Pietrowski
@ 2015-06-13  9:58     ` Krzysztof Kozlowski
  2015-08-05 13:07       ` Thomas Pietrowski
  0 siblings, 1 reply; 9+ messages in thread
From: Krzysztof Kozlowski @ 2015-06-13  9:58 UTC (permalink / raw)
  To: Thomas-Karl Pietrowski; +Cc: linux-samsung-soc

W dniu 13.06.2015 o 18:11, Thomas-Karl Pietrowski pisze:
> Am Samstag, 13. Juni 2015, 15:09:32 schrieb Krzysztof Kozlowski:
>> W dniu 12.06.2015 o 22:53, Thomas Pietrowski pisze:
>>> Hello again,
>>>
>>> first after fixing my booting problems I now get different warnings after
>>> boot. - config: http://pastebin.com/MHUu51pF
>>> - log: http://pastebin.com/9YMgMmFL
>>> - repo: arm-soc
>>> I read the last days that you are still working on the clk drivers, so
>>> as these warnings appear to be related to that you might be interested
>>> on them.
>>
>> You mean the warnings on clocks (starting from runtime PM and FIMC)? I
>> hit them also on Trats2. Just need to find a person (or time) who will
>> fix it...
>>
> Ok then. I just don't remember that I had them when using 3.19.x code by 
> Tobias Jakobi before. I'll test his latest code later. The reason I wanted to 
> use code from arm-soc is that I thought his changes/patches are already here 
> after two linux releases. However, in the past the CPU fan wasn't working at 
> all (0 RPM) and /sys/class/thermal/[...]/temp reported more than 70°C (was 
> compiling a newer kernel code at this moment). Don't believe it was healthy.
> 
>>> Additionally my CPU fan it running at full RPM all the time. I checked
>>> the temperature just to be sure and it says it's 20°C. Is this problem
>>> known or can I help debugging it?
>>
>> After quick look at DTS, I think that on Exynos4412 Odroid family there
>> is no fan defined. No fan so cooling is done only through CPU
>> frequencies. The fan probably works on default setting from bootloader
>> or default register values.
>>
> Just something at this point which comes to my mind. Imagine we have the fan 
> regulation/driver working. What happens when we simply disconnect the fan and 
> the CPU hits higher temperatures? Will it still work on high frequencies or 
> will it self-protect by using lower CPU frequencies?

I do not know how the thermal/cooler driver works. I suppose it depends
whether cooling device (the pwm-fan) reports non-working fan. You should
rather ask on linux-pm@vger.kernel.org mailing list. That is the place
for thermal framework.

> CPU throttling and CPU cooling should be two seperate processes, isn't it?

Although cpu frequency driver is a separate thing but for cooling it
behaves like a cooling device. A fan can be a cooling device. CPU
frequency as well.

> 
>> The stock Odroid U3 has only passive cooler.
>>
> That's true, but they (hardkernel) also sell a optional fan and the socket for 
> is is still there when buying the board.
> See, 
> http://www.hardkernel.com/main/products/prdt_info.php?g_code=G138760358261 for 
> the optional fan.
> Therefore, it is not a hardware hack or whatever.

Right.

> 
>> Probably support for fan could be added... If you would like to take
>> care of it, go for it! :)
>>
> I just browsed Tobias Jakobi's "linux-odroid" repository at github and found 
> out, that work is already made on that by people at Samsung :)
> https://github.com/tobiasjakobi/linux-odroid/commit/a2236c8f74321357e505be0a054032325a7e3a5d
> Hopefully both are also in this mailing list and interested in getting this 
> upstream. In the mean time I'll test it based on tobiasjakobi's codebase 
> again.


I wonder why this is not in mainline... Or maybe it is?

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Feedback on hardware support for Odroid U3
  2015-06-13  9:58     ` Krzysztof Kozlowski
@ 2015-08-05 13:07       ` Thomas Pietrowski
  2015-08-05 23:31         ` Krzysztof Kozłowski
  0 siblings, 1 reply; 9+ messages in thread
From: Thomas Pietrowski @ 2015-08-05 13:07 UTC (permalink / raw)
  To: Krzysztof Kozlowski; +Cc: linux-samsung-soc

Sorry for the long delay. Got now linux-stable 4.2.0-rc5 running here,
but I still miss the fan support.
Searched the web again for the patch and found this on LKML:
https://lkml.org/lkml/2014/12/18/138

Then looked at the "arm-soc" repository but found this patch nowhere.
Are you planing to add this in future releases?

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Feedback on hardware support for Odroid U3
  2015-08-05 13:07       ` Thomas Pietrowski
@ 2015-08-05 23:31         ` Krzysztof Kozłowski
  2015-10-10  8:46           ` Thomas Pietrowski
  0 siblings, 1 reply; 9+ messages in thread
From: Krzysztof Kozłowski @ 2015-08-05 23:31 UTC (permalink / raw)
  To: Thomas Pietrowski; +Cc: linux-samsung-soc

2015-08-05 22:07 GMT+09:00 Thomas Pietrowski <thopiekar@gmail.com>:
> Sorry for the long delay. Got now linux-stable 4.2.0-rc5 running here,
> but I still miss the fan support.
> Searched the web again for the patch and found this on LKML:
> https://lkml.org/lkml/2014/12/18/138
>
> Then looked at the "arm-soc" repository but found this patch nowhere.
> Are you planing to add this in future releases?

Me? No, I am not planning to work on this. Also I don't have an Odroid
U3 with a fan.

I think you should ask the author of the patchset.

Best regards,
Krzysztof

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Feedback on hardware support for Odroid U3
  2015-08-05 23:31         ` Krzysztof Kozłowski
@ 2015-10-10  8:46           ` Thomas Pietrowski
  2015-10-11 19:20             ` Thomas Pietrowski
  0 siblings, 1 reply; 9+ messages in thread
From: Thomas Pietrowski @ 2015-10-10  8:46 UTC (permalink / raw)
  To: linux-samsung-soc

Thanks! The patch for fan support is obviously on a good way to be applied.

However, I just took now 4.2.3 from linux-stable and ran into the same
problem of this conversation: https://lkml.org/lkml/2015/2/17/434

Is there a way I can help getting this debugged? I could apply this
patch and test it if you like. Just asking because someone is maybe
working on a better fix :)

Here a more detailed output of this problem: http://pastebin.com/ity4cEAS

Regards

2015-08-06 1:31 GMT+02:00 Krzysztof Kozłowski <k.kozlowski.k@gmail.com>:
> 2015-08-05 22:07 GMT+09:00 Thomas Pietrowski <thopiekar@gmail.com>:
>> Sorry for the long delay. Got now linux-stable 4.2.0-rc5 running here,
>> but I still miss the fan support.
>> Searched the web again for the patch and found this on LKML:
>> https://lkml.org/lkml/2014/12/18/138
>>
>> Then looked at the "arm-soc" repository but found this patch nowhere.
>> Are you planing to add this in future releases?
>
> Me? No, I am not planning to work on this. Also I don't have an Odroid
> U3 with a fan.
>
> I think you should ask the author of the patchset.
>
> Best regards,
> Krzysztof

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: Feedback on hardware support for Odroid U3
  2015-10-10  8:46           ` Thomas Pietrowski
@ 2015-10-11 19:20             ` Thomas Pietrowski
  0 siblings, 0 replies; 9+ messages in thread
From: Thomas Pietrowski @ 2015-10-11 19:20 UTC (permalink / raw)
  To: linux-samsung-soc

Probably also interesting for you:

~$ zcat /proc/config.gz | grep CONFIG_DRM_EXYNOS
CONFIG_DRM_EXYNOS=m
CONFIG_DRM_EXYNOS_IOMMU=y
CONFIG_DRM_EXYNOS_FIMD=y
CONFIG_DRM_EXYNOS5433_DECON=y
CONFIG_DRM_EXYNOS7_DECON=y
CONFIG_DRM_EXYNOS_DPI=y
CONFIG_DRM_EXYNOS_DSI=y
CONFIG_DRM_EXYNOS_DP=y
CONFIG_DRM_EXYNOS_HDMI=y
CONFIG_DRM_EXYNOS_VIDI=y
CONFIG_DRM_EXYNOS_G2D=y
CONFIG_DRM_EXYNOS_IPP=y
CONFIG_DRM_EXYNOS_FIMC=y
CONFIG_DRM_EXYNOS_ROTATOR=y
CONFIG_DRM_EXYNOS_MIC=y

Regards :)

2015-10-10 10:46 GMT+02:00 Thomas Pietrowski <thopiekar@gmail.com>:
> Thanks! The patch for fan support is obviously on a good way to be applied.
>
> However, I just took now 4.2.3 from linux-stable and ran into the same
> problem of this conversation: https://lkml.org/lkml/2015/2/17/434
>
> Is there a way I can help getting this debugged? I could apply this
> patch and test it if you like. Just asking because someone is maybe
> working on a better fix :)
>
> Here a more detailed output of this problem: http://pastebin.com/ity4cEAS
>
> Regards
>
> 2015-08-06 1:31 GMT+02:00 Krzysztof Kozłowski <k.kozlowski.k@gmail.com>:
>> 2015-08-05 22:07 GMT+09:00 Thomas Pietrowski <thopiekar@gmail.com>:
>>> Sorry for the long delay. Got now linux-stable 4.2.0-rc5 running here,
>>> but I still miss the fan support.
>>> Searched the web again for the patch and found this on LKML:
>>> https://lkml.org/lkml/2014/12/18/138
>>>
>>> Then looked at the "arm-soc" repository but found this patch nowhere.
>>> Are you planing to add this in future releases?
>>
>> Me? No, I am not planning to work on this. Also I don't have an Odroid
>> U3 with a fan.
>>
>> I think you should ask the author of the patchset.
>>
>> Best regards,
>> Krzysztof

^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2015-10-11 19:20 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-06-12 13:53 Feedback on hardware support for Odroid U3 Thomas Pietrowski
     [not found] ` <557AFEC3.3050806@samsung.com>
2015-06-12 15:53   ` Thomas-Karl Pietrowski
2015-06-13  6:09 ` Krzysztof Kozlowski
2015-06-13  9:11   ` Thomas-Karl Pietrowski
2015-06-13  9:58     ` Krzysztof Kozlowski
2015-08-05 13:07       ` Thomas Pietrowski
2015-08-05 23:31         ` Krzysztof Kozłowski
2015-10-10  8:46           ` Thomas Pietrowski
2015-10-11 19:20             ` Thomas Pietrowski

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox