All of lore.kernel.org
 help / color / mirror / Atom feed
From: Magnus Damm <magnus.damm@gmail.com>
To: linux-sh@vger.kernel.org
Subject: Re: [PATCH] ARM: shmobile: defconfig: enable HDMI output for RCar
Date: Wed, 14 Oct 2015 08:36:56 +0000	[thread overview]
Message-ID: <CANqRtoR0HPu2gjeBqUcOV4jBODSAkZoECXV=Dt+7++sgZVawEQ@mail.gmail.com> (raw)
In-Reply-To: <1407974835-11064-1-git-send-email-khilman@linaro.org>

Hi Geert,

On Wed, Oct 14, 2015 at 5:22 PM, Geert Uytterhoeven
<geert@linux-m68k.org> wrote:
> Hi Magnus,
>
> On Wed, Oct 14, 2015 at 10:09 AM, Magnus Damm <magnus.damm@gmail.com> wrote:
>>> Or do you mean disabling them upfront?
>>> There are some clocks that must not be disabled. Some of them are not
>>> documented.
>>
>> Disabling them upfront with sparse documentation seems fun, but also a
>> bit like asking for trouble. =)
>
> I have hackish code that does it for all boards I have, and use it actively.

Seems like a good feature to me!

>> So disabling clocks upfront may be a troublesome but good way to
>> trigger incorrect dependencies, and unless I'm mistaken it should give
>> the same behaviour as the updated u-boot version, right?
>
> More or less (U-Boot still keeps a few enabled for its own use, e.g. Ethernet).

So it fails to cleanup after itself then... But by disabling clocks
upfront I guess you can be even more aggressive.

Anyway, it would be nice to see that feature upstream at some point.
Let me know if there is anything I can do to help!!

Cheers,

/ magnus

  parent reply	other threads:[~2015-10-14  8:36 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-08-14  0:07 [PATCH] ARM: shmobile: defconfig: enable initrd and atag dtb compat Kevin Hilman
2014-08-14  0:07 ` Kevin Hilman
2014-08-14  0:07 ` Kevin Hilman
2014-08-14  5:25 ` Simon Horman
2014-08-14  5:25   ` Simon Horman
2014-08-14  5:25   ` Simon Horman
2014-08-17  0:37   ` Simon Horman
2014-08-17  0:37     ` Simon Horman
2014-08-17  0:37     ` Simon Horman
2014-08-14 23:03 ` [PATCH] ARM: shmobile: defconfig: enable ARM_ATAG_DTB_COMPAT_CMDLINE_EXTEND Simon Horman
2014-08-14 23:03   ` Simon Horman
2014-08-17  0:38   ` Simon Horman
2014-08-17  0:38     ` Simon Horman
2015-10-10 19:14 ` [PATCH] ARM: shmobile: defconfig: enable HDMI output for RCar Wolfram Sang
2015-10-12  0:19 ` Simon Horman
2015-10-12  6:38 ` Wolfram Sang
2015-10-13  1:03 ` Simon Horman
2015-10-14  1:09 ` Laurent Pinchart
2015-10-14  5:52 ` Wolfram Sang
2015-10-14  7:07 ` Geert Uytterhoeven
2015-10-14  7:44 ` Magnus Damm
2015-10-14  7:54 ` Geert Uytterhoeven
2015-10-14  8:09 ` Magnus Damm
2015-10-14  8:22 ` Geert Uytterhoeven
2015-10-14  8:25 ` Wolfram Sang
2015-10-14  8:36 ` Magnus Damm [this message]
2015-10-14 11:41 ` Laurent Pinchart
2015-10-14 12:11 ` Khiem Nguyen
2015-10-14 12:55 ` Laurent Pinchart
2015-10-14 13:18 ` Khiem Nguyen
2015-10-14 13:32 ` Geert Uytterhoeven
2015-10-14 14:11 ` Khiem Nguyen
2015-11-30 13:35 ` [PATCH] ARM: shmobile: defconfig: Enable CONFIG_PRINTK_TIME Geert Uytterhoeven
2015-11-30 13:35   ` Geert Uytterhoeven
2015-12-01  6:44   ` Simon Horman
2015-12-01  6:44     ` Simon Horman
2015-12-09  9:28     ` Geert Uytterhoeven
2015-12-09  9:28       ` Geert Uytterhoeven
2015-12-10  7:04       ` Simon Horman
2015-12-10  7:04         ` Simon Horman

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to='CANqRtoR0HPu2gjeBqUcOV4jBODSAkZoECXV=Dt+7++sgZVawEQ@mail.gmail.com' \
    --to=magnus.damm@gmail.com \
    --cc=linux-sh@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.