[PATCH V2 2/4] mtd: bcm47xxpart.c: Adding Xaiomi board support

Rafał Miłecki zajec5 at gmail.com
Sun Jul 26 00:05:31 PDT 2015


On 25 July 2015 at 16:37, Joseph East <eastyjr at gmail.com> wrote:
> ---

Oh and now you're sending my patch pretending its your. NACK!


>  drivers/mtd/bcm47xxpart.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/mtd/bcm47xxpart.c b/drivers/mtd/bcm47xxpart.c
> index 55a1462..a27d185 100644
> --- a/drivers/mtd/bcm47xxpart.c
> +++ b/drivers/mtd/bcm47xxpart.c
> @@ -33,6 +33,7 @@
>  /* Magics */
>  #define BOARD_DATA_MAGIC               0x5246504D      /* MPFR */
>  #define BOARD_DATA_MAGIC2              0xBD0D0BBD
> +#define BOARD_DATA_XIAOMI_MAGIC        0x474D4442      /* GMDB */
>  #define CFE_MAGIC                      0x43464531      /* 1EFC */
>  #define FACTORY_MAGIC                  0x59544346      /* FCTY */
>  #define NVRAM_HEADER                   0x48534C46      /* FLSH */
> @@ -263,7 +264,8 @@ static int bcm47xxpart_parse(struct mtd_info *master,
>                 }
>
>                 /* Some devices (ex. WNDR3700v3) don't have a standard 'MPFR' */
> -               if (buf[0x000 / 4] == BOARD_DATA_MAGIC2) {
> +               if (buf[0x000 / 4] == BOARD_DATA_MAGIC2 ||
> +                   le32_to_cpu(buf[0x000 / 4]) == BOARD_DATA_XIAOMI_MAGIC) {

le32_to_cpu won't work here obviously, so it seems you didn't even
test it on a proper device. Nack.

-- 
Rafał



More information about the linux-mtd mailing list