mbox

[GIT,PULL,1/2] Broadcom Device Tree changes for 4.4 (part 2)

Message ID 1445280190-8812-2-git-send-email-f.fainelli@gmail.com
State New
Headers show

Pull-request

http://github.com/Broadcom/stblinux tags/arm-soc/for-4.4/devicetree-part2

Message

Florian Fainelli Oct. 19, 2015, 6:43 p.m. UTC
The following changes since commit eef228e3191de0b62ef5913be216f25ba6b23a4c:

  ARM: dts: move aliases back to .dts in Cygnus (2015-09-24 18:30:10 -0700)

are available in the git repository at:

  http://github.com/Broadcom/stblinux tags/arm-soc/for-4.4/devicetree-part2

for you to fetch changes up to da6b53a6f7abc3d84260a9f0ff9849a88b44e11f:

  ARM: bcm2835: Switch to using the new clock driver support. (2015-10-19 11:32:24 -0700)

----------------------------------------------------------------
This pull request contains the DT changes for BCM2835 in 4.4.  It
pulls in clk/clk-bcm2835 (which Stephen Boyd has said would be stable)
because the DT changes to enable the clock driver need the driver
itself to be present. These changes include the following:

- Eric Anholt, moves the bcm2835 clock driver under bcm/ where it belongs with
  other Broadcom clock providers drivers, defines the binding for new clock
  driver, adds support for programming the BCM2835 audio domain, adds the DDC I2C
  controller to Device Tree, and finally migrates the Device Tree to use the new
  clock driver binding

- Lubomir Rintel adds support for the Raspberry Pi Model A+ and B revision 2, and
  remove the I2S controller which is non-existent on Raspberry Pi Model B

- Stefan Wahren adds an uart0 label for referencing the UART adapter

----------------------------------------------------------------
Eric Anholt (5):
      ARM: bcm2835: Add the DDC I2C controller to the device tree.
      clk: bcm2835: Move under bcm/ with other Broadcom SoC clk drivers.
      clk: bcm2835: Add binding docs for the new platform clock driver.
      clk: bcm2835: Add support for programming the audio domain clocks
      ARM: bcm2835: Switch to using the new clock driver support.

Lubomir Rintel (3):
      ARM: bcm2835: dt: Raspberry Pi Model B had no I2S
      ARM: bcm2835: dt: Add Raspberry Pi Model B rev2
      ARM: bcm2835: dt: Add Raspberry Pi Model A+

Stefan Wahren (1):
      ARM: bcm2835: add label for uart0

 .../bindings/clock/brcm,bcm2835-cprman.txt         |   45 +
 arch/arm/boot/dts/Makefile                         |    4 +-
 arch/arm/boot/dts/bcm2835-rpi-a-plus.dts           |   30 +
 arch/arm/boot/dts/bcm2835-rpi-b-rev2.dts           |   23 +
 arch/arm/boot/dts/bcm2835-rpi-b.dts                |    8 +-
 arch/arm/boot/dts/bcm2835-rpi.dtsi                 |    4 +
 arch/arm/boot/dts/bcm2835.dtsi                     |   64 +-
 drivers/clk/Makefile                               |    1 -
 drivers/clk/bcm/Makefile                           |    1 +
 drivers/clk/bcm/clk-bcm2835.c                      | 1575 ++++++++++++++++++++
 drivers/clk/clk-bcm2835.c                          |   55 -
 include/dt-bindings/clock/bcm2835.h                |   47 +
 12 files changed, 1768 insertions(+), 89 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/clock/brcm,bcm2835-cprman.txt
 create mode 100644 arch/arm/boot/dts/bcm2835-rpi-a-plus.dts
 create mode 100644 arch/arm/boot/dts/bcm2835-rpi-b-rev2.dts
 create mode 100644 drivers/clk/bcm/clk-bcm2835.c
 delete mode 100644 drivers/clk/clk-bcm2835.c
 create mode 100644 include/dt-bindings/clock/bcm2835.h

Comments

Olof Johansson Oct. 23, 2015, 6:26 p.m. UTC | #1
Hi Florian,

On Mon, Oct 19, 2015 at 11:43:10AM -0700, Florian Fainelli wrote:
> The following changes since commit eef228e3191de0b62ef5913be216f25ba6b23a4c:
> 
>   ARM: dts: move aliases back to .dts in Cygnus (2015-09-24 18:30:10 -0700)
> 
> are available in the git repository at:
> 
>   http://github.com/Broadcom/stblinux tags/arm-soc/for-4.4/devicetree-part2
> 
> for you to fetch changes up to da6b53a6f7abc3d84260a9f0ff9849a88b44e11f:
> 
>   ARM: bcm2835: Switch to using the new clock driver support. (2015-10-19 11:32:24 -0700)
> 
> ----------------------------------------------------------------
> This pull request contains the DT changes for BCM2835 in 4.4.  It
> pulls in clk/clk-bcm2835 (which Stephen Boyd has said would be stable)
> because the DT changes to enable the clock driver need the driver
> itself to be present. These changes include the following:
> 
> - Eric Anholt, moves the bcm2835 clock driver under bcm/ where it belongs with
>   other Broadcom clock providers drivers, defines the binding for new clock
>   driver, adds support for programming the BCM2835 audio domain, adds the DDC I2C
>   controller to Device Tree, and finally migrates the Device Tree to use the new
>   clock driver binding
> 
> - Lubomir Rintel adds support for the Raspberry Pi Model A+ and B revision 2, and
>   remove the I2S controller which is non-existent on Raspberry Pi Model B
> 
> - Stefan Wahren adds an uart0 label for referencing the UART adapter
> 
> ----------------------------------------------------------------
> Eric Anholt (5):
>       ARM: bcm2835: Add the DDC I2C controller to the device tree.
>       clk: bcm2835: Move under bcm/ with other Broadcom SoC clk drivers.

I see this patch in the branch, and it's signed off both by Stephen and you,
which makes me suspect you cherry-picked it over from their tree?

That means it'll exist in two copies and might cause pretty nasty conflicts at
merge time if other changes have gone in on top of the separate commits.
Instead, we ask that you share a branch with the same exact commits, since git
will handle that appropriately and the patch will only be in the tree once.


-Olof
Florian Fainelli Oct. 23, 2015, 7:32 p.m. UTC | #2
On 23/10/15 11:26, Olof Johansson wrote:
> Hi Florian,
> 
> On Mon, Oct 19, 2015 at 11:43:10AM -0700, Florian Fainelli wrote:
>> The following changes since commit eef228e3191de0b62ef5913be216f25ba6b23a4c:
>>
>>   ARM: dts: move aliases back to .dts in Cygnus (2015-09-24 18:30:10 -0700)
>>
>> are available in the git repository at:
>>
>>   http://github.com/Broadcom/stblinux tags/arm-soc/for-4.4/devicetree-part2
>>
>> for you to fetch changes up to da6b53a6f7abc3d84260a9f0ff9849a88b44e11f:
>>
>>   ARM: bcm2835: Switch to using the new clock driver support. (2015-10-19 11:32:24 -0700)
>>
>> ----------------------------------------------------------------
>> This pull request contains the DT changes for BCM2835 in 4.4.  It
>> pulls in clk/clk-bcm2835 (which Stephen Boyd has said would be stable)
>> because the DT changes to enable the clock driver need the driver
>> itself to be present. These changes include the following:
>>
>> - Eric Anholt, moves the bcm2835 clock driver under bcm/ where it belongs with
>>   other Broadcom clock providers drivers, defines the binding for new clock
>>   driver, adds support for programming the BCM2835 audio domain, adds the DDC I2C
>>   controller to Device Tree, and finally migrates the Device Tree to use the new
>>   clock driver binding
>>
>> - Lubomir Rintel adds support for the Raspberry Pi Model A+ and B revision 2, and
>>   remove the I2S controller which is non-existent on Raspberry Pi Model B
>>
>> - Stefan Wahren adds an uart0 label for referencing the UART adapter
>>
>> ----------------------------------------------------------------
>> Eric Anholt (5):
>>       ARM: bcm2835: Add the DDC I2C controller to the device tree.
>>       clk: bcm2835: Move under bcm/ with other Broadcom SoC clk drivers.
> 
> I see this patch in the branch, and it's signed off both by Stephen and you,
> which makes me suspect you cherry-picked it over from their tree?
> 
> That means it'll exist in two copies and might cause pretty nasty conflicts at
> merge time if other changes have gone in on top of the separate commits.
> Instead, we ask that you share a branch with the same exact commits, since git
> will handle that appropriately and the patch will only be in the tree once.

Just to clarify, are you okay with me merging a branch from Eric's tree,
or do you want to see individual commits, except for those that appear
in Stephen's tree (the clock changes basically)?
Olof Johansson Oct. 23, 2015, 7:42 p.m. UTC | #3
On Fri, Oct 23, 2015 at 12:32 PM, Florian Fainelli <f.fainelli@gmail.com> wrote:
> On 23/10/15 11:26, Olof Johansson wrote:
>> Hi Florian,
>>
>> On Mon, Oct 19, 2015 at 11:43:10AM -0700, Florian Fainelli wrote:
>>> The following changes since commit eef228e3191de0b62ef5913be216f25ba6b23a4c:
>>>
>>>   ARM: dts: move aliases back to .dts in Cygnus (2015-09-24 18:30:10 -0700)
>>>
>>> are available in the git repository at:
>>>
>>>   http://github.com/Broadcom/stblinux tags/arm-soc/for-4.4/devicetree-part2
>>>
>>> for you to fetch changes up to da6b53a6f7abc3d84260a9f0ff9849a88b44e11f:
>>>
>>>   ARM: bcm2835: Switch to using the new clock driver support. (2015-10-19 11:32:24 -0700)
>>>
>>> ----------------------------------------------------------------
>>> This pull request contains the DT changes for BCM2835 in 4.4.  It
>>> pulls in clk/clk-bcm2835 (which Stephen Boyd has said would be stable)
>>> because the DT changes to enable the clock driver need the driver
>>> itself to be present. These changes include the following:
>>>
>>> - Eric Anholt, moves the bcm2835 clock driver under bcm/ where it belongs with
>>>   other Broadcom clock providers drivers, defines the binding for new clock
>>>   driver, adds support for programming the BCM2835 audio domain, adds the DDC I2C
>>>   controller to Device Tree, and finally migrates the Device Tree to use the new
>>>   clock driver binding
>>>
>>> - Lubomir Rintel adds support for the Raspberry Pi Model A+ and B revision 2, and
>>>   remove the I2S controller which is non-existent on Raspberry Pi Model B
>>>
>>> - Stefan Wahren adds an uart0 label for referencing the UART adapter
>>>
>>> ----------------------------------------------------------------
>>> Eric Anholt (5):
>>>       ARM: bcm2835: Add the DDC I2C controller to the device tree.
>>>       clk: bcm2835: Move under bcm/ with other Broadcom SoC clk drivers.
>>
>> I see this patch in the branch, and it's signed off both by Stephen and you,
>> which makes me suspect you cherry-picked it over from their tree?
>>
>> That means it'll exist in two copies and might cause pretty nasty conflicts at
>> merge time if other changes have gone in on top of the separate commits.
>> Instead, we ask that you share a branch with the same exact commits, since git
>> will handle that appropriately and the patch will only be in the tree once.
>
> Just to clarify, are you okay with me merging a branch from Eric's tree,
> or do you want to see individual commits, except for those that appear
> in Stephen's tree (the clock changes basically)?

I'm OK with you merging from Eric. I don't know how Stephen checked in
the clock patches -- if they're just applied patches in the full
history of his tree then it'll be really awkward to bring them in. If
he applied them on a topic branch then you should base your work on
top of that branch (without rebasing it).


-Olof