2023-03-23 11:41:14

by Bhavya Kapoor

[permalink] [raw]
Subject: [PATCH v2] arm64: defconfig: Enable CAN PHY transceiver driver

Enable CAN PHY transceiver driver to be built as a module.

Signed-off-by: Bhavya Kapoor <[email protected]>
---
changelog v1->v2:
- changed position of config in the file

arch/arm64/configs/defconfig | 1 +
1 file changed, 1 insertion(+)

diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index 7790ee42c68a..6e123de3a9cf 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -1292,6 +1292,7 @@ CONFIG_RESET_QCOM_PDC=m
CONFIG_RESET_RZG2L_USBPHY_CTRL=y
CONFIG_RESET_TI_SCI=y
CONFIG_PHY_XGENE=y
+CONFIG_PHY_CAN_TRANSCEIVER=m
CONFIG_PHY_SUN4I_USB=y
CONFIG_PHY_CADENCE_TORRENT=m
CONFIG_PHY_CADENCE_SIERRA=m
--
2.34.1


2023-03-23 13:54:48

by Bjorn Andersson

[permalink] [raw]
Subject: Re: [PATCH v2] arm64: defconfig: Enable CAN PHY transceiver driver

On Thu, Mar 23, 2023 at 05:03:24PM +0530, Bhavya Kapoor wrote:
> Enable CAN PHY transceiver driver to be built as a module.

Please use the commit message to describe why the driver should be
enabled. The patch and the subject already states clearly that the
driver is being enabled.

Regards,
Bjorn

>
> Signed-off-by: Bhavya Kapoor <[email protected]>
> ---
> changelog v1->v2:
> - changed position of config in the file
>
> arch/arm64/configs/defconfig | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
> index 7790ee42c68a..6e123de3a9cf 100644
> --- a/arch/arm64/configs/defconfig
> +++ b/arch/arm64/configs/defconfig
> @@ -1292,6 +1292,7 @@ CONFIG_RESET_QCOM_PDC=m
> CONFIG_RESET_RZG2L_USBPHY_CTRL=y
> CONFIG_RESET_TI_SCI=y
> CONFIG_PHY_XGENE=y
> +CONFIG_PHY_CAN_TRANSCEIVER=m
> CONFIG_PHY_SUN4I_USB=y
> CONFIG_PHY_CADENCE_TORRENT=m
> CONFIG_PHY_CADENCE_SIERRA=m
> --
> 2.34.1
>

2023-03-23 16:10:54

by Geert Uytterhoeven

[permalink] [raw]
Subject: Re: [PATCH v2] arm64: defconfig: Enable CAN PHY transceiver driver

On Thu, Mar 23, 2023 at 2:53 PM Bjorn Andersson
<[email protected]> wrote:
> On Thu, Mar 23, 2023 at 05:03:24PM +0530, Bhavya Kapoor wrote:
> > Enable CAN PHY transceiver driver to be built as a module.
>
> Please use the commit message to describe why the driver should be
> enabled. The patch and the subject already states clearly that the
> driver is being enabled.

And if no one has inspiration: I wouldn't mind "because it is needed
for CAN-FD on the Renesas White-Hawk development board" ;-)

> > --- a/arch/arm64/configs/defconfig
> > +++ b/arch/arm64/configs/defconfig
> > @@ -1292,6 +1292,7 @@ CONFIG_RESET_QCOM_PDC=m
> > CONFIG_RESET_RZG2L_USBPHY_CTRL=y
> > CONFIG_RESET_TI_SCI=y
> > CONFIG_PHY_XGENE=y
> > +CONFIG_PHY_CAN_TRANSCEIVER=m
> > CONFIG_PHY_SUN4I_USB=y
> > CONFIG_PHY_CADENCE_TORRENT=m
> > CONFIG_PHY_CADENCE_SIERRA=m

For the actual change:
Reviewed-by: Geert Uytterhoeven <[email protected]>

Gr{oetje,eeting}s,

Geert

--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- [email protected]

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds

2023-03-24 06:17:19

by Bhavya Kapoor

[permalink] [raw]
Subject: Re: [EXTERNAL] Re: [PATCH v2] arm64: defconfig: Enable CAN PHY transceiver driver


On 23/03/23 21:39, Geert Uytterhoeven wrote:
> On Thu, Mar 23, 2023 at 2:53 PM Bjorn Andersson
> <[email protected]> wrote:
>> On Thu, Mar 23, 2023 at 05:03:24PM +0530, Bhavya Kapoor wrote:
>>> Enable CAN PHY transceiver driver to be built as a module.
>> Please use the commit message to describe why the driver should be
>> enabled. The patch and the subject already states clearly that the
>> driver is being enabled.
> And if no one has inspiration: I wouldn't mind "because it is needed
> for CAN-FD on the Renesas White-Hawk development board" ;-)

Ahh , i do have the inspiration,lol ????. I just wanted to keep it
concise. Lemme send a v3 with necessary details.

Thanks.

>
>>> --- a/arch/arm64/configs/defconfig
>>> +++ b/arch/arm64/configs/defconfig
>>> @@ -1292,6 +1292,7 @@ CONFIG_RESET_QCOM_PDC=m
>>> CONFIG_RESET_RZG2L_USBPHY_CTRL=y
>>> CONFIG_RESET_TI_SCI=y
>>> CONFIG_PHY_XGENE=y
>>> +CONFIG_PHY_CAN_TRANSCEIVER=m
>>> CONFIG_PHY_SUN4I_USB=y
>>> CONFIG_PHY_CADENCE_TORRENT=m
>>> CONFIG_PHY_CADENCE_SIERRA=m
> For the actual change:
> Reviewed-by: Geert Uytterhoeven <[email protected]>
>
> Gr{oetje,eeting}s,
>
> Geert
>