On 10/19, York Sun wrote: > > > On 10/19/2015 05:36 PM, Stephen Boyd wrote: > > On 10/20, York Sun wrote: > >> > >> You can see, when the fixed-rate clock is registered, the device id of > >> si5338 is unknown yet. (I am using one 25MHz clock to feed multiple > >> si5338, where maybe I should create multiple 25MHz clocks, but that's > >> another discussion.) > >> > >> I hope I have made it clear. > > > > Yes. It would be great if we could modify the i2c framework to > > let us create an i2c device but not call device_register() until a later > > time. So something like this could be done in the platform > > driver: > > > > for (i = 0; i < NUM_SI5338_CHIPS; i++) { > > adap = i2c_get_adapter(private->i2c_adp->nr + 1 + i); > > private->i2c_client[i] = i2c_new_device_unregistered(adap, > > &si5338_info[i], i2c_si5338_addr); > > clkdev_create(clk, NULL, dev_name(private->i2c_client[i]->dev)); > > device_register(private->i2c_client[i]->dev); > > } > > > > Then in the si5338 driver we call devm_clk_get(i2c->dev, NULL) > > and we get the xtal clock. > > > > Then what do we do before we have this i2c_new_device_unregistered()? Does my > proposal make sense?
There's nothing to do before we make this new API. Make a patch to add the new API, and then use it in the pcie driver. Your proposal makes sense, but it isn't necessary or desirable. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project -- To unsubscribe from this list: send the line "unsubscribe linux-clk" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html