Re: [PATCH v3 0/3] Adding DT support for TI HECC module

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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


[Index of Archives]     [Linux Arm (vger)]     [ARM Kernel]     [ARM MSM]     [Linux Tegra]     [Linux WPAN Networking]     [Linux Wireless Networking]     [Maemo Users]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite Trails]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux