On Tue, Apr 16, 2019 at 08:01:56PM -0700, Frank Rowand wrote:
> Hi Rob,
>
> On 4/16/19 5:29 PM, Florian Fainelli wrote:
> >
> >
> > On 16/04/2019 13:05, Petr Štetiar wrote:
> >> From: John Crispin <john@xxxxxxxxxxx>
> >>
> >> Many embedded devices have information such as MAC addresses stored
> >> inside MTD devices. This patch allows us to add a property inside a node
> >> describing a network interface. The new property points at a MTD
> >> partition with an offset where the MAC address can be found.
> >>
> >> This patch has originated in OpenWrt some time ago, so in order to
> >> consider usefulness of this patch, here are some real-world numbers
> >> which hopefully speak for themselves:
> >>
> >> * mtd-mac-address used 497 times in 357 device tree files
> >> * mtd-mac-address-increment used 74 times in 58 device tree files
> >> * mtd-mac-address-increment-byte used 1 time in 1 device tree file
> >>
> >> Signed-off-by: John Crispin <john@xxxxxxxxxxx>
> >> Signed-off-by: Felix Fietkau <nbd@xxxxxxxx>
> >> [cleanup of the patch for upstream submission]
> >> Signed-off-by: Petr Štetiar <ynezz@xxxxxxx>
> >> ---
> >
> > [snip]
> >
> >> +static const void *of_get_mac_address_mtd(struct device_node *np)
> >> +{
> >> +#ifdef CONFIG_MTD
> >> + void *addr;
> >> + size_t retlen;
> >> + int size, ret;
> >> + u8 mac[ETH_ALEN];
> >> + phandle phandle;
> >> + const char *part;
> >> + const __be32 *list;
> >> + struct mtd_info *mtd;
> >> + struct property *prop;
> >> + u32 mac_inc = 0;
> >> + u32 inc_idx = ETH_ALEN-1;
> >> + struct device_node *mtd_np = NULL;
> >
> > Reverse christmas tree would look a bit nicer here.
>
> Do we a variable declaration format preference for drivers/of/*?
We'd better get one. It's all the rage.
How about fallen Christmas tree:
int a;
bool fallen;
char christmas_tree;
int for_our;
int dt;
Rob
