On 02/17/2017 10:23 AM, Yegor Yefremov wrote: > On Fri, Feb 17, 2017 at 9:51 AM, Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx> wrote: >> On 02/17/2017 09:50 AM, Yegor Yefremov wrote: >>> On Fri, Feb 17, 2017 at 9:46 AM, Marc Kleine-Budde <mkl@xxxxxxxxxxxxxx> wrote: >>>> On 02/15/2017 03:48 PM, yegorslists@xxxxxxxxxxxxxx wrote: >>>>> From: Yegor Yefremov <yegorslists@xxxxxxxxxxxxxx> >>>>> >>>>> This is an attempt to revive DT support for TI HECC that was started in 2015 [1]. >>>>> >>>>> Changes v2 -> v3: >>>>> - rename compatible property to "ti,am3517-hecc" (though there is a stipped >>>>> device named 3505, EMAC driver already uses am3517-emac name, so keep this >>>>> property consistent) >>>>> - use reg-names to specify different HECC I/O mapping (suggested by Tony Lindgren) >>>>> - provide fallback offsets and sizes for board files >>>> >>>> Are there any board files using te_hecc intree? I don't see any on the >>>> first glance. What about making the driver DT only? >>> >>> Good idea. I aslo wanted to ask this. So this means we could >>> completely get rid of include/linux/can/platform/ti_hecc.h? >> >> If there isn't any in-tree user, thes yes. > > Btw. what should we do about "@transceiver_switch: platform specific > callback fn for transceiver control"? Model it via something like > this: > > enable-gpios = <&gpio6 15 GPIO_ACTIVE_HIGH>; /* gpio 175 */ Make it a regulator as in the flexcan: > reg_xceiver = devm_regulator_get(&pdev->dev, "xceiver"); Marc -- Pengutronix e.K. | Marc Kleine-Budde | Industrial Linux Solutions | Phone: +49-231-2826-924 | Vertretung West/Dortmund | Fax: +49-5121-206917-5555 | Amtsgericht Hildesheim, HRA 2686 | http://www.pengutronix.de |
Attachment:
signature.asc
Description: OpenPGP digital signature
