From mboxrd@z Thu Jan 1 00:00:00 1970 From: Hans de Goede Subject: Re: [PATCH] dt: bindings: Add a generic ethernet device binding Date: Sat, 16 Jul 2016 12:12:37 +0200 Message-ID: <888927c4-0987-ccfb-e2de-d03f6d4ab319@redhat.com> References: <1468405204-5845-1-git-send-email-hdegoede@redhat.com> <20160714.161707.2089949241813985527.davem@davemloft.net> <47a052a1-cc8b-0f75-e44a-450c4a0ac075@redhat.com> <20160715.105158.2028840258568316933.davem@davemloft.net> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Cc: robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org, mark.rutland-5wv7dgnIgG8@public.gmane.org, maxime.ripard-wi1+55ScJUtKEb57/3fJTNBPR1lH4CV8@public.gmane.org, wens-jdAy2FN1RRM@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: David Miller Return-path: In-Reply-To: <20160715.105158.2028840258568316933.davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: netdev.vger.kernel.org Hi, On 15-07-16 19:51, David Miller wrote: > From: Hans de Goede > Date: Fri, 15 Jul 2016 08:40:00 +0200 > >> Hi, >> >> On 15-07-16 01:17, David Miller wrote: >>> From: Hans de Goede >>> Date: Wed, 13 Jul 2016 12:20:04 +0200 >>> >>>> On some boards (android tablets) different batches use different sdio >>>> wifi modules. This is not a problem since sdio is a discoverable bus, >>>> so we only need to describe and activate the mmc controller in dt and >>>> then the kernel will automatically load the right driver. >>>> >>>> But sometimes it is useful to specify certain ethernet properties for >>>> these "unknown" sdio devices, specifically we want the boot-loader >>>> to be able to set "local-mac-address" as some of these sdio wifi >>>> modules come without an eeprom / without a factory programmed mac >>>> address. >>>> >>>> Since the exact device is unknown (differs per batch) we cannot use >>>> a wifi-chip specific compatible. This commit adds a new >>>> "generic,ethernet" binding for use in dt-nodes describing such an >>>> unknown ethernet device. >>>> >>>> Cc: Maxime Ripard >>>> Signed-off-by: Hans de Goede >>> >>> Precedence exists for a "system ethernet address" as far back as the >>> original sparc device tree implementation, so please just specify it >>> that way rather than trying to force having to make an alias or >>> reference to it from a specific device. >> >> Some boards where this is applicable have both a wired and a wireless >> ethernet, so one global setting will not work. > > Then call it "eth:local-mac-address" and "wifi:local-mac-address" Until we get a board with 2 ethernet interfaces, really the alias thing is working fine here, that is not the problem. Regards, Hans -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 From: hdegoede@redhat.com (Hans de Goede) Date: Sat, 16 Jul 2016 12:12:37 +0200 Subject: [PATCH] dt: bindings: Add a generic ethernet device binding In-Reply-To: <20160715.105158.2028840258568316933.davem@davemloft.net> References: <1468405204-5845-1-git-send-email-hdegoede@redhat.com> <20160714.161707.2089949241813985527.davem@davemloft.net> <47a052a1-cc8b-0f75-e44a-450c4a0ac075@redhat.com> <20160715.105158.2028840258568316933.davem@davemloft.net> Message-ID: <888927c4-0987-ccfb-e2de-d03f6d4ab319@redhat.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, On 15-07-16 19:51, David Miller wrote: > From: Hans de Goede > Date: Fri, 15 Jul 2016 08:40:00 +0200 > >> Hi, >> >> On 15-07-16 01:17, David Miller wrote: >>> From: Hans de Goede >>> Date: Wed, 13 Jul 2016 12:20:04 +0200 >>> >>>> On some boards (android tablets) different batches use different sdio >>>> wifi modules. This is not a problem since sdio is a discoverable bus, >>>> so we only need to describe and activate the mmc controller in dt and >>>> then the kernel will automatically load the right driver. >>>> >>>> But sometimes it is useful to specify certain ethernet properties for >>>> these "unknown" sdio devices, specifically we want the boot-loader >>>> to be able to set "local-mac-address" as some of these sdio wifi >>>> modules come without an eeprom / without a factory programmed mac >>>> address. >>>> >>>> Since the exact device is unknown (differs per batch) we cannot use >>>> a wifi-chip specific compatible. This commit adds a new >>>> "generic,ethernet" binding for use in dt-nodes describing such an >>>> unknown ethernet device. >>>> >>>> Cc: Maxime Ripard >>>> Signed-off-by: Hans de Goede >>> >>> Precedence exists for a "system ethernet address" as far back as the >>> original sparc device tree implementation, so please just specify it >>> that way rather than trying to force having to make an alias or >>> reference to it from a specific device. >> >> Some boards where this is applicable have both a wired and a wireless >> ethernet, so one global setting will not work. > > Then call it "eth:local-mac-address" and "wifi:local-mac-address" Until we get a board with 2 ethernet interfaces, really the alias thing is working fine here, that is not the problem. Regards, Hans