On Wed, Feb 23, 2011 at 08:06:47PM +0000, Daniel Drake wrote: > On 23 February 2011 19:54, Andres Salomon <dilinger@xxxxxxxxxx> wrote: > >> Wait; why are you binding to a device based on name? Binding by name > >> and/or device_type is strongly discouraged for new code. Use > >> compatible instead. > >> > > > > Daniel posted a separate patch showing his code, would you mind > > commenting on that? I noticed he didn't cc you though, here's the > > patch: > > > > https://patchwork.kernel.org/patch/574901/ > > It does bind to the name. > We don't currently have a compatible property for the battery device. > We could fix that with a firmware upgrade, but it would break > compatibility with all existing installs. I guess this is still the > recommended approach... Hi Daniel, As mentioned in my reply to your patch, the solution to this is fix up the missing compatible property in the board support code before it pollutes the global matching namespace. g. -- To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
