On Wed, Apr 12, 2017 at 02:41:50PM -0500, Franklin S Cooper Jr wrote:
> 
> 
> On 04/12/2017 02:33 PM, Tom Rini wrote:
> > On Wed, Apr 12, 2017 at 02:26:21PM -0500, Franklin S Cooper Jr wrote:
> >>
> >>
> >> On 04/12/2017 02:13 PM, Tom Rini wrote:
> >>> On Tue, Apr 11, 2017 at 01:37:25PM -0500, Franklin S Cooper Jr wrote:
> >>>
> >>>> Now that the davinci I2C driver is converted to driver model enable
> >>>> it in 66AK2Gx defconfig
> >>>>
> >>>> Signed-off-by: Franklin S Cooper Jr <fcoo...@ti.com>
> >>>> ---
> >>>>  configs/k2e_evm_defconfig  | 1 +
> >>>>  configs/k2g_evm_defconfig  | 1 +
> >>>>  configs/k2hk_evm_defconfig | 1 +
> >>>>  configs/k2l_evm_defconfig  | 1 +
> >>>>  4 files changed, 4 insertions(+)
> >>>
> >>> Should we be imply/select'ing DM_I2C from the Kconfig file now?
> >>
> >> Is the goal to reduce the size of the defconfig? Or is it to essentially
> >> force all K2 boards (including possible future non TI boards) to use the
> >> DM so we can deprecate/remove non DM code in the future?
> >>
> >> If its for the latter then I don't have an issue doing so.
> > 
> > Well, both.  I think 'imply' is a great way to specify default values in
> > such a way that making new boards/defconfigs is easy and likely to be
> > correct.  But also, yes, non-DM will go away at some point.
> > 
> 
> I'm ok with that. Do you want me to resend this patchset and switch this
> patch from enabling DM_I2C via defconfig to KConfig? Or you can just
> drop this patch from the patchset and then I can just send a separate
> patch enabling DM_I2C via KConfig.
> 
> I'm ok with what ever you prefer.

Do a new 8/8 that does imply instead please, thanks!

-- 
Tom

Attachment: signature.asc
Description: Digital signature

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to