On Wed, 2 Feb 2011, Rajendra Nayak wrote: > The _add_optional_clock_alias function expects an entry > already existing in the clkdev table in the form of > <dev-id=NULL, con-id=role> which might not be the case > always. > > Instead, just check if an entry already exists in clkdev > in the <dev-id=dev_name, con-id=role> form, else go ahead > and add one. > > Remove any assumption of an entry already existing in clkdev > table in any form. > > Since this means, adding a new entry in clkdev if it does > not already exist, and not really adding an 'alias', > also rename the function name > (s/_add_optional_clock_alias/_add_optional_clock_clkdev) > to reflect this. > > Signed-off-by: Rajendra Nayak <rnayak@xxxxxx> > Reported-by: Sumit Semwal <sumit.semwal@xxxxxx> > Cc: Sumit Semwal <sumit.semwal@xxxxxx> > Cc: Paul Walmsley <paul@xxxxxxxxx> > Cc: Benoit Cousson <b-cousson@xxxxxx> > Cc: Kevin Hilman <khilman@xxxxxx> > Cc: Partha Basak <p-basak2@xxxxxx> Thanks. I changed the subject line to read: OMAP2+: omap_device/clock: Do not expect an entry in clkdev for opt_clks before queuing for 2.6.39 in the branch 'omap_device_a_2.6.39' of git://git.pwsan.com/linux-2.6 Also added to the 'integration-2.6.39' branch of git://git.pwsan.com/linux-2.6 and is part of the stable tag 'integration-2.6.39-20110210-002' - Paul -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
