On Wed, Oct 23, 2019 at 05:59:23PM +0100, Mark Brown wrote: > On Wed, Oct 23, 2019 at 09:54:17AM -0700, Nathan Chancellor wrote: > > On Wed, Oct 23, 2019 at 05:36:56PM +0100, Mark Brown wrote: > > > On Wed, Oct 23, 2019 at 09:26:28AM -0700, Nathan Chancellor wrote: > > > > On Wed, Oct 23, 2019 at 11:43:04AM +0100, Mark Brown wrote: > > > > > > The driver should also have supported s3c6400 as well. > > > > > Kconfig says otherwise, unless I am missing something. > > > > Note the XX in the config option. > > > But what about the depends and the help text? > > Viresh asked why the driver was written with s3c6410 support optional. > I explained that the reason that it was written this way was to > accomodate s3c6400 support. Ah understood, thanks for the clarification and sorry for the misunderstanding! Cheers, Nathan