[U-Boot] [PATCH] arm: socfpga: use imply instead of select for spi/mmc

[Lukasz Majewski <lukma@denx.de>]

Hi Simon, Marek,

> On 02/12/2018 04:16 PM, Simon Goldschmidt wrote:
> > On 12.02.2018 15:54, Marek Vasut wrote:  
> >> On 02/12/2018 03:52 PM, Simon Goldschmidt wrote:  
> >>> In arch/arm/Kconfig, ARCH_SOCFPGA currently selects some
> >>> thins for spi flash and mmc that are not required by the hw.  
> >> things ?
> >>
> >> what hw, all supported boards you mean ?
> >>  
> >>> Give users more freedom of choice and use imply here instead
> >>> of select.
> >>>
> >>> This should allow disabling spi support completely or using
> >>> sd/mmc boot in "raw mode" (no partitions).  
> >> You can just turn them off in the defconfigs, but I get what you're
> >> trying to say.
> >>
> >> Just reword the commit message and do make savedefconfig for all
> >> the socfpga platforms (in a separate patch) to sync the defconfigs
> >> up.  
> > 
> > OK, thanks. Given that the same files are touched, should I wait
> > for the CONFIG_HW_WATCHDOG patches from Lukasz to be applied
> > first?  
> 
> You two figure it out :)

It is always better to ask socfpga veteran for his opinion :-)

> 

Simon, feel free to perform the clean up. I will have to look more
closely to the WDT move to Kconfig.

Even better, you are more than welcome to use/squash this patch to your
work:
http://patchwork.ozlabs.org/patch/871578/

It will simplify my WDT conversion work.

Thanks for help.


Best regards,

Lukasz Majewski

--

DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 488 bytes
Desc: OpenPGP digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20180212/85ca4e8b/attachment.sig>