U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Marek Vasut <marex@denx.de>
To: u-boot@lists.denx.de
Subject: [U-Boot] [PATCH] phycore-pcl060: U-boot support for Phytec phyCORE PCL060
Date: Tue, 23 Apr 2019 13:16:07 +0200	[thread overview]
Message-ID: <3f7f367f-dfe5-3dcc-86ac-5cd4a269f944@denx.de> (raw)
In-Reply-To: <4892f04b-fd1d-4eab-8171-63380282791c@phytec.de>

On 4/23/19 11:33 AM, Wadim Egorov wrote:
> Hi,
> 
> On 19.04.19 17:20, Marek Vasut wrote:
>> On 4/19/19 5:06 PM, Parthiban Nallathambi wrote:
>>> Hello Marek,
>>>
>>> On 4/19/19 3:35 PM, Marek Vasut wrote:
>>>> On 4/19/19 3:18 PM, Parthiban Nallathambi wrote:
>>>>
>>>> Hi,
>>>>
>>>> [...]
>>>>
>>>>>>> diff --git a/arch/arm/mach-omap2/Kconfig b/arch/arm/mach-omap2/Kconfig
>>>>>>> index d29f1ca0b5..9336439340 100644
>>>>>>> --- a/arch/arm/mach-omap2/Kconfig
>>>>>>> +++ b/arch/arm/mach-omap2/Kconfig
>>>>>>> @@ -186,6 +186,7 @@ source "board/ti/am43xx/Kconfig"
>>>>>>>  source "board/ti/am335x/Kconfig"
>>>>>>>  source "board/compulab/cm_t335/Kconfig"
>>>>>>>  source "board/compulab/cm_t43/Kconfig"
>>>>>>> +source "board/phytec/phycore_pcl060/Kconfig"
>>>>>> Here [1] it says the name of the SoM is PCM-060 , what is PCL-060 ?
>>>>>>
>>>>>> [1]
>>>>>> https://www.phytec.eu/product-eu/system-on-modules/phycore-am335x-download/
>>>>>>  [...]
>>>>> This differs only by the connector. PCM variants are pluggable and PCL variants
>>>>> are direct soliderable to the carrier board.
>>>>>
>>>>> Copied from [1]:
>>>>> The PCL-060 System On Module is a connector-less, BGA style variant of the
>>>>> PCM-060/phyCORE-AM335x R2 SOM. Unlike traditional Phytec SOM products that support
>>>>> high density connectors, the PCL-060 SOM is directly soldered down to the
>>>>> phyBOARD-Wega AM335x using Phytec's Direct Solder Connect technology (DSC). This
>>>>> solution offers an ultra-low cost Single Board Computer for the AM335x processor, while
>>>>> maintaining most of the advantages of the SOM concept.
>>>>>
>>>>> [1] https://www.phytec.de/fileadmin/user_upload/downloads/Manuals/L-845e_1.pdf
>>>> Ah damn, this looks like a consistency problem is coming up. We have
>>>> multiple PCM* SoMs in U-Boot, one PCL* SoM and now another PCL/PCM SoM.
>>>> But the PCL063 isn't even manufactured in variant with connectors, so I
>>>> guess we can ignore that one.
>>>>
>>>> I wonder whether we should stick to PCM* for all of the Phytec SoMs for
>>>> consistency sake and document that PCL060 is also supported or maybe
>>>> there's a better way ?
>>> Does PCX/PCx makes sense? But we have the same problem with variscite [1] SoM's
>>> (either SODIMM or solderable).
>>>
>>> [1] https://lists.denx.de/pipermail/u-boot/2019-April/365667.html
>> PCX would introduce another option, in addition to PCM/PCL, one which
>> cannot be easily found on the internet, so I'd like to avoid that. I
>> am banking toward the sticking with PCM where possible (simply because
>> that's what $searchengine spits out first when you look for that SoM,
>> and because we already have plenty of PCM-nnn SoMs), but maybe someone
>> has a better idea .
> 
> why not simply use the full name of the module instead?
> 
> - phyCORE-AM335x
> - phyCORE-AM335x R2
> 
> I did the same for the phyCORE-RK3288.

Well, that's fine by me, and I think it's better than the pcm060 or
pcl060 too. Let's go with this, unless there are objections.

-- 
Best regards,
Marek Vasut

  reply	other threads:[~2019-04-23 11:16 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-04-18 15:01 [U-Boot] [PATCH] phycore-pcl060: U-boot support for Phytec phyCORE PCL060 Niel Fourie
2019-04-19  9:47 ` Marek Vasut
2019-04-19 13:18   ` Parthiban Nallathambi
2019-04-19 13:35     ` Marek Vasut
2019-04-19 15:06       ` Parthiban Nallathambi
2019-04-19 15:20         ` Marek Vasut
2019-04-23  9:33           ` Wadim Egorov
2019-04-23 11:16             ` Marek Vasut [this message]
2019-04-25  8:31   ` Niel Fourie
2019-04-25 10:43     ` Marek Vasut
2019-04-25 12:41       ` Eugeniu Rosca

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=3f7f367f-dfe5-3dcc-86ac-5cd4a269f944@denx.de \
    --to=marex@denx.de \
    --cc=u-boot@lists.denx.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox