Message ID | 1421665231-28221-1-git-send-email-stefan.wahren@i2se.com |
---|---|
State | Superseded, archived |
Headers | show |
On 01/19/2015 04:00 AM, Stefan Wahren wrote: > This patch adds root compatible properties for the following boards: > - Raspberry Pi Model A > - Raspberry Pi Model A+ > - Raspberry Pi Model B > - Raspberry Pi Model B (no P5) > - Raspberry Pi Model B rev2 > - Raspberry Pi Model B+ > - Raspberry Pi Compute Module Acked-by: Stephen Warren <swarren@wwwdotorg.org> This seems sane to me. One thing I should have asked: What compatible values does the RPi Foundation's downstream for all these cases? Aligning with that would be nice if possible. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi Stephen, > Stephen Warren <swarren@wwwdotorg.org> hat am 19. Januar 2015 um 18:13 > geschrieben: > > > On 01/19/2015 04:00 AM, Stefan Wahren wrote: > > This patch adds root compatible properties for the following boards: > > - Raspberry Pi Model A > > - Raspberry Pi Model A+ > > - Raspberry Pi Model B > > - Raspberry Pi Model B (no P5) > > - Raspberry Pi Model B rev2 > > - Raspberry Pi Model B+ > > - Raspberry Pi Compute Module > > Acked-by: Stephen Warren <swarren@wwwdotorg.org> > > This seems sane to me. One thing I should have asked: What compatible > values does the RPi Foundation's downstream for all these cases? good point. Unfortunately my patches to linux-rpi-kernel are held back because of too many recipients, so it's possible that someone complain later. I have looked at chapter 3.1 in [1], but didn't find the any other variants than B and B+: Here, the presence or absence of the "-plus" is the significant thing, not the "b" -- Model A's and A+'s will use the "b" and "b-plus" variants, respectively. Also in the newest branch [2]: bcm2835-rpi-b.dts -> model = Raspberry Pi Model B bcm2708-rpi-b.dts -> model = Raspberry Pi Model B bcm2708-rpi-b-plus.dts -> model = Raspberry Pi Model B+ Sorry, i don't have any experience with the downstream kernel. So please correct me if looked at the wrong places. [1] - https://github.com/raspberrypi/documentation/blob/master/configuration/device-tree.md [2] - https://github.com/raspberrypi/linux/tree/rpi-3.18.y/ > Aligning with that would be nice if possible. Stefan -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 01/19/2015 12:01 PM, Stefan Wahren wrote: > Hi Stephen, > >> Stephen Warren <swarren@wwwdotorg.org> hat am 19. Januar 2015 um 18:13 >> geschrieben: >> >> >> On 01/19/2015 04:00 AM, Stefan Wahren wrote: >>> This patch adds root compatible properties for the following boards: >>> - Raspberry Pi Model A >>> - Raspberry Pi Model A+ >>> - Raspberry Pi Model B >>> - Raspberry Pi Model B (no P5) >>> - Raspberry Pi Model B rev2 >>> - Raspberry Pi Model B+ >>> - Raspberry Pi Compute Module >> >> Acked-by: Stephen Warren <swarren@wwwdotorg.org> >> >> This seems sane to me. One thing I should have asked: What compatible >> values does the RPi Foundation's downstream for all these cases? > > good point. Unfortunately my patches to linux-rpi-kernel are held back because > of too many recipients, so it's possible that someone complain later. > > I have looked at chapter 3.1 in [1], but didn't find the any other variants than > B and B+: > > Here, the presence or absence of the "-plus" is the significant thing, not the > "b" -- Model A's and A+'s will > use the "b" and "b-plus" variants, respectively. > > Also in the newest branch [2]: > > bcm2835-rpi-b.dts -> model = Raspberry Pi Model B > bcm2708-rpi-b.dts -> model = Raspberry Pi Model B > bcm2708-rpi-b-plus.dts -> model = Raspberry Pi Model B+ > > Sorry, i don't have any experience with the downstream kernel. So please correct > me if looked at the wrong places. > > [1] - > https://github.com/raspberrypi/documentation/blob/master/configuration/device-tree.md > [2] - https://github.com/raspberrypi/linux/tree/rpi-3.18.y/ This all matches what I found. I think differentiating between A/B and A+/B+ makes sense, so the compatible values in your patch should be fine; I'll reply to v2 in just a minute. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/Documentation/devicetree/bindings/arm/bcm2835.txt b/Documentation/devicetree/bindings/arm/bcm2835.txt index ac68348..c78576b 100644 --- a/Documentation/devicetree/bindings/arm/bcm2835.txt +++ b/Documentation/devicetree/bindings/arm/bcm2835.txt @@ -1,8 +1,35 @@ Broadcom BCM2835 device tree bindings ------------------------------------------- -Boards with the BCM2835 SoC shall have the following properties: +Raspberry Pi Model A +Required root node properties: +compatible = "raspberrypi,model-a", "brcm,bcm2835"; -Required root node property: +Raspberry Pi Model A+ +Required root node properties: +compatible = "raspberrypi,model-a-plus", "brcm,bcm2835"; +Raspberry Pi Model B +Required root node properties: +compatible = "raspberrypi,model-b", "brcm,bcm2835"; + +Raspberry Pi Model B (no P5) +early model B with I2C0 rather than I2C1 routed to the expansion header +Required root node properties: +compatible = "raspberrypi,model-b-i2c0", "brcm,bcm2835"; + +Raspberry Pi Model B rev2 +Required root node properties: +compatible = "raspberrypi,model-b-rev2", "brcm,bcm2835"; + +Raspberry Pi Model B+ +Required root node properties: +compatible = "raspberrypi,model-b-plus", "brcm,bcm2835"; + +Raspberry Pi Compute Module +Required root node properties: +compatible = "raspberrypi,compute-module", "brcm,bcm2835"; + +Generic BCM2835 board +Required root node properties: compatible = "brcm,bcm2835";
This patch adds root compatible properties for the following boards: - Raspberry Pi Model A - Raspberry Pi Model A+ - Raspberry Pi Model B - Raspberry Pi Model B (no P5) - Raspberry Pi Model B rev2 - Raspberry Pi Model B+ - Raspberry Pi Compute Module Signed-off-by: Stefan Wahren <stefan.wahren@i2se.com> --- Changes in V2: - add all currently known Raspberry Pi boards as suggested by Stephen Warren Documentation/devicetree/bindings/arm/bcm2835.txt | 31 +++++++++++++++++++-- 1 file changed, 29 insertions(+), 2 deletions(-)