From: Mark Brown <broonie@kernel.org>
To: Jean-Francois Moine <moinejf@free.fr>
Cc: Russell King - ARM Linux <linux@arm.linux.org.uk>,
Rob Clark <robdclark@gmail.com>, Dave Airlie <airlied@gmail.com>,
linux-arm-kernel@lists.infradead.org,
dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH v5 00/23]
Date: Mon, 3 Feb 2014 12:46:59 +0000 [thread overview]
Message-ID: <20140203124659.GW22609@sirena.org.uk> (raw)
In-Reply-To: <20140202190606.6fa193ce@armhf>
[-- Attachment #1: Type: text/plain, Size: 527 bytes --]
On Sun, Feb 02, 2014 at 07:06:06PM +0100, Jean-Francois Moine wrote:
> - the .of_match_table is not needed because the i2c client is created by
> the i2c subsystem from the 'reg' in the DT,
It's generally better to have an explict set of OF IDs even if the
default does work - matching purely on the device name does work almost
all the time but there are collisions out there with different
manufacturers using the same prefix for their chips (the example I
always trot out is that both Wolfson and Wondermedia use "wm").
[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: broonie@kernel.org (Mark Brown)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v5 00/23]
Date: Mon, 3 Feb 2014 12:46:59 +0000 [thread overview]
Message-ID: <20140203124659.GW22609@sirena.org.uk> (raw)
In-Reply-To: <20140202190606.6fa193ce@armhf>
On Sun, Feb 02, 2014 at 07:06:06PM +0100, Jean-Francois Moine wrote:
> - the .of_match_table is not needed because the i2c client is created by
> the i2c subsystem from the 'reg' in the DT,
It's generally better to have an explict set of OF IDs even if the
default does work - matching purely on the device name does work almost
all the time but there are collisions out there with different
manufacturers using the same prefix for their chips (the example I
always trot out is that both Wolfson and Wondermedia use "wm").
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140203/201b93b6/attachment.sig>
next prev parent reply other threads:[~2014-02-03 12:47 UTC|newest]
Thread overview: 103+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-29 9:01 [PATCH v5 00/23] Jean-Francois Moine
2014-01-29 9:01 ` Jean-Francois Moine
2014-01-29 9:01 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 23/23] drm/i2c: tda998x: adjust the audio clock divider for S/PDIF Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 17/23] drm/i2c: tda998x: set the PLL division factor in range 0..3 Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-29 15:16 ` Joe Perches
2014-01-29 15:16 ` Joe Perches
2014-01-25 17:14 ` [PATCH v5 16/23] drm/i2c: tda998x: fix the ENABLE_SPACE register Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 15/23] drm/i2c: tda998x: add DT documentation Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 13/23] drm/i2c: tda998x: always enable EDID read IRQ Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 14/23] drm/i2c: tda998x: use irq for connection status and EDID read Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 22/23] drm/i2c: tda998x: code optimization Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 21/23] drm/i2c: tda998x: change the frequence in the audio channel Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 20/23] drm/i2c: tda998x: add the active aspect in HDMI AVI frame Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 18/23] drm/i2c: tda998x: make the audio code more readable Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 19/23] drm/i2c: tda998x: remove the unused variable ca_i2s Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 08/23] drm/i2c: tda998x: use HDMI constants Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 12/23] drm/i2c: tda998x: add DT support Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 09/23] drm/i2c: tda998x: don't read write-only registers Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-02-02 16:23 ` Russell King - ARM Linux
2014-02-02 16:23 ` Russell King - ARM Linux
2014-02-02 17:45 ` Jean-Francois Moine
2014-02-02 17:45 ` Jean-Francois Moine
2014-02-02 17:45 ` Jean-Francois Moine
2014-02-02 17:57 ` Russell King - ARM Linux
2014-02-02 17:57 ` Russell King - ARM Linux
2014-01-25 17:14 ` [PATCH v5 10/23] drm/i2c: tda998x: free the CEC device on encoder_destroy Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 11/23] drm/i2c: tda998x: check the CEC device creation Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 07/23] drm/i2c: tda998x: fix bad value in the AIF Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 05/23] drm/i2c: tda998x: don't freeze the system at audio startup time Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 02/23] drm/i2c: tda998x: check more I/O errors Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-02-02 16:20 ` Russell King - ARM Linux
2014-02-02 16:20 ` Russell King - ARM Linux
2014-02-02 17:30 ` Jean-Francois Moine
2014-02-02 17:30 ` Jean-Francois Moine
2014-02-02 17:30 ` Jean-Francois Moine
2014-02-02 17:56 ` Russell King - ARM Linux
2014-02-02 17:56 ` Russell King - ARM Linux
2014-01-25 17:14 ` [PATCH v5 06/23] drm/i2c: tda998x: force the page register at startup time Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 03/23] drm/i2c: tda998x: code cleanup Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 04/23] drm/i2c: tda998x: change probe message origin Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` [PATCH v5 01/23] drm/i2c: tda998x: simplify the i2c read/write functions Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-01-25 17:14 ` Jean-Francois Moine
2014-02-02 12:43 ` [PATCH v5 00/23] Russell King - ARM Linux
2014-02-02 12:43 ` Russell King - ARM Linux
2014-02-02 12:43 ` Russell King - ARM Linux
2014-02-02 18:04 ` Russell King - ARM Linux
2014-02-02 18:04 ` Russell King - ARM Linux
2014-02-02 19:00 ` Jean-Francois Moine
2014-02-02 19:00 ` Jean-Francois Moine
2014-02-02 19:00 ` Jean-Francois Moine
2014-02-02 18:06 ` Jean-Francois Moine
2014-02-02 18:06 ` Jean-Francois Moine
2014-02-02 18:23 ` Russell King - ARM Linux
2014-02-02 18:23 ` Russell King - ARM Linux
2014-02-02 18:23 ` Russell King - ARM Linux
2014-02-02 18:41 ` Sebastian Hesselbarth
2014-02-02 18:41 ` Sebastian Hesselbarth
2014-02-02 18:54 ` Jean-Francois Moine
2014-02-02 18:54 ` Jean-Francois Moine
2014-02-02 18:54 ` Jean-Francois Moine
2014-02-02 19:15 ` Russell King - ARM Linux
2014-02-02 19:15 ` Russell King - ARM Linux
2014-02-02 20:07 ` Jean-Francois Moine
2014-02-02 20:07 ` Jean-Francois Moine
2014-02-02 20:07 ` Jean-Francois Moine
2014-02-03 12:46 ` Mark Brown [this message]
2014-02-03 12:46 ` Mark Brown
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=20140203124659.GW22609@sirena.org.uk \
--to=broonie@kernel.org \
--cc=airlied@gmail.com \
--cc=dri-devel@lists.freedesktop.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@arm.linux.org.uk \
--cc=moinejf@free.fr \
--cc=robdclark@gmail.com \
/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.