On Thu, Mar 6, 2014 at 7:41 PM, Alex Elder <elder@xxxxxxxx> wrote: >> This patchset adds very minimal support for the BCM63138 SoC which is >> a xDSL SoC using a dual Cortex A9 CPU complex. > > > Why do you use "63xx" for the generic name, when the specific > part you are defining here is 63138? I would have expected > "63xxx". The product line has a mix of 4-digit and 5-digit part numbers. The drivers for onchip peripherals all used "bcm63xx": $ find drivers/ | grep 63xx drivers/tty/serial/bcm63xx_uart.c drivers/pcmcia/bcm63xx_pcmcia.h drivers/pcmcia/bcm63xx_pcmcia.c drivers/usb/gadget/bcm63xx_udc.c drivers/watchdog/max63xx_wdt.c drivers/watchdog/bcm63xx_wdt.c drivers/char/hw_random/bcm63xx-rng.c drivers/mtd/bcm63xxpart.c drivers/net/phy/bcm63xx.c drivers/net/ethernet/broadcom/bcm63xx_enet.h drivers/net/ethernet/broadcom/bcm63xx_enet.c drivers/spi/spi-bcm63xx.c drivers/spi/spi-bcm63xx-hsspi.c -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
