Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Subject: Re: [PATCH v2 0/3] Add Aspeed SSIF BMC driver
To:     minyard@acm.org
Cc:     Rob Herring <robh+dt@kernel.org>, Joel Stanley <joel@jms.id.au>,
        Andrew Jeffery <andrew@aj.id.au>,
        Wolfram Sang <wsa@kernel.org>,
        Philipp Zabel <p.zabel@pengutronix.de>,
        openipmi-developer@lists.sourceforge.net,
        devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
        linux-aspeed@lists.ozlabs.org, linux-kernel@vger.kernel.org,
        linux-i2c@vger.kernel.org, openbmc@lists.ozlabs.org,
        Open Source Submission <patches@amperecomputing.com>,
        Phong Vo <phong@os.amperecomputing.com>,
        "Thang Q . Nguyen" <thang@os.amperecomputing.com>
References: <20210330141029.20412-1-quan@os.amperecomputing.com>
 <20210402142124.GV507977@minyard.net>
From:   Quan Nguyen <quan@os.amperecomputing.com>
Message-ID: <0e7fceff-4a01-5cb0-72eb-8b47d598f1c3@os.amperecomputing.com>
Date:   Wed, 7 Apr 2021 20:09:50 +0700
User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.15; rv:78.0)
 Gecko/20100101 Thunderbird/78.9.0
In-Reply-To: <20210402142124.GV507977@minyard.net>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Language: en-US
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-MessageSentRepresentingType: 1
Received: from [10.38.33.17] (118.69.219.201) by HK2P15301CA0008.APCP153.PROD.OUTLOOK.COM (2603:1096:202:1::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4042.3 via Frontend Transport; Wed, 7 Apr 2021 13:09:56 +0000
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id: 28630cce-ceb8-4ac2-f653-08d8f9c67380
X-MS-TrafficTypeDiagnostic: MWHPR01MB2734:
X-MS-Exchange-Transport-Forked: True
X-Microsoft-Antispam-PRVS: <MWHPR01MB273487ACA3C40B80CCF95C8DF2759@MWHPR01MB2734.prod.exchangelabs.com>
X-MS-Oob-TLC-OOBClassifiers: OLM:10000;
X-MS-Exchange-SenderADCheck: 1
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info: ETv8uoc6Dkbc7a2RBZXF5b4N2k+daHxi7QQaIIPTS9Avr+JkxHInO9D3RLbKi35tDMaEqZn6IeHxLDx8Qa0L7eFq7vxEGK+GiNTAtcHakXiO2ssMmZ+heEmICwGJjfSjrq6luuRc1EddOusCeIQAEpUEhLQoqmjFbeeRlhfgNEJJe8MiyulZPaEyD23ObhnWolXJ7ckVCYIP75e3E0gWzBBofjkcDtpATyn4LdeDCFANt4Ax2zcQBcmX/Vo6EkcFdrOEEBm1G1J2WaTc+rt0/J2ZFR5GARe3oJT1XaFHI5YzdR/JcGZdfRkFflMcf8aUK7qbyERwPo0P2dj+x47mUzz22Re5pl8YeiIwFBODIvLs1xwVMtn+hXKYOYBV0ORSMlZfKlCxVCno6/GsE3GtNt+mTjdiw2JZGx65O/Yt3yMDu04jUt7QBzjzedBBM+qVEPQV4zn+3MRd0kY6UxVXU1xw14gOz9/tYxxERB5jB8TuLMBMKNgyJQpUdlAUQN1iyX2tmoGyk3gul+u1exy73fHc0156p2ekbZQdoH66e0BHzDF1OnmOgFy2mOSdBdcMY+G9blJGzNK70VDAqvHw3iRsVDeP7Sh80EkO70eMfonIQSpabl8eN85qd+0O/IBRW6i2S4ctqdRV9E3c1hIi6FdQRWIkZrVgovKfiPHTVRUbqnC2pwRoBxrjN+XYXq3QmYKwbXwUdKUYsFBjABfKYyJDtqFuF5F4vqJWR8PPOP0/oiV6vj8ua13NuQ12giKEzEErMLU8a0u+Sh7VJbC71nBpQCFEgjhHmX0Mhq4mDO8YG1cUQju1rgidLhMTBriK1sjCpiUC8Otu9oGaNsQ3ew==
X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:MW2PR0102MB3482.prod.exchangelabs.com;PTR:;CAT:NONE;SFS:(4636009)(346002)(39840400004)(136003)(396003)(376002)(366004)(53546011)(31696002)(5660300002)(83380400001)(16526019)(186003)(6916009)(2906002)(16576012)(66556008)(8676002)(54906003)(66476007)(66946007)(38350700001)(6486002)(38100700001)(8936002)(7416002)(26005)(6666004)(966005)(4326008)(107886003)(52116002)(2616005)(956004)(316002)(86362001)(31686004)(478600001)(45980500001)(43740500002);DIR:OUT;SFP:1102;
X-MS-Exchange-AntiSpam-MessageData: =?utf-8?B?L21jd052WDV4NTE1Zm04a1FsOTIxTkVnTDVqY0xhQUpEYjhlU0NoOFVCZEJw?=
 =?utf-8?B?US9Qa0NnS0t4a20wM25iRGdyVHk4RjZ0bW45bW1CK3IxUUtnMWlDMmZpRzQy?=
 =?utf-8?B?dlp5ZXdXcFRDMEhsOGgrbU5HNEE2QzByY0l2QUE3Rll0TnhtTTdqcEYreHNq?=
 =?utf-8?B?Z2ZHYnFYNWp4K0pacDN2K2poMFc2SWFGdEVlNUJKcjdOQXhQeW9Oak9RMk1j?=
 =?utf-8?B?REtQL0llakFMU3hEUDdlRmxCVEZpL0Y3UWRhbVBxaXk3czJhY1Q4elkxY1Fv?=
 =?utf-8?B?WnBSc2NwdkhWc0FZd0RYVjVMSTBUbE9MU1BDZ2l0OVFsOVJibkdIVnBHYXU5?=
 =?utf-8?B?U05zQ1NBV2xCUGtmUlVwSy9RVFFMLzdNOUpaYkFGYjVURjkxOGp4UkxhU1Bq?=
 =?utf-8?B?RUEzTU5PZ1FVa3ZJK3E0d1lDeUFmcldSSXc3OUg0U2VOdVNXOExyM2NkZTIz?=
 =?utf-8?B?L1pIYlVGTmczNnluSzYrZk8yWGxJUklFOWtOUU9Pb0dZUG9yMkVTcStGZmwx?=
 =?utf-8?B?bEkrcFVFZzliazV2WDBPajVLV0FpUmhFWS8xc1ptaDFpdFVkOUZtY1JPSDRI?=
 =?utf-8?B?MDZ1S3Zock1vNytvQkRvR1dXVXFLWTRkM1pTaE9LeWtNb3lWUUZGVE9JNlFZ?=
 =?utf-8?B?VXBBdGhDYjNBYXhqMUZzQnFmZXRrbVkvU29sb3pUU3RtaVBQWFp3dS83VUJF?=
 =?utf-8?B?U1RIQjZJaURFSjFBVkdjMzdNYmZwNlJMQnVjWDFqTlRXanZZSVh2Z0hxc1BR?=
 =?utf-8?B?Q09yQ2ZjSXUzVWdpVmRPQ0l6Y1dubzllTWVhWW9mVHV1RVJ4Y1oxV3pybTNW?=
 =?utf-8?B?dE5DWkJyaEo2YjBpc1F3T1ptU0RxSVA4MkkxSWpnU0poSHdEaVh4Qm9nbmVa?=
 =?utf-8?B?SytmKzJEQ2QzMWNSNHcxSDA0Z0hVS0RPZmxhT1NhblVEbzhTTG1WalFmbDBp?=
 =?utf-8?B?QzVvZFIvWmNKQVZFR0NKUTdLblJqZ2RCcDgvK3c3Z0pDSWIwWnVJSnVRYVJz?=
 =?utf-8?B?SUNiZUNRdk80N3JOOVgwTDV1T1RUNjNyWFNURnIyRTdLZnpxNlNkclUyTGJh?=
 =?utf-8?B?K3psSTVOeFpaN3NhdEJET2FZY3VDT2x6QkxXeHp3a3JyWmtPZ29iYzBVRXRM?=
 =?utf-8?B?SDRMN3VSYS9KWXFxcmFKMXBIRDZ4QlZ2VVRXS0xlNlRxUTQ2RFBQeFlLSGxo?=
 =?utf-8?B?UUwzWDF2ZFdWWm9PUURIUXB0dmkwU0FOaXA5V2VEZG9NYTZLeGZxdEhpRWJq?=
 =?utf-8?B?QjE2WTF4L0NYclVUSXdMb3hUNnVOWWIwT00vbGN2cittMkFmbndENldHN1RE?=
 =?utf-8?B?QmdUNjJzak15Qmhmenl6NEJBc2xXS3V5NFJuZUV4aWloazVyRlFvTzAyUFFZ?=
 =?utf-8?B?WDM0Q2hxeVZSL1JTcXZTaXhrVE5qbDhhNmJYczV1Vm1LYWhMelBIamF5SnAz?=
 =?utf-8?B?eXFwZ3lyOFJRZ0o5ZUp3UExkSEZkbnlUYm1SNHdlYUxYT0VYdjUzWEpDZnVU?=
 =?utf-8?B?bU13cSszSlA4dWpqV04vbTVaS2ppRm1xZFhkcEJYY0lxTU42TmNiUmltQjRp?=
 =?utf-8?B?bGhUSjdqSjNrYlVQREFlNXdWSUJQSXV3VlRMeUZnRFd2Z1k3VHpEeCtXWTRx?=
 =?utf-8?B?OXVlSFNpVGljUUhlR3VZZ2ZpT3I3WGJoa3NZRmVGQ1VDYWhXbnoydDJXM2o4?=
 =?utf-8?B?aUxzYWRoSzlOU2x1Q21XYTFRWTJXTkE0aWFZZWNKOStkNm1udkNTQzhHNVhU?=
 =?utf-8?Q?GP12p8n/zESfaNVRUqk4twG91IxytSGRqsiKmyG?=
X-OriginatorOrg: os.amperecomputing.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 28630cce-ceb8-4ac2-f653-08d8f9c67380
X-MS-Exchange-CrossTenant-AuthSource: MW2PR0102MB3482.prod.exchangelabs.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 07 Apr 2021 13:10:01.3735
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3bc2b170-fd94-476d-b0ce-4229bdc904a7
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: Smz3mGe5/PgE/Tbhmfp7KovILLQx3GiGymFDRx4aGKKh0NnPuVGrtK+Ik7/UXfYYyWYpw9Zsxm2NK03auY8+XWhiHfo7SzajzQd872qGXO8=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: MWHPR01MB2734
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Corey,

Thank you for reviewing
I'll put my respond inline below.

-Quan

On 02/04/2021 21:21, Corey Minyard wrote:
> On Tue, Mar 30, 2021 at 09:10:26PM +0700, Quan Nguyen wrote:
>> This series add support for the Aspeed specific SSIF BMC driver which
>> is to perform in-band IPMI communication with the host in management
>> (BMC) side.
> 
> I don't have any specific feedback for this, but I'm wondering if it's
> really necessary.
> 
> Why can't the BMC just open the I2C device and use it?  Is there any
> functionality that this provides that cannot be accomplished from
> userland access to the I2C device?  I don't see any.
>
> If it tied into some existing framework to give abstract access to a BMC
> slave side interface, I'd be ok with this.  But I don't see that.
> 

The SSIF at the BMC side acts as an I2C slave and we think that the 
kernel driver is unavoidable to handle the I2c slave events 
(https://www.kernel.org/doc/html/latest/i2c/slave-interface.html)

And to make it works with existing OpenBMC IPMI stack, a userspace part, 
ssifbridge, is needed (https://github.com/openbmc/ssifbridge). This 
ssifbridge is to connect this driver with the OpenBMC IPMI stack so the 
IPMI stack can communicate via SSIF channel in similar way that was 
implemented with BT and KCS (ie: btbridge/kcsbridge and its corespondent 
kernel drivers (https://github.com/openbmc/btbridge and 
https://github.com/openbmc/kcsbridge))

> Unless there is a big need to have this in the kernel, I'm against
> including this and would suggest you do all this work in userland.
> Perhaps write a library.  Sorry, but I'm trying to do my part to reduce
> unnecessary things in the kernel.
> 
> Thanks,
> 
> -corey
>