Re: [PATCH] bcma: define some additional cores IDs

[Julian Calaby <julian.calaby@xxxxxxxxx>]

Hi Rafał,

Minor point:

On Tue, Jun 26, 2012 at 6:12 AM, Rafał Miłecki <zajec5@xxxxxxxxx> wrote:
> Some of them are BCM4706 specific AFAWK. Most of them was confirmed on
> Netgear WNDR450.
>
> Signed-off-by: Rafał Miłecki <zajec5@xxxxxxxxx>
> ---
>  drivers/bcma/scan.c       |    6 ++++++
>  include/linux/bcma/bcma.h |    7 +++++++
>  2 files changed, 13 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/bcma/scan.c b/drivers/bcma/scan.c
> index 5ed0718..a342058 100644
> --- a/drivers/bcma/scan.c
> +++ b/drivers/bcma/scan.c
> @@ -70,6 +70,13 @@ struct bcma_host_ops {
>
>  /* Core-ID values. */
>  #define BCMA_CORE_OOB_ROUTER           0x367   /* Out of band */
> +#define BCMA_CORE_4706_CHIPCOMMON      0x500
> +#define BCMA_CORE_4706_SOC_RAM         0x50E
> +#define BCMA_CORE_4706_MAC_GBIT                0x52D
> +#define BCMA_CORE_AMEMC                        0x52E   /* DDR1/2 memory controller core */
> +#define BCMA_CORE_ALTA                 0x534   /* I2S core */
> +#define BCMA_CORE_4706_MAC_GBIT_COMMON 0x5DC
> +#define BCMA_CORE_DDR23_PHY            0x5DD

You don't define a name for the DDR23_PHY core.

Thanks,

-- 
Julian Calaby

Email: julian.calaby@xxxxxxxxx
Profile: http://www.google.com/profiles/julian.calaby/
.Plan: http://sites.google.com/site/juliancalaby/
--
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