Re: [U-Boot] [PATCH 2/3] arm: mx6: tqma6: Update to optionally configure an alternative SPI setup

2015-05-05 Thread Stefano Babic
On 05/05/2015 11:37, Stefan Roese wrote: Hi Stefano, On 12.03.2015 13:34, Stefan Roese wrote: By making the tqma6_iomuxc_spi() weak, this patch adds the possibility to add a different function for this SPI configuration. This can be used by other baseboards, that might have a different SPI

Re: [U-Boot] [PATCH 2/3] arm: mx6: tqma6: Update to optionally configure an alternative SPI setup

2015-05-05 Thread Stefan Roese
Hi Stefano, On 12.03.2015 13:34, Stefan Roese wrote: By making the tqma6_iomuxc_spi() weak, this patch adds the possibility to add a different function for this SPI configuration. This can be used by other baseboards, that might have a different SPI setup. This patch will be used by the

Re: [U-Boot] [PATCH 2/3] arm: mx6: tqma6: Update to optionally configure an alternative SPI setup

2015-03-13 Thread Markus Niebel
Hello Stefan, Am 12.03.2015 um 16:18 schrieb Stefan Roese: Hi Markus, On 12.03.2015 15:25, Markus Niebel wrote: Am 12.03.2015 um 13:34 schrieb Stefan Roese: By making the tqma6_iomuxc_spi() weak, this patch adds the possibility to add a different function for this SPI configuration. This

Re: [U-Boot] [PATCH 2/3] arm: mx6: tqma6: Update to optionally configure an alternative SPI setup

2015-03-13 Thread Stefan Roese
Hi Markus, On 13.03.2015 10:05, Markus Niebel wrote: -static void tqma6_iomuxc_spi(void) +__weak void tqma6_iomuxc_spi(void) { unsigned i; When implementing an baseboard specific init handler, we will get a warning about unused tqma6_ecspi1_pads and tqma6_ecspi1_cs, or did I miss

Re: [U-Boot] [PATCH 2/3] arm: mx6: tqma6: Update to optionally configure an alternative SPI setup

2015-03-13 Thread Stefan Roese
On 13.03.2015 14:25, Stefan Roese wrote: But OK, this looks not very nice As I said, not completely ready, but would prevent some code duplication. With my current approach we have code duplication. Ups. I meant we have *no* code duplication. Sorry! Thanks, Stefan

Re: [U-Boot] [PATCH 2/3] arm: mx6: tqma6: Update to optionally configure an alternative SPI setup

2015-03-13 Thread Markus Niebel
Hello Stefan, Am 13.03.2015 um 14:25 schrieb Stefan Roese: Hi Markus, On 13.03.2015 10:05, Markus Niebel wrote: Not sure if I understand this correctly. Could you give an example? Again, my current baseboard has no SPI at all. tqma6_iomux_spi is for the SPI controller the serial nor is

[U-Boot] [PATCH 2/3] arm: mx6: tqma6: Update to optionally configure an alternative SPI setup

2015-03-12 Thread Stefan Roese
By making the tqma6_iomuxc_spi() weak, this patch adds the possibility to add a different function for this SPI configuration. This can be used by other baseboards, that might have a different SPI setup. This patch will be used by the upcoming WRU-IV board support which also uses the TQMa6 SoM.

Re: [U-Boot] [PATCH 2/3] arm: mx6: tqma6: Update to optionally configure an alternative SPI setup

2015-03-12 Thread Markus Niebel
Hello Stefan, Am 12.03.2015 um 13:34 schrieb Stefan Roese: By making the tqma6_iomuxc_spi() weak, this patch adds the possibility to add a different function for this SPI configuration. This can be used by other baseboards, that might have a different SPI setup. This patch will be used by

Re: [U-Boot] [PATCH 2/3] arm: mx6: tqma6: Update to optionally configure an alternative SPI setup

2015-03-12 Thread Stefan Roese
Hi Markus, On 12.03.2015 15:25, Markus Niebel wrote: Am 12.03.2015 um 13:34 schrieb Stefan Roese: By making the tqma6_iomuxc_spi() weak, this patch adds the possibility to add a different function for this SPI configuration. This can be used by other baseboards, that might have a different SPI