On 07/24/2013 04:55 PM, John W. Linville wrote: > Did Hauke respond to this? > > On Tue, Jun 25, 2013 at 10:13:46AM +0200, Rafał Miłecki wrote: >> With recent patches ssb can fetch info about serial flash and register >> it as a platform device. No more reasons to mark it BROKEN. >> >> Signed-off-by: Rafał Miłecki <zajec5@xxxxxxxxx> >> Cc: Hauke Mehrtens <hauke@xxxxxxxxxx> >> --- >> Hauke: does it look OK for you? Did you give it a try? >> --- >> drivers/ssb/Kconfig | 2 +- >> drivers/ssb/driver_chipcommon_sflash.c | 8 +++----- >> 2 files changed, 4 insertions(+), 6 deletions(-) >> >> diff --git a/drivers/ssb/Kconfig b/drivers/ssb/Kconfig >> index 4f0dd37..775251d 100644 >> --- a/drivers/ssb/Kconfig >> +++ b/drivers/ssb/Kconfig >> @@ -139,7 +139,7 @@ config SSB_DRIVER_MIPS >> >> config SSB_SFLASH >> bool "SSB serial flash support" >> - depends on SSB_DRIVER_MIPS && BROKEN >> + depends on SSB_DRIVER_MIPS >> default y >> >> # Assumption: We are on embedded, if we compile the MIPS core. >> diff --git a/drivers/ssb/driver_chipcommon_sflash.c b/drivers/ssb/driver_chipcommon_sflash.c >> index e84cf04..50328de 100644 >> --- a/drivers/ssb/driver_chipcommon_sflash.c >> +++ b/drivers/ssb/driver_chipcommon_sflash.c >> @@ -151,8 +151,8 @@ int ssb_sflash_init(struct ssb_chipcommon *cc) >> sflash->size = sflash->blocksize * sflash->numblocks; >> sflash->present = true; >> >> - pr_info("Found %s serial flash (blocksize: 0x%X, blocks: %d)\n", >> - e->name, e->blocksize, e->numblocks); >> + pr_info("Found %s serial flash (size: %dKiB, blocksize: 0x%X, blocks: %d)\n", >> + e->name, sflash->size / 1024, e->blocksize, e->numblocks); >> >> /* Prepare platform device, but don't register it yet. It's too early, >> * malloc (required by device_private_init) is not available yet. */ >> @@ -160,7 +160,5 @@ int ssb_sflash_init(struct ssb_chipcommon *cc) >> sflash->size; >> ssb_sflash_dev.dev.platform_data = sflash; >> >> - pr_err("Serial flash support is not implemented yet!\n"); >> - >> - return -ENOTSUPP; >> + return 0; >> } Hi, I do not own a device with a serial flash chip connected to ssb. This patch looks ok to me, please apply it. Hauke -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html