Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add sound-dai-cells to I2S def #1502

Merged
merged 1 commit into from
May 31, 2016

Conversation

clivem
Copy link

@clivem clivem commented May 29, 2016

Add '#sound-dai-cells = <0>;' to the I2S defintion in bcm2708_common.dtsi

Not having it specified, whilst not causing an issue right now with
rpi-4.4.y, is going to cause an issue going forward with the use of
simple-card driver. So it doesn't fall through the cracks, patch it
in now.

Hopefully @msperl has taken care of getting a patch submitted for the
upstream Pi dts, as it was he who first run into the issue with the
current upstream kernel....
msperl#3 (comment)

@pelwell
Copy link
Contributor

pelwell commented May 29, 2016

ARNING: 'defintion' may be misspelled - perhaps 'definition'?                  |
#6:                                                                             
Add '#sound-dai-cells = <0>;' to the I2S defintion in bcm2708_common.dtsi       

total: 0 errors, 1 warnings, 7 lines checked                                    

Add '#sound-dai-cells = <0>;' to the I2S definition in
bcm2708_common.dtsi

Not having it specified, whilst not causing an issue right now with
rpi-4.4.y, is going to cause an issue going forward with the use of
simple-card driver. So it doesn't fall through the cracks, patch it
in now.

Hopefully Martin has taken care of getting a patch submitted for the
upstream Pi dts, as it was he who first run into the issue with the
current upstream kernel....
msperl#3 (comment)

Signed-off-by: DigitalDreamtime <clive.messer@digitaldreamtime.co.uk>
@pelwell pelwell merged commit 6f16b3c into raspberrypi:rpi-4.4.y May 31, 2016
popcornmix added a commit to raspberrypi/firmware that referenced this pull request May 31, 2016
See: raspberrypi/linux#1485

kernel: bcm2835-camera: Fix max/min error when looping over cameras/resolutions
See: raspberrypi/linux#1447 (comment)

kernel: Add dt param to force HiFiBerry DAC+ Pro into slave mode
See: raspberrypi/linux#1501

kernel: simple: add sound-dai-cells to I2S def
See: raspberrypi/linux#1502

kernel: BCM2835-V4L2: Increase minimum resolution to 32x32
See: raspberrypi/linux#1505

kernel: config: Add support for Logitech Rumblepad
See: raspberrypi/linux#1496

firmware: arm_loader: Add simple-framebuffer support for vc4-kms-v3d

firmware: dtoverlay: Improve compatibility
libdtovl: Simplify target-path handling
See: https://www.raspberrypi.org/forums/viewtopic.php?f=107&t=139732

firmware: bootcode: Fix LED for A+ and B+
See: https://www.raspberrypi.org/forums/viewtopic.php?f=28&t=135813

firmware: cmake: generate and install pkgconfig files
See: raspberrypi/userland#245

documentaton: IL Components: Documentation update from latest source
popcornmix added a commit to Hexxeh/rpi-firmware that referenced this pull request May 31, 2016
See: raspberrypi/linux#1485

kernel: bcm2835-camera: Fix max/min error when looping over cameras/resolutions
See: raspberrypi/linux#1447 (comment)

kernel: Add dt param to force HiFiBerry DAC+ Pro into slave mode
See: raspberrypi/linux#1501

kernel: simple: add sound-dai-cells to I2S def
See: raspberrypi/linux#1502

kernel: BCM2835-V4L2: Increase minimum resolution to 32x32
See: raspberrypi/linux#1505

kernel: config: Add support for Logitech Rumblepad
See: raspberrypi/linux#1496

firmware: arm_loader: Add simple-framebuffer support for vc4-kms-v3d

firmware: dtoverlay: Improve compatibility
libdtovl: Simplify target-path handling
See: https://www.raspberrypi.org/forums/viewtopic.php?f=107&t=139732

firmware: bootcode: Fix LED for A+ and B+
See: https://www.raspberrypi.org/forums/viewtopic.php?f=28&t=135813

firmware: cmake: generate and install pkgconfig files
See: raspberrypi/userland#245

documentaton: IL Components: Documentation update from latest source
@clivem clivem deleted the rpi-4.4.y-i2s branch June 1, 2016 04:52
neuschaefer pushed a commit to neuschaefer/raspi-binary-firmware that referenced this pull request Feb 27, 2017
See: raspberrypi/linux#1485

kernel: bcm2835-camera: Fix max/min error when looping over cameras/resolutions
See: raspberrypi/linux#1447 (comment)

kernel: Add dt param to force HiFiBerry DAC+ Pro into slave mode
See: raspberrypi/linux#1501

kernel: simple: add sound-dai-cells to I2S def
See: raspberrypi/linux#1502

kernel: BCM2835-V4L2: Increase minimum resolution to 32x32
See: raspberrypi/linux#1505

kernel: config: Add support for Logitech Rumblepad
See: raspberrypi/linux#1496

firmware: arm_loader: Add simple-framebuffer support for vc4-kms-v3d

firmware: dtoverlay: Improve compatibility
libdtovl: Simplify target-path handling
See: https://www.raspberrypi.org/forums/viewtopic.php?f=107&t=139732

firmware: bootcode: Fix LED for A+ and B+
See: https://www.raspberrypi.org/forums/viewtopic.php?f=28&t=135813

firmware: cmake: generate and install pkgconfig files
See: raspberrypi/userland#245

documentaton: IL Components: Documentation update from latest source
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants