From mboxrd@z Thu Jan 1 00:00:00 1970 From: Lee Jones Subject: Re: [PATCH V5 2/2] mfd: stmpe: Update DT support in stmpe driver Date: Thu, 6 Dec 2012 09:50:19 +0000 Message-ID: <20121206095019.GN2718@gmail.com> References: <2dcd7cb4c4022fa24b5328974e4226f5aaf89419.1354199865.git.viresh.kumar@linaro.org> <121653def4e985b0c1b59045637dd4518f97e73a.1354199865.git.viresh.kumar@linaro.org> <20121130105731.GN3176@sortiz-mobl> <20121130154542.GG23648@gmail.com> <20121205224203.691153E0E22@localhost> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: Sender: linux-kernel-owner@vger.kernel.org To: Viresh Kumar Cc: Grant Likely , Samuel Ortiz , rabin.vincent@stericsson.com, shiraz.hashim@st.com, devicetree-discuss@lists.ozlabs.org, linux-kernel@vger.kernel.org, spear-devel@list.st.com, linus.walleij@linaro.org, Vipul Kumar Samar List-Id: devicetree@vger.kernel.org > > But regardless, it is the responsiblity of the probe function to go= and > > look if of_driver_match_device() matches against anything if it car= es > > about the of_match_table entries (for instance, if there is extra d= ata > > attached). >=20 > Ok, so filling .data field in of_device_id[] is not required for our = case as > we aren't doing anything special in our drivers. This is exactly my point, and the reason I bought it up in the first place. Normally when you specify an ID table and populate the .data attribute, you parse for it in the code and then cast it back to some kind of useful data. However, you're not doing that, which is precisely why I wondered if the table was necessary at all. In all my testing, the DT portion worked and the correct STMPE chip was identified without it. So, are you adding the table for good reason, or because you think it's the right thing to do? --=20 Lee Jones Linaro ST-Ericsson Landing Team Lead Linaro.org =E2=94=82 Open source software for ARM SoCs =46ollow Linaro: Facebook | Twitter | Blog