All of lore.kernel.org
 help / color / mirror / Atom feed
From: alex.elder@linaro.org (Alex Elder)
To: linux-arm-kernel@lists.infradead.org
Subject: [GIT PULL] bcm driver updates for 3.14
Date: Thu, 09 Jan 2014 07:09:03 -0600	[thread overview]
Message-ID: <52CE9F6F.8000507@linaro.org> (raw)
In-Reply-To: <CAEPRUYpZTTX8bB68V5WMzBroj+arADsbbmXVSgBtn0At1SmRmw@mail.gmail.com>

On 01/09/2014 01:52 AM, Christian Daudt wrote:
> On Wed, Jan 8, 2014 at 10:04 PM, Olof Johansson <olof@lixom.net> wrote:
>> On Sun, Jan 05, 2014 at 10:58:36PM -0800, Christian Daudt wrote:
>>> The following changes since commit 319e2e3f63c348a9b66db4667efa73178e18b17d:
>>>
>>>   Linux 3.13-rc4 (2013-12-15 12:31:33 -0800)
>>>
>>> are available in the git repository at:
>>>
>>>   git://github.com/broadcom/bcm11351.git tags/bcm-for-3.14-drivers
>>
>> Hi,
>>
>> Is there a reason you're sending this through arm-soc instead of through Mike's
>> clk tree? I don't see any dependencies on SoC code so I think it can go
>> independently through clk?
>>
>>
>> -Olof
> The reason for the clk patches is that the file
> include/dt-bindings/clock/bcm281xx.h is  used both by the driver and
> the by the bcm11351.dtsi file (which has other mods going through
> arm-soc). But now that you mentioned it, I looked and didn't find the
> patch that modifies dts to include it. Alex - did I miss it?

Yes you did.  Or I guess I may have, or may not have communicated
the situation adequately.

There were four commits:
    clk: bcm281xx: define kona clock binding
    clk: bcm281xx: add initial clock framework support
    clk: bcm281xx: define U32_MAX conditionally for now
and
    ARM: dts: bcm281xx: define real clocks

The first one is in your tagged branch "bcm-for-3.14-dt".
The second two are in "bcm-for-3.14-drivers".  The last one
is missing.

HOWEVER there's more to it than that.  My DT change depended
on both code and DT changes from Tim Kryger--from another
series you were trying to coordinate taking through your tree.
    https://lkml.org/lkml/2013/12/5/508
And because those seem to be missing from your tree we can't
just use the DT patch as-is.  As I understand it we have been
waiting for one unresponsive maintainer on that series (but
I haven't been following closely).

For what it's worth, the DT patch is available here:
    git://git.linaro.org/landing-teams/working/broadcom/kernel.git
as the last commit in branch "pull/csd-kona-clocks".

But to get this resolved we have to either:
- include Tim's series in the pull request;
- update my DT file to *not* define clocks affected by
  Tim's series;
- or make a trivial update to my code so it does *not*
  disable unused clocks.

(Or maybe something else.)

I'm here to help and can respond quickly.  Christian and Tim are
on the US West coast so it'll be a few hours before they're around
to help out.

					-Alex

  reply	other threads:[~2014-01-09 13:09 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-06  6:58 [GIT PULL] bcm driver updates for 3.14 Christian Daudt
2014-01-09  6:04 ` Olof Johansson
2014-01-09  7:52   ` Christian Daudt
2014-01-09 13:09     ` Alex Elder [this message]
2014-01-09 14:50       ` Matt Porter
2014-01-23 17:04 ` Kevin Hilman

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=52CE9F6F.8000507@linaro.org \
    --to=alex.elder@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.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.