On Thu, Oct 01, 2009 at 11:32:01PM +0530, Sekhar Nori wrote:
> This patch simplifies selection of peripherals on the UI card by
> eliminating the DA830_UI config option. Instead, the successful
> probe of GPIO expander present on the UI card is used to detect
> its presence. For this reason, GPIO_PCF857X is auto-selected when
> DA830 EVM is configured. In case the UI card is absent, the probe
> fails in reasonable time.
> 
> The choice menu for UI card peripheral selection is eliminated in
> favor of using the driver config item itself to configure the IO
> expander.
> 
> This has the advantage of users not needing to select the peripheral
> from System menu after selecting the driver from the Device drivers
> menu. As a bonus, this also removes the non-intuitive need to select
> DA830_UI_LCD to get the MMC/SD on the DSK board working with the UI
> card plugged in.
> 
> Signed-off-by: Sekhar Nori <[email protected]>
> ---

I don't know Sekhar...  This looks like a step backwards to me with all
the #ifdef-ery.  Why not keep the code as it is and change the #ifdefs
that are already there (e.g., change the CONFIG_UI ones to CONFIG_FB,
etc)?

Mark
--

_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to