* Device tree conversion of spi device vs. controller_data @ 2012-04-08 10:24 Roland Stigge 2012-04-13 3:29 ` Grant Likely 0 siblings, 1 reply; 5+ messages in thread From: Roland Stigge @ 2012-04-08 10:24 UTC (permalink / raw) To: Grant Likely; +Cc: spi-devel-general, linux-kernel Hi, upon DT conversion of LPC32xx, I came across the at25 spi eeprom which needs .controller_data in struct spi_board_info for slave registration via spi_register_board_info() (non-DT-case). In the DT case, we need to eliminate this explicit registering call. Therefore, I need to somehow replace the .controller_data passing in the DT case to pass this data to the spi core. But I can't find such a mechanism (v3.4-rc2). Is there already a concept or API that I overlooked (please just point me to it) or what can I do to solve this? (Looks like a general issue for spi slave registration via DT). Thanks in advance, Roland ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Device tree conversion of spi device vs. controller_data 2012-04-08 10:24 Device tree conversion of spi device vs. controller_data Roland Stigge @ 2012-04-13 3:29 ` Grant Likely 2012-04-13 7:10 ` Roland Stigge 0 siblings, 1 reply; 5+ messages in thread From: Grant Likely @ 2012-04-13 3:29 UTC (permalink / raw) To: Roland Stigge; +Cc: spi-devel-general, linux-kernel On Sun, 08 Apr 2012 12:24:50 +0200, Roland Stigge <stigge@antcom.de> wrote: > Hi, > > upon DT conversion of LPC32xx, I came across the at25 spi eeprom which > needs .controller_data in struct spi_board_info for slave registration > via spi_register_board_info() (non-DT-case). > > In the DT case, we need to eliminate this explicit registering call. > Therefore, I need to somehow replace the .controller_data passing in the > DT case to pass this data to the spi core. But I can't find such a > mechanism (v3.4-rc2). Is there already a concept or API that I > overlooked (please just point me to it) or what can I do to solve this? > (Looks like a general issue for spi slave registration via DT). Can you point me at specific code? .controller_data is owned by the spi_master driver. The eeprom driver shouldn't be accessing it directly at all; but I'll need to look at specific code to really understand what is going on. g. -- Grant Likely, B.Sc, P.Eng. Secret Lab Technologies,Ltd. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Device tree conversion of spi device vs. controller_data 2012-04-13 3:29 ` Grant Likely @ 2012-04-13 7:10 ` Roland Stigge 2012-04-27 18:03 ` Grant Likely 0 siblings, 1 reply; 5+ messages in thread From: Roland Stigge @ 2012-04-13 7:10 UTC (permalink / raw) To: Grant Likely; +Cc: spi-devel-general, linux-kernel On 04/13/2012 05:29 AM, Grant Likely wrote: >> upon DT conversion of LPC32xx, I came across the at25 spi eeprom which >> needs .controller_data in struct spi_board_info for slave registration >> via spi_register_board_info() (non-DT-case). >> >> In the DT case, we need to eliminate this explicit registering call. >> Therefore, I need to somehow replace the .controller_data passing in the >> DT case to pass this data to the spi core. But I can't find such a >> mechanism (v3.4-rc2). Is there already a concept or API that I >> overlooked (please just point me to it) or what can I do to solve this? >> (Looks like a general issue for spi slave registration via DT). > > Can you point me at specific code? > > .controller_data is owned by the spi_master driver. The eeprom driver > shouldn't be accessing it directly at all; but I'll need to look at > specific code to really understand what is going on. Please have a look at arch/arm/mach-lpc32xx/phy3250.c : static struct pl022_config_chip spi0_chip_info = { .com_mode = INTERRUPT_TRANSFER, .iface = SSP_INTERFACE_MOTOROLA_SPI, .hierarchy = SSP_MASTER, .slave_tx_disable = 0, .rx_lev_trig = SSP_RX_4_OR_MORE_ELEM, .tx_lev_trig = SSP_TX_4_OR_MORE_EMPTY_LOC, .ctrl_len = SSP_BITS_8, .wait_state = SSP_MWIRE_WAIT_ZERO, .duplex = SSP_MICROWIRE_CHANNEL_FULL_DUPLEX, .cs_control = phy3250_spi_cs_set, }; ... static struct spi_eeprom eeprom = { .name = "at25256a", .byte_len = 0x8000, .page_size = 64, .flags = EE_ADDR2, }; static struct spi_board_info info[] = { { .modalias = "at25", .max_speed_hz = 5000000, .bus_num = 0, .chip_select = 0, .mode = SPI_MODE_0, .platform_data = &eeprom, .controller_data = &spi0_chip_info, }, }; spi_register_board_info(info, ARRAY_SIZE(info)); Trying to replace .controller_data at this point with device tree settings passing, I struggled with missing resp. node properties. Any hint how this can be done? Thanks in advance, Roland ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Device tree conversion of spi device vs. controller_data 2012-04-13 7:10 ` Roland Stigge @ 2012-04-27 18:03 ` Grant Likely 2012-04-28 10:39 ` Roland Stigge 0 siblings, 1 reply; 5+ messages in thread From: Grant Likely @ 2012-04-27 18:03 UTC (permalink / raw) To: Roland Stigge; +Cc: spi-devel-general, linux-kernel On Fri, 13 Apr 2012 09:10:15 +0200, Roland Stigge <stigge@antcom.de> wrote: > On 04/13/2012 05:29 AM, Grant Likely wrote: > >> upon DT conversion of LPC32xx, I came across the at25 spi eeprom which > >> needs .controller_data in struct spi_board_info for slave registration > >> via spi_register_board_info() (non-DT-case). > >> > >> In the DT case, we need to eliminate this explicit registering call. > >> Therefore, I need to somehow replace the .controller_data passing in the > >> DT case to pass this data to the spi core. But I can't find such a > >> mechanism (v3.4-rc2). Is there already a concept or API that I > >> overlooked (please just point me to it) or what can I do to solve this? > >> (Looks like a general issue for spi slave registration via DT). > > > > Can you point me at specific code? > > > > .controller_data is owned by the spi_master driver. The eeprom driver > > shouldn't be accessing it directly at all; but I'll need to look at > > specific code to really understand what is going on. > > Please have a look at arch/arm/mach-lpc32xx/phy3250.c : > > static struct pl022_config_chip spi0_chip_info = { > .com_mode = INTERRUPT_TRANSFER, > .iface = SSP_INTERFACE_MOTOROLA_SPI, > .hierarchy = SSP_MASTER, > .slave_tx_disable = 0, > .rx_lev_trig = SSP_RX_4_OR_MORE_ELEM, > .tx_lev_trig = SSP_TX_4_OR_MORE_EMPTY_LOC, > .ctrl_len = SSP_BITS_8, > .wait_state = SSP_MWIRE_WAIT_ZERO, > .duplex = SSP_MICROWIRE_CHANNEL_FULL_DUPLEX, > .cs_control = phy3250_spi_cs_set, phy3250_spi_cs_set is only a wrapper around gpio_set_value(). GPIO CS manipulation should be handled by the core pl022 spi driver, so that gets rid of the hardest part of this conversion (callbacks in platform data). The pl022_config_chip structure is 100% owned by the spi controller (pl022). If the bus driver depends on this structure, then it is the responsibility of the bus driver to allocate one and set the controller data pointer for each of the child devices. You may need to add pl022-specific properties to each of the child nodes to populate this data correctly if it cannot be determined with other means. There is no existing API for this since it is entirely pl022 specific code. > }; > > ... > > static struct spi_eeprom eeprom = { > .name = "at25256a", > .byte_len = 0x8000, > .page_size = 64, > .flags = EE_ADDR2, This data is easy and is 'owned' by the eeprom data. It can be directly converted into device tree properties in the eeprom node. > }; > > static struct spi_board_info info[] = { > { > .modalias = "at25", > .max_speed_hz = 5000000, > .bus_num = 0, > .chip_select = 0, > .mode = SPI_MODE_0, > .platform_data = &eeprom, > .controller_data = &spi0_chip_info, > }, > }; Everything in spi_board_info directly translates over to DT today. You shouldn't have any problems here. > > spi_register_board_info(info, ARRAY_SIZE(info)); > > Trying to replace .controller_data at this point with device tree > settings passing, I struggled with missing resp. node properties. > > Any hint how this can be done? > > Thanks in advance, > > Roland -- Grant Likely, B.Sc, P.Eng. Secret Lab Technologies, Ltd. ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: Device tree conversion of spi device vs. controller_data 2012-04-27 18:03 ` Grant Likely @ 2012-04-28 10:39 ` Roland Stigge 0 siblings, 0 replies; 5+ messages in thread From: Roland Stigge @ 2012-04-28 10:39 UTC (permalink / raw) To: Grant Likely; +Cc: spi-devel-general, linux-kernel On 27/04/12 20:03, Grant Likely wrote: >>> Can you point me at specific code? >>> >>> .controller_data is owned by the spi_master driver. The eeprom driver >>> shouldn't be accessing it directly at all; but I'll need to look at >>> specific code to really understand what is going on. >> >> Please have a look at arch/arm/mach-lpc32xx/phy3250.c : >> >> static struct pl022_config_chip spi0_chip_info = { >> .com_mode = INTERRUPT_TRANSFER, >> .iface = SSP_INTERFACE_MOTOROLA_SPI, >> .hierarchy = SSP_MASTER, >> .slave_tx_disable = 0, >> .rx_lev_trig = SSP_RX_4_OR_MORE_ELEM, >> .tx_lev_trig = SSP_TX_4_OR_MORE_EMPTY_LOC, >> .ctrl_len = SSP_BITS_8, >> .wait_state = SSP_MWIRE_WAIT_ZERO, >> .duplex = SSP_MICROWIRE_CHANNEL_FULL_DUPLEX, >> .cs_control = phy3250_spi_cs_set, > > phy3250_spi_cs_set is only a wrapper around gpio_set_value(). GPIO CS > manipulation should be handled by the core pl022 spi driver, so that > gets rid of the hardest part of this conversion (callbacks in platform > data). > > The pl022_config_chip structure is 100% owned by the spi controller > (pl022). If the bus driver depends on this structure, then it is the > responsibility of the bus driver to allocate one and set the > controller data pointer for each of the child devices. You may need > to add pl022-specific properties to each of the child nodes to > populate this data correctly if it cannot be determined with other > means. Yes, that's what I'll need to do. Thanks! Roland ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2012-04-28 10:39 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2012-04-08 10:24 Device tree conversion of spi device vs. controller_data Roland Stigge 2012-04-13 3:29 ` Grant Likely 2012-04-13 7:10 ` Roland Stigge 2012-04-27 18:03 ` Grant Likely 2012-04-28 10:39 ` Roland Stigge
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).